From: Christian Schoenebeck <qemu_oss@crudebyte.com>
To: qemu-devel@nongnu.org
Cc: Jeff King <peff@peff.net>,
Christian Schoenebeck <qemu_oss@crudebyte.com>,
berrange@redhat.com, stefanha@gmail.com,
Greg Kurz <groug@kaod.org>,
git@vger.kernel.org, antonios.motakis@huawei.com,
dgilbert@redhat.com, Ian Kelling <iank@fsf.org>
Subject: git format.from (was: 9p: Fix file ID collisions)
Date: Tue, 24 Sep 2019 11:03:38 +0200 [thread overview]
Message-ID: <3312839.Zbq2WQg2AT@silver> (raw)
In-Reply-To: <20190923222415.GA22495@sigill.intra.peff.net>
On Dienstag, 24. September 2019 00:24:15 CEST Jeff King wrote:
> > On the other hand, considering the already existing --from argument and
> > "format.from" config option:
> > https://git-scm.com/docs/git-config#Documentation/git-config.txt-formatfro
> > m
> >
> > Wouldn't it make sense to just drop the currently existing sender !=
> > author
> > string comparison in git and simply always add the "From:" line to the
> > email's body if "format.from yes" is used, instead of introducing a
> > suggested 2nd (e.g. "always-from") option? I mean sure automatically
> > removing redundant information in the generated emails if sender ==
> > author sounds nice on first thought, but does it address anything useful
> > in practice to justify introduction of a 2nd related option?
>
> Yes, the resulting mail would be correct, in the sense that it could be
> applied just fine by git-am. But I think it would be uglier. IOW, I
> consider the presence of the in-body From to be a clue that something
> interesting is going on (like forwarding somebody else's patch). So from
> my perspective, it would just be useless noise. Other communities may
> have different opinions, though (I think I have seen some kernel folks
> always including all of the possible in-body headers, including Date).
> But it seems like it makes sense to keep both possibilities.
Exactly, current git behaviour is solely "prettier" (at first thought only
though), but does not address anything useful in real life.
Current git behaviour does cause real life problems though: Many email lists
are munging emails of patch senders whose domain is configured for requiring
domain's emails being DKIM signed and/or being subject to SPF rules (a.k.a
DMARC). So original sender's From: header is then automatically replaced by an
alias (by e.g. mailman): https://en.wikipedia.org/wiki/DMARC#From:_rewriting
For instance the email header:
From: "Bob Bold" <bold@foo.com>
is automatically replaced by lists by something like
From: "Bob Bold via Somelist" <somelist@gnu.org>
And since git currently always drops the From: line from the email's body if
sender == author, as a consequence maintainers applying patches from such
lists, always need to rewrite git history subsequently and have to replace
patch author's identity manually for each commit to have their correct, real
email address and real name in git history instead of something like
"Bob Bold via Somelist" <somelist@gnu.org>
So what do you find "uglier"? I prefer key info not being lost as default
behaviour. :-)
Best regards,
Christian Schoenebeck
next prev parent reply other threads:[~2019-09-24 9:03 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <cover.1566503584.git.qemu_oss@crudebyte.com>
[not found] ` <46fe6c73-961f-d72a-77de-88491b6f223c@redhat.com>
[not found] ` <4642438.ai5u8AxThJ@silver>
[not found] ` <1897173.eDCz7oYxVq@silver>
2019-09-09 14:05 ` [Qemu-devel] [PATCH v6 0/4] 9p: Fix file ID collisions Eric Blake
2019-09-09 14:25 ` Jeff King
2019-09-23 11:19 ` Christian Schoenebeck
2019-09-23 22:24 ` Jeff King
2019-09-24 9:03 ` Christian Schoenebeck [this message]
2019-09-24 21:36 ` git format.from (was: 9p: Fix file ID collisions) Jeff King
2019-09-09 18:41 ` [Qemu-devel] [PATCH v6 0/4] 9p: Fix file ID collisions Junio C Hamano
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
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3312839.Zbq2WQg2AT@silver \
--to=qemu_oss@crudebyte.com \
--cc=antonios.motakis@huawei.com \
--cc=berrange@redhat.com \
--cc=dgilbert@redhat.com \
--cc=git@vger.kernel.org \
--cc=groug@kaod.org \
--cc=iank@fsf.org \
--cc=peff@peff.net \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@gmail.com \
/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.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).