Josh Triplett [off-list ref] writes:
quoted
While I understand what you said about imap-send, I really would
feel better if this was optional. Do not change the default
output format, please.
So rather than the --no-thread option provided in the second patch of
this series, you'd prefer a --thread option to enable setting the
In-Reply-To/References headers?
Eh, that's not what I meant.
I do not mind the code you added to log-tree.c and revision.h,
and honestly I do not care which of threading or non-threading
mode is the default, although I think your explanation that it
parallels what send-email does makes a lot of sense.
But I do mind that the code added by the first patch to
cmd_format_patch runs by default, and worse yet, there is no
option turn it off. Setting message_id and ref_message_id in
rev_info struct should be something the end user should ask for
explicitly by invoking the command with an option, perhaps
--with-message-id, which you probably would also want to turn on
when any of --no-thread, --thread or --initial-reply-to options
are given.
-