qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 0/7] virtio: avoid inappropriate QEMU terminatio


From: Greg Kurz
Subject: Re: [Qemu-devel] [PATCH 0/7] virtio: avoid inappropriate QEMU termination
Date: Wed, 21 Sep 2016 16:34:34 +0200

On Wed, 21 Sep 2016 22:01:30 +0800
Fam Zheng <address@hidden> wrote:

> On Wed, 09/21 15:44, Greg Kurz wrote:
> > On Wed, 21 Sep 2016 06:35:04 -0700 (PDT)
> > address@hidden wrote:
> >   
> > > Hi,
> > > 
> > > Your series failed automatic build test. Please find the testing commands 
> > > and
> > > their output below. If you have docker installed, you can probably 
> > > reproduce it
> > > locally.
> > >   
> > 
> > Heh... of course patchew doesn't know about Stefan's series. :)  
> 
> Usually those patchsets fail to apply and patchew keeps quiet. This series is
> the first victim of a exception, seemingly. :)
> 

This patchset applies indeed but calls a function that only exists in Stefan's
series. It is of course impossible for patchew to guess that :)

The default behaviour of keeping quiest when patchsets fail to apply is good
IMHO.

> > 
> > Is there an appropriate way to avoid complaints when sending a patchset that
> > isn't based on QEMU master ?  
> 
> Currently we don't, but we can invent one. Like:
> 
> Base: http://github.com/someone/qemu branch
> 
> for the "base on maintainer tree" case, or just inform patchew to "pull"
> instead of apply:
> 
> Git: http://github.com/someone/qemu topic
> 
> for the "base on the other series" case.
> 
> Supporting this is not complicated, the tricky thing is to remember to put 
> that
> magic in cover letters every time a series doesn't base on master.
> 
> Any thoughts on the harder question? :)
> 

Be smart enough ? :)

> Fam

Cheers.

--
Greg



reply via email to

[Prev in Thread] Current Thread [Next in Thread]