[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/5] Add a git-publish configuration file
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH 1/5] Add a git-publish configuration file |
Date: |
Wed, 14 Feb 2018 13:34:34 +0000 |
User-agent: |
Mutt/1.9.2 (2017-12-15) |
On Wed, Feb 14, 2018 at 09:01:11AM +0800, Fam Zheng wrote:
> On Tue, 02/13 18:09, Daniel P. Berrangé wrote:
> > On Tue, Feb 13, 2018 at 05:34:25PM +0000, Stefan Hajnoczi wrote:
> > > +[gitpublishprofile "block"]
> > > +base = master
> > > +prefix = PATCH
> > > +to = address@hidden
> > > +cc = address@hidden
> > > +cccmd = scripts/get_maintainer.pl --noroles --norolestats --nogit
> > > --nogit-fallback 2>/dev/null
> >
> > Why is a custom entry needed for block here (and other things
> > below). Won't running get_maintainer.pl already correctly
> > report when a patch needs cc'ing to address@hidden
> > based on MAINTAINER rules ?
>
> Yeah, dropping them should be fine. What do you think, Stefan?
scripts/get_maintainer.pl only works when patches touch files associated
with address@hidden in MAINTAINERS. It would be
counter-intuitive for --profile block to not include
address@hidden, so I think these entries do serve a purpose.
On the other hand, I personally just use --cc qemu-block instead of a
profile for this. I'll probably only use the default profile.
Fam, if you want it dropped, let me know. Otherwise, we can leave it
in if there is no further discussion.
Stefan
signature.asc
Description: PGP signature
- [Qemu-devel] [PATCH 0/5] Block patches, Stefan Hajnoczi, 2018/02/13
- [Qemu-devel] [PATCH 3/5] vl: pause vcpus before stopping iothreads, Stefan Hajnoczi, 2018/02/13
- [Qemu-devel] [PATCH 1/5] Add a git-publish configuration file, Stefan Hajnoczi, 2018/02/13
- Re: [Qemu-devel] [PATCH 1/5] Add a git-publish configuration file, Paolo Bonzini, 2018/02/14
- Re: [Qemu-devel] [PATCH 1/5] Add a git-publish configuration file, Paolo Bonzini, 2018/02/14
- Re: [Qemu-devel] [PATCH 1/5] Add a git-publish configuration file, Fam Zheng, 2018/02/25
Re: [Qemu-devel] [PATCH 1/5] Add a git-publish configuration file, Philippe Mathieu-Daudé, 2018/02/14
[Qemu-devel] [PATCH 2/5] README: Document 'git-publish' workflow, Stefan Hajnoczi, 2018/02/13
[Qemu-devel] [PATCH 5/5] misc: fix spelling, Stefan Hajnoczi, 2018/02/13
[Qemu-devel] [PATCH 4/5] ratelimit: don't align wait time with slices, Stefan Hajnoczi, 2018/02/13