Re: [PULL 00/33] Trivial branch patches

["Daniel P. Berrangé" <berrange@redhat.com>]

On Tue, Mar 10, 2020 at 01:00:12PM +0100, Philippe Mathieu-Daudé wrote:
> On 3/10/20 12:45 PM, Kevin Wolf wrote:
> > Am 10.03.2020 um 09:02 hat Laurent Vivier geschrieben:
> > > Le 09/03/2020 à 20:30, Philippe Mathieu-Daudé a écrit :
> > > > On 3/9/20 8:17 PM, Michael S. Tsirkin wrote:
> > > > > On Mon, Mar 09, 2020 at 04:08:04PM +0100, Laurent Vivier wrote:
> > > > > > The following changes since commit
> > > > > > 7a5853cec479a448edae0fb2aaf4e2f78c9c774d:
> > > > > > 
> > > > > >     Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into
> > > > > > staging (2020-03-09 10:32:53 +0000)
> > > > > > 
> > > > > > are available in the Git repository at:
> > > > > > 
> > > > > >     git://github.com/vivier/qemu.git tags/trivial-branch-pull-request
> > > > > > 
> > > > > > for you to fetch changes up to 916c92503bd5348a33e561db600d8894bde636bb:
> > > > > > 
> > > > > >     monitor/hmp-cmds: Remove redundant statement in
> > > > > > hmp_rocker_of_dpa_groups() (2020-03-09 15:59:31 +0100)
> > > > > 
> > > > > 
> > > > > Could you avoid CC everyone involved on the whole PULL req?
> > > > > I was involved in a patch or two and don't really need to see
> > > > > the whole series. Just the cover and the relevant patches
> > > > > would be enough - if I do want it there's always lore.
> > > > 
> > > > I suppose Laurent used git-publish, which has this limitation.
> > > > 
> > > 
> > > Yes, I use git publish and CC are automatically added.
> > > 
> > > Philippe, you told me some time ago you have a patch to fix this
> > > behaviour and only CC people on a patch base. Is it available?
> 
> I guess you mean https://github.com/philmd/git-publish/commit/5bab6e2cc77, I
> haven't improved it but plan to spend time on it during my next holidays.
> 
> > Actually, I don't think it's even necessary to CC anyone at all (except
> > the relevant lists) for pull requests. These patches were already
> > reviewed.
> 
> Well as a contributor I find useful to get notified when patches are merged.

Yeah, I tend to agree with that - it is good to know when something is
merged, more so if you are an infrequent contributor to an area of code
or QEMU in general.

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|