From: Geert Uytterhoeven <geert@linux-m68k.org>
To: Marek Vasut <marek.vasut@gmail.com>
Cc: Eduardo Valentin <edubezval@gmail.com>,
Linux PM list <linux-pm@vger.kernel.org>,
Linux-Renesas <linux-renesas-soc@vger.kernel.org>,
Marek Vasut <marek.vasut+renesas@gmail.com>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
Wolfram Sang <wsa+renesas@sang-engineering.com>,
Zhang Rui <rui.zhang@intel.com>,
Geert Uytterhoeven <geert+renesas@glider.be>
Subject: Re: [PATCH 1/6] thermal: split thermal_zone_of_sensor_register{,_param}()
Date: Sun, 16 Dec 2018 21:08:00 +0100 [thread overview]
Message-ID: <CAMuHMdUnH5-YeGcQdbyJP2KNZFS=zxhSb7PZ1ZhLCqFxiw=TnQ@mail.gmail.com> (raw)
In-Reply-To: <04bcb195-0668-4d8d-1f86-c53c3a142183@gmail.com>
Hi Marek,
On Sun, Dec 16, 2018 at 6:49 PM Marek Vasut <marek.vasut@gmail.com> wrote:
> On 12/16/2018 06:42 PM, Geert Uytterhoeven wrote:
> > On Sun, Dec 16, 2018 at 6:25 PM Marek Vasut <marek.vasut@gmail.com> wrote:
> >> On 12/16/2018 09:39 AM, Geert Uytterhoeven wrote:
> >>> On Sat, Dec 15, 2018 at 9:13 PM Marek Vasut <marek.vasut@gmail.com> wrote:
> >>>> On 12/15/2018 09:00 PM, Geert Uytterhoeven wrote:
> >>>>> On Sat, Dec 15, 2018 at 8:07 PM Marek Vasut <marek.vasut@gmail.com> wrote:
> >>>>>> On 12/15/2018 07:54 PM, Geert Uytterhoeven wrote:
> >>>>>>> On Sat, Dec 15, 2018 at 7:49 PM Marek Vasut <marek.vasut@gmail.com> wrote:
> >>>>>>>> On 12/15/2018 07:47 PM, Geert Uytterhoeven wrote:
> >>>>>>>>> On Sat, Dec 15, 2018 at 7:38 PM Marek Vasut <marek.vasut@gmail.com> wrote:
> >>>>>>>>>> On 12/15/2018 06:23 PM, Eduardo Valentin wrote:
> >>>>>>>>>>> On Wed, Dec 12, 2018 at 02:49:22AM +0100, Marek Vasut wrote:
> >>>>>>>>>>>> Introduce new thermal_zone_of_sensor_register_params() function, which
> >>>>>>>>>>>> allows passing struct thermal_zone_params into it and convert original
> >>>>>>>>>>>> thermal_zone_of_sensor_register() to call it with params set to NULL.
> >>>>>>>>>>>>
> >>>>>>>>>>>> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
> >>>>>>>>>>>
> >>>>>>>>>>> Git complains about mismatch between From: and this SOB.
> >>>>>>>>>>
> >>>>>>>>>> I recall a discussion about gmail stripping the +foo tags from email
> >>>>>>>>>> addresses. I can add a From: tag into the patch to override this
> >>>>>>>>>> braindeath, or is there a better solution ?
> >>>>>>>>>
> >>>>>>>>> Run the "git format-patch" command from a git repo where user.email
> >>>>>>>>> is marek.vasut@gmail.com, so it will retain the original From: tag in the
> >>>>>>>>> email body, as it is different?
> >>>>>>>>
> >>>>>>>> I can also manually patch the From tags or add them, but it's all
> >>>>>>>> workarounds.
> >>>>>>>
> >>>>>>> Use a different outgoing email server? I use my ISP's.
> >>>>>>
> >>>>>> Or maybe it'd make sense to fix git to handle the +tags correctly ?
> >>>>>
> >>>>> What needs to be fixed?
> >>>>> If user.email != From, git format-patch generates a From: header, else
> >>>>> it doesn't. Doesn't that make sense?
> >>>>
> >>>> I believe the complaint here is that email address in From does not
> >>>> match email address in SoB line, because some SMTP servers scrub the
> >>>> +foo tag from From: and not from SoB-line . And yet, the SoB line is
> >>>
> >>> Some SMTP servers (Hi Gmail!) scrub the +foo tag from the From: _header_
> >>> in the email. As the SoB is in the _body_ it is not affected.
> >>> Hence the solution is to include the correct From: in the _body_.
> >>
> >> This is basically what I said.
> >>
> >>> Git actually does that automatically, assumed your user.email config matches
> >>> the From: address that is used in your outgoing email delivery path (i.e. the
> >>> scrubbed one, when using Gmail's SMTP server).
> >>> If you lie to git in your user.email config, git cannot do the right
> >>> thing, obviously.
> >>
> >> My git user.email obviously matches the From: field , before the
> >> scrubbing, which I believe is the correct thing to do.
> >
> > I disagree, because that is not how the emails are actually going out from the
> > SMTP server you are using.
>
> Can you summarize, clearly, what you believe is the right thing to
> configure and where ?
According to git-send-email(1), you can either pass your scrubbed email
address to --from, or configure it in the sendemail.from config option.
Does that work for you?
> >>>> from the same person/email address as the email address in From, so they
> >>>> are equal.
> >>>
> >>> If they differ, they are not equal ;-)
> >>
> >> Depends on how you define 'equal' . Here I think foo+bar@example.com
> >> should be considered equal to foo@example.com .
> >
> > That is domain-specific knowledge, which you cannot rely upon.
> >> Aha, so maybe that enhancement needs further enhancement to scrub the
> >> +tags before the check ?
> >
> > Again, that is domain-specific knowledge, which you cannot rely upon.
>
> How so, please elaborate .
In general, you cannot assume the "+foo" part can be ignored. Only the sender
knows.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
next prev parent reply other threads:[~2018-12-16 20:08 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-12 1:49 [PATCH 0/6] thermal: Align devm_thermal_zone_{device,of_sensor}_register Marek Vasut
2018-12-12 1:49 ` [PATCH 1/6] thermal: split thermal_zone_of_sensor_register{,_param}() Marek Vasut
2018-12-15 17:23 ` Eduardo Valentin
2018-12-15 18:38 ` Marek Vasut
2018-12-15 18:47 ` Geert Uytterhoeven
2018-12-15 18:49 ` Marek Vasut
2018-12-15 18:54 ` Geert Uytterhoeven
2018-12-15 19:07 ` Marek Vasut
2018-12-15 20:00 ` Geert Uytterhoeven
2018-12-15 20:13 ` Marek Vasut
2018-12-16 8:39 ` Geert Uytterhoeven
2018-12-16 17:25 ` Marek Vasut
2018-12-16 17:42 ` Geert Uytterhoeven
2018-12-16 17:48 ` Marek Vasut
2018-12-16 20:08 ` Geert Uytterhoeven [this message]
2018-12-16 20:43 ` Marek Vasut
2018-12-17 13:26 ` Geert Uytterhoeven
2018-12-17 13:28 ` Marek Vasut
2018-12-17 13:36 ` Geert Uytterhoeven
2018-12-17 13:41 ` Marek Vasut
2018-12-17 14:15 ` Geert Uytterhoeven
2018-12-17 15:52 ` Marek Vasut
2018-12-12 1:49 ` [PATCH 2/6] thermal: split devm_thermal_zone_of_sensor_register{,_param}() Marek Vasut
2018-12-12 1:49 ` [PATCH 3/6] thermal: Register hwmon in thermal_zone_of_sensor_register_param() Marek Vasut
2018-12-12 1:49 ` [PATCH 4/6] thermal: stm32: Convert to devm_thermal_zone_of_sensor_register_params() Marek Vasut
2018-12-12 1:49 ` [PATCH 5/6] thermal: rcar_thermal: " Marek Vasut
2018-12-12 1:49 ` [PATCH 6/6] thermal: rcar_gen3_thermal: Register hwmon sysfs interface Marek Vasut
2018-12-14 5:28 ` kbuild test robot
2018-12-15 2:02 ` [PATCH 0/6] thermal: Align devm_thermal_zone_{device,of_sensor}_register Eduardo Valentin
2018-12-15 2:08 ` Marek Vasut
2018-12-15 17:23 ` Eduardo Valentin
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAMuHMdUnH5-YeGcQdbyJP2KNZFS=zxhSb7PZ1ZhLCqFxiw=TnQ@mail.gmail.com' \
--to=geert@linux-m68k.org \
--cc=daniel.lezcano@linaro.org \
--cc=edubezval@gmail.com \
--cc=geert+renesas@glider.be \
--cc=linux-pm@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=marek.vasut+renesas@gmail.com \
--cc=marek.vasut@gmail.com \
--cc=rui.zhang@intel.com \
--cc=wsa+renesas@sang-engineering.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.
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 NNTP newsgroup(s).