Re: [PATCH v1] docs: reminder to not expose potentially private email addresses

[Thorsten Leemhuis <linux@leemhuis.info>]

On 13.11.24 12:40, Mauro Carvalho Chehab wrote:
> Em Wed, 13 Nov 2024 11:59:39 +0100
> Simona Vetter <simona.vetter@ffwll.ch> escreveu:
>> On Wed, 13 Nov 2024 at 11:55, Thorsten Leemhuis <linux@leemhuis.info> wrote:
>>> On 13.11.24 11:26, Laurent Pinchart wrote:  
>>>>> +Note, remember to respect other people's privacy when adding these tags:
>>>>> +
>>>>> + - Only specify email addresses, if owners explicitly permitted their use or
>>>>> +   are fine with exposing them to the public based on previous actions found in
>>>>> +   the lore archives. 
> 
> There is no comma between "addresses" and "if".
> 
> "previous actions" sounds a little to vague. Also, the text doesn't cover
> everything, as lore archives may contain gaps.  I would, instead be clear:
> 
> 	 - Only specify email addresses if owners explicitly permitted their use or
> 	   if such e-mail was previously used publicly for Linux contributions, which
> 	   can be checked by looking at the lore archives and at the git log. 
> 
> I added "git log there" because, in practice, nobody has the time to double-check
> what e-mails are public: developers rely that scripts/checkpatch.pl will
> check git log when creating the Cc: list.

Thx. I went with a slightly changed variant for now, hope that's okay:

"""
Only specify email addresses if owners explicitly permitted their use or
if the addresses have previously been used publicly for contributions to
the Linux kernel found in the lore archives or the commit history.
"""

Regarding the other points Simona and Laurent brought up: many thx for
that, I will take a closer look soon (I need to check if the suggested
approaches really work; while at it I also want to check if
5.Posting.rst mentions the "no tag forgeries" aspect at all; from a
quick look that seems to be missing, so I might add a patch that puts it
in an appropriate place).

Ciao, Thorsten