Re: [PATCH v3 13/13] format-patch: learn --infer-cover-subject option

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

 



Philip Oakley <philipoakley@iee.email> writes:

> Perhaps the `--infer-cover-subject` the config option needs to be
> multi-valued to include:
>      "subject" (always expect short first lines) or
>      "message" (always the long paragraph description, still use
> ***Subject Here***),
>      with the "true" being used when expecting both as previously
> described.

The idea to have three choices feels that this is getting better,
but I notice that the choice is no longer about "subject".

I've always felt that the name of this option is way suboptimal.
One reason is because the option only says it is about the subject
of the cover (letter), and the verb "infer" conveys almost no
information---especially it does not say anything about what affects
the inference (hint: the branch description value gets used, in a
single hardcoded ways right now, but now with the patch we have a
choice to control how it gets used).







[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]

  Powered by Linux