From: Don Slutz <dslutz@verizon.com>
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: Matthew Daley <mattjd@gmail.com>, Keir Fraser <keir@xen.org>,
Jan Beulich <JBeulich@suse.com>, Don Slutz <dslutz@verizon.com>,
xen-devel@lists.xen.org
Subject: Re: [PATCH v3 3/3] get_maintainer.pl: Adjust to review comments
Date: Tue, 5 Nov 2013 09:53:42 -0500 [thread overview]
Message-ID: <52790676.5030206@terremark.com> (raw)
In-Reply-To: <1383663021.13961.87.camel@kazak.uk.xensource.com>
On 11/05/13 09:50, Ian Campbell wrote:
> On Tue, 2013-11-05 at 09:43 -0500, Don Slutz wrote:
>> On 11/05/13 09:19, Ian Campbell wrote:
>>
>>> On Tue, 2013-11-05 at 09:11 -0500, Don Slutz wrote:
>>>> From: Don Slutz <dslutz@verizon.com>
>>> Please can you list the specific changes for the changelog. No need to
>>> resend, I'll fold the list into the commit message.
>> Not sure how much you want.
> Sorry, I meant a short bullet list. I think what you gave could be
> boiled down to:
>
> * Disable git fallback by default: it has a tendency to mail
> anyone who did a single oneline change and should not be
> necessary for a project of Xen's size.
> * Disable rolestats: Makes cut-and-paste from the output into the
> commit message easy.
> * Drop "THE REST" fallback: Don't spam Keir *too* much.
>
> Is that an accurate summary? If so it can be inserted into the commit
> log.
Looks great to me.
-Don Slutz
>>> Just to confirm: This doesn't suppress any xen-devel@ CC, which should
>>> always be there, does it?
>> Nope. It only drops the e-mail (currently Keir Fraser) for
>> "supporter:THE REST" role when any other role is found.
> Ah, because M and L are different tags. Clever.
>
> Thanks
> Ian.
>
>
>
next prev parent reply other threads:[~2013-11-05 14:53 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-05 14:11 [PATCH v3 0/3] Add get_maintainer.pl to Xen Don Slutz
2013-11-05 14:11 ` [PATCH v3 1/3] Add linux version of get_maintainer.pl Don Slutz
2013-11-05 14:11 ` [PATCH v3 2/3] get_maintainer.pl: Convert to Xen tree Don Slutz
2013-11-05 14:11 ` [PATCH v3 3/3] get_maintainer.pl: Adjust to review comments Don Slutz
2013-11-05 14:19 ` Ian Campbell
2013-11-05 14:43 ` Don Slutz
2013-11-05 14:50 ` Ian Campbell
2013-11-05 14:53 ` Don Slutz [this message]
2013-11-05 14:22 ` [PATCH v3 0/3] Add get_maintainer.pl to Xen Ian Campbell
2013-11-19 14:03 ` Ian Campbell
2013-11-19 15:50 ` Don Slutz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=52790676.5030206@terremark.com \
--to=dslutz@verizon.com \
--cc=Ian.Campbell@citrix.com \
--cc=JBeulich@suse.com \
--cc=keir@xen.org \
--cc=mattjd@gmail.com \
--cc=xen-devel@lists.xen.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.