From: Eric Blake <ebl...@redhat.com>

Documenting that we should not add new lines to work around SPF
rewrites sounds foreboding; the intent is instead that new lines here
are okay, but indicate a second problem elsewhere in our build process
that we should also consider fixing at the same time, to keep the
section from growing without bounds.

Mentioning DMARC alongside SPF may also help people grep for this
scenario, as well as documenting the 'git config' workaround that can
be used by submitters to avoid the munging issue in the first place.

Fixes: 3bd2608d ("maint: Add .mailmap entries for patches claiming list 
authorship")
CC: Andrey Drobyshev <andrey.drobys...@virtuozzo.com>
Cc: Peter Maydell <peter.mayd...@linaro.org>
Signed-off-by: Eric Blake <ebl...@redhat.com>
---

I'm sending this email with a temporary 'git config sendemail.from
ebl...@redhat.com', to see if the added advice actually adds the extra
line. It did not show up in my editor window, though, so this patch
may need further tweaking to get the instructions right.  Since I
don't normally suffer from SPF/DMARC munging, I may not be the best
person to test the right workaround.  Or maybe 'git config' does not
yet have the right workaround already available as a turnkey solution
we can suggest.

 .mailmap | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/.mailmap b/.mailmap
index 64ef9f4de6f..6fb8202715a 100644
--- a/.mailmap
+++ b/.mailmap
@@ -40,7 +40,16 @@ Nick Hudson <hn...@vmware.com> hn...@vmware.com 
<hn...@vmware.com>
 # for the cvs2svn initialization commit e63c3dc74bf.

 # Next, translate a few commits where mailman rewrote the From: line due
-# to strict SPF, although we prefer to avoid adding more entries like that.
+# to strict SPF and DMARC.  Usually, our build process should be flagging
+# commits like these before maintainer merges; if you find the need to add
+# a line here, please also report a bug against the part of the build
+# process that let the mis-attribution slip through in the first place.
+#
+# If the mailing list munges your emails, consider using something like
+# 'git config --global sendemail.from your.em...@example.com'
+# to add an extra From: line in the body of your email that takes
+# precedence over any munged From: in the mail's headers.
+# See https://lists.openembedded.org/g/openembedded-core/message/166515
 Ed Swierk <eswi...@skyportsystems.com> Ed Swierk via Qemu-devel 
<qemu-devel@nongnu.org>
 Ian McKellar <ianl...@google.com> Ian McKellar via Qemu-devel 
<qemu-devel@nongnu.org>
 Julia Suvorova <jus...@mail.ru> Julia Suvorova via Qemu-devel 
<qemu-devel@nongnu.org>
-- 
2.41.0


Reply via email to