All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jonathan Corbet <corbet@lwn.net>
To: Thorsten Leemhuis <linux@leemhuis.info>
Cc: workflows@vger.kernel.org, linux-doc@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	Simona Vetter <simona.vetter@ffwll.ch>,
	Mauro Carvalho Chehab <mchehab+huawei@kernel.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Shuah Khan <skhan@linuxfoundation.org>
Subject: Re: [PATCH v4] docs: clarify rules wrt tagging other people
Date: Tue, 18 Feb 2025 13:42:03 -0700	[thread overview]
Message-ID: <875xl7nfxg.fsf@trenco.lwn.net> (raw)
In-Reply-To: <8b87b297-b68b-4276-95ae-e04650c3360f@leemhuis.info>

Sorry, fell behind on things again...

Thorsten Leemhuis <linux@leemhuis.info> writes:

>> - It would be awfully nice if we could provide this advice in exactly
>>   one place in the document.  This is one of our most important docs,
>>   and it is far too long to expect new contributors to read through and
>>   absorb.  Avoiding making it longer and more repetitive would be
>>   better, if we can.
>
> Well, in 5.Posting.rst that was possible. In submitting-patches.rst that
> conflicted with existing text in three areas, so some changes were
> needed; in one case the new text even got a little shorter, but overall
> those changes did not add a single new line.
>
> But sure, the new paragraph added a few lines. And it is identical in
> both documents. But that is a more complex and existing situation this
> patch can't solve. But of course I could avoid adding the new paragraph
> to submitting-patches.rst and change the "(see 'Tagging people requires
> permission' below for details)" added there already into "(see 'Tagging
> people requires permission' in Documentation/process/5.Posting.rst for
> details)". Given that people requested a even more detailed paragraph
> (see the other reply I just sent to Laurent) that might be wise; OTOH
> submitting-patches.rst right now AFAICS tries to be stand-alone, so it
> feels wrong at the same time.
>
> IOW: both is fine for me. Could you let me please know what you prefer?

Adding more cross references certainly won't help, I guess we'll leave
it as-is for now.

>> - I wonder if it would make sense to say that, if an implicit-permission
>>   tag has been added, the person named in it should get at least one
>>   copy of the change before it is merged?
>
> Hah, that is where I'd start to say "that seems like a bit much". And it
> does not help, as the cat is out of the bag once that copy is out, as
> the name and the email address someone might prefer to keep private
> would have made it to mailing list archives then already.

The cat is out of the bag but not in the repository; the thought was
that it's polite to give the person involved a heads-up that their name
is being taken in vain.  Certainly I've seen enough "what, no, I don't
want that tag there" reactions over the years to think it would
occasionally head off a use that the owner of the name doesn't want.

jon

  reply	other threads:[~2025-02-18 20:42 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-02-06 14:30 [PATCH v4] docs: clarify rules wrt tagging other people Thorsten Leemhuis
2025-02-07  1:42 ` Bagas Sanjaya
2025-02-07  8:24   ` Thorsten Leemhuis
2025-02-10 16:16     ` Mauro Carvalho Chehab
2025-02-11  8:45       ` Thorsten Leemhuis
2025-02-07  9:05 ` Laurent Pinchart
2025-02-08 15:36   ` Thorsten Leemhuis
2025-02-10 11:15     ` Laurent Pinchart
2025-02-11  8:43       ` Thorsten Leemhuis
2025-02-10 18:12 ` Jonathan Corbet
2025-02-11  8:48   ` Thorsten Leemhuis
2025-02-18 20:42     ` Jonathan Corbet [this message]
2025-03-06 13:31       ` Thorsten Leemhuis
2025-03-12 22:39         ` Jonathan Corbet
2025-03-17 22:49           ` Jonathan Corbet

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=875xl7nfxg.fsf@trenco.lwn.net \
    --to=corbet@lwn.net \
    --cc=gregkh@linuxfoundation.org \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@leemhuis.info \
    --cc=mchehab+huawei@kernel.org \
    --cc=simona.vetter@ffwll.ch \
    --cc=skhan@linuxfoundation.org \
    --cc=workflows@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.