From: Maxim Cournoyer <maxim.cournoyer@gmail.com>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: liliana.prikler@gmail.com, 58813@debbugs.gnu.org,
zimon.toutoune@gmail.com
Subject: bug#58813: [PATCH] doc: Document how to use Patman for patches submission.
Date: Mon, 06 Mar 2023 20:07:07 -0500 [thread overview]
Message-ID: <87fsahz7wk.fsf@gmail.com> (raw)
In-Reply-To: <87y1oeyxbt.fsf@gnu.org> ("Ludovic Courtès"'s message of "Fri, 03 Mar 2023 10:54:14 +0100")
Hi Ludovic,
Ludovic Courtès <ludo@gnu.org> writes:
> Hi,
>
> Maxim Cournoyer <maxim.cournoyer@gmail.com> skribis:
>
>> Fixes <https://issues.guix.gnu.org/58813>.
>
> It’s only tangentially related, no?
It's not totally a tangent, because it removes examples which do not
work, and replace them with ones that do work, and recommend a tool that
can do what the previous shell command substitution failed to do
correctly (due to shell parsing rules).
>> * doc/contributing.texi (Sending a Patch Series): Mention Patman. Adjust the
>> examples to no longer showcase broken command substitutions. Add a section
>> about how to use Patman, with examples.
>
> I’m not convinced we’d want to advocate for yet another tool. I feel
> like this would make patch submission guidelines even more complex, or
> at least look more complex.
>
> Also, how many of the ~40 committers would be able to provide guidance
> with patman? That shouldn’t be the only criterion, but it certainly is
> an important one.
Since it's just documented as another tool on top, I don't think this
matters too much (it can be adopted or not). It's also a very simple
tool, which is more often than not invoked as simply 'patman' or 'patman
-n' (for dry-run).
I've also discovered about '--cc-cmd', which could be used with the
recently introduced get-maintainer mode (which was added for patman
support); it can be used like this:
--8<---------------cut here---------------start------------->8---
git send-email --cc-cmd='etc/teams.scm get-maintainer' --dry-run -1
--8<---------------cut here---------------end--------------->8---
It does the same thing as the copy/pasting of the output of
--8<---------------cut here---------------start------------->8---
etc/teams.scm cc-members HEAD^ HEAD
--8<---------------cut here---------------end--------------->8---
To the git send-email command, but with one difference: it uses '--cc'
for the email addresses instead of the nicer --add-header="X-Debbugs-Cc:
mail@example.org" ones. The later is best because when initially
sending the message to Debbugs, there's no bug # known yet, and the
receivers would be left to guess and perhaps even reply erroneously to
guix-patches@gnu.org and create a new ticket.
For this reason, I'm toying with the idea of contributing a "--x-cmd"
option to git send-email, which would be a script that outputs arbitrary
git send-email options to add to its command line.
To be continued...
--
Thanks,
Maxim
next prev parent reply other threads:[~2023-03-07 1:43 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-27 3:50 bug#58813: can't substitute etc/teams.scm command as doc suggests Maxim Cournoyer
2022-10-27 6:08 ` Liliana Marie Prikler
2022-10-27 12:40 ` Maxim Cournoyer
2022-10-27 16:27 ` Liliana Marie Prikler
2023-02-26 3:55 ` bug#58813: [PATCH] doc: Document how to use Patman for patches submission Maxim Cournoyer
2023-02-26 8:25 ` Liliana Marie Prikler
2023-02-26 17:11 ` Maxim Cournoyer
2023-02-27 0:45 ` Tobias Geerinckx-Rice via Bug reports for GNU Guix
2023-02-27 16:43 ` Maxim Cournoyer
2023-03-03 9:54 ` Ludovic Courtès
2023-03-03 17:26 ` Liliana Marie Prikler
2023-03-06 14:53 ` bug#58813: ‘guix style’ flaws Ludovic Courtès
2023-03-06 17:42 ` Maxim Cournoyer
2023-03-07 1:07 ` Maxim Cournoyer [this message]
2023-03-10 9:01 ` bug#58813: [PATCH] doc: Document how to use Patman for patches submission Ludovic Courtès
2023-03-10 14:07 ` Maxim Cournoyer
2023-04-23 16:04 ` bug#58813: [PATCH 0/5] Fix teams.scm by use of a new --header-cmd git send-email option Maxim Cournoyer
2023-04-23 16:04 ` bug#58813: [PATCH 1/5] gnu: git: Apply patch adding the --header-cmd feature Maxim Cournoyer
2023-04-23 17:00 ` Liliana Marie Prikler
2023-04-24 0:46 ` Maxim Cournoyer
2023-04-23 16:04 ` bug#58813: [PATCH 2/5] teams: Add 'cc-members-header-cmd' action Maxim Cournoyer
2023-04-23 16:04 ` bug#58813: [PATCH 3/5] teams: Add a configure-git action Maxim Cournoyer
2023-04-23 17:03 ` Liliana Marie Prikler
2023-04-24 0:59 ` Maxim Cournoyer
2023-04-24 2:29 ` bug#58813: [PATCH v2 0/5] Fix teams.scm by use of a new --header-cmd git send-email option Maxim Cournoyer
2023-04-24 2:29 ` bug#58813: [PATCH v2 1/5] gnu: git: Apply patch adding the --header-cmd feature Maxim Cournoyer
2023-04-24 2:29 ` bug#58813: [PATCH v2 2/5] teams: Add 'cc-members-header-cmd' action Maxim Cournoyer
2023-04-24 2:29 ` bug#58813: [PATCH v2 3/5] Makefile.am: Auto-configure Git on 'make' Maxim Cournoyer
2023-04-24 4:22 ` Liliana Marie Prikler
2023-04-24 12:37 ` Maxim Cournoyer
2023-04-28 14:28 ` Josselin Poiret via Bug reports for GNU Guix
2023-05-01 16:38 ` bug#58813: can't substitute etc/teams.scm command as doc suggests Maxim Cournoyer
2023-04-24 2:29 ` bug#58813: [PATCH v2 4/5] teams: Add a 'cc-mentors-header-cmd' action Maxim Cournoyer
2023-04-24 2:29 ` bug#58813: [PATCH v2 5/5] doc: Simplify contributing section by automating git configuration Maxim Cournoyer
2023-04-23 16:04 ` bug#58813: [PATCH 4/5] teams: Add a 'cc-mentors-header-cmd' action Maxim Cournoyer
2023-04-23 16:04 ` bug#58813: [PATCH 5/5] doc: Simplify contributing section by automating git configuration Maxim Cournoyer
2023-03-07 11:35 ` bug#58813: [PATCH v2] doc: Document how to use Patman for patches submission Simon Tournier
2023-03-07 16:46 ` Maxim Cournoyer
2023-03-07 17:39 ` Simon Tournier
2023-03-08 5:29 ` Liliana Marie Prikler
2023-03-08 8:47 ` Simon Tournier
2023-03-08 16:26 ` Maxim Cournoyer
2023-03-08 17:05 ` Simon Tournier
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=87fsahz7wk.fsf@gmail.com \
--to=maxim.cournoyer@gmail.com \
--cc=58813@debbugs.gnu.org \
--cc=liliana.prikler@gmail.com \
--cc=ludo@gnu.org \
--cc=zimon.toutoune@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 external index
https://git.savannah.gnu.org/cgit/guix.git
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.