Re: [PATCH] Add 'git format-patch --to=' option and 'format.to' configuration variable.

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

 



On Tue, 16 Feb 2010, Junio C Hamano wrote:

> I don't _mind_ adding such a config (I didn't check the patch text,
> though), but one thing I don't understand is what kind of workflow do
> people who use format.cc, format.to, --cc and --to are using.

In the commit that add '--cc', Daniel Barkalow wrote:
> When you have particular reviewers you want to sent particular series
> to, it's nice to be able to generate the whole series with them as
> additional recipients, without configuring them into your general
> headers or adding them by hand afterwards.

In the commit that added 'format.cc' Miklos Vajna wrote:
> Some projects prefer to always CC patches to a given mailing list. In
> these cases, it's handy to configure that address once.

I think it's weired to have an option & config variable for Cc and not To.

> My impression has always been that you drive your _MUA_ to set these
> values, and MUAs are either (1) not so cooperating to read these headers
> from format-patch output and use them to decide where to send mails from,
Any MUA that does not cooperate with reading the To header is going to
give trouble with any header (e.g. Subject, Date).

> or (2) is a git-send-email which is customizable already per repository so
> you do not have to do anything funky when running format-patch.
Haven't got around to using git-send-email yet!

> [...] how does having To:
> and Cc: header pregenerated by format-patch help you?
Mostly having to remember/lookup the email address.

-- 
Steven
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]