[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: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH 1/5] Add a git-publish configuration file |
Date: |
Sun, 25 Feb 2018 14:04:36 +0800 |
User-agent: |
Mutt/1.9.2 (2017-12-15) |
On Wed, 02/14 15:08, Paolo Bonzini wrote:
> On 13/02/2018 19:09, Daniel P. Berrangé wrote:
> > 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 ?
>
> Sometimes you want to CC block people even on a non-block patch, for
> example if you want their opinion about using the API.
>
> We should probably add at least arm and ppc profiles, too.
>
> > --nogit --nogit-fallback
>
> QEMU's getmaintainer (unlike the Linux version!) already tries to avoid
> falling back to git if a maintainer is listed. Are these options necessary?
>
Falling back to git is usually more noisy than useful even when a maintainer is
not listed, in my past experience. That's why I don't use it myself and propose
such here. But I'm open to different opinions.
Fam
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
Re: [Qemu-devel] [PATCH 0/5] Block patches, Fam Zheng, 2018/02/13