Re: [PATCH RESEND] send-email: add 'specify-author' option

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

 



On Fri, 06 Apr 2012 15:30:32 -0700, Junio C Hamano wrote:

> Felipe Contreras <felipe.contreras@xxxxxxxxx> writes:
>
>> I can't foresee that, but I guess we can do it anyway. So which would
>> be the fields to repeat? From, Date, and Subject?
> 
> I would say From and Subject are equally worth considering.

Famous last words: "I can't foresee that."

My impression from Junio's initial comment:

>>> The first paragraph of the proposed commit log message states the problem
>>> it tries to address very clearly, which is good, but is "From: " the only
>>> thing that needs this?  I am wondering if this should be named and behave
>>> more like "--duplicate-header" or "--in-body-header".

is that a more general solution should be sought, particularly because we
indeed can't foresee people's needs. I envisioned `--in-body-header' taking
as an argument a colon-separated list of headers that should be inlined;
let the user mix and match as he pleases, and save us trouble in the
future when new headers pop up (I can't forsee that happening, but, well,
who knows?).

Naturally, a configuration variable could be useful.

Sincerely,
Michael Witten
--
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]