Re: [PATCH 0/3] man/man2/syscalls.2: srcfix (0/3)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Branden,

On Sat, Jul 27, 2024 at 04:09:27PM GMT, G. Branden Robinson wrote:
> Hi Alex,
> 
> At 2024-07-27T22:30:54+0200, Alejandro Colomar wrote:
> > Do these changes only apply to syscalls(2)?
> 
> Yes.
> 
> > No other page has anything like that?
> 
> Yes, there are other pages with something like that.
> 
> man/man3/stdio.3
> man/man7/namespaces.7
> man/man7/signal-safety.7
> man/man7/signal.7
> man/man7/suffixes.7
> 
> I have a script that pattern-matches macro-style man page cross
> references and font-style cross references.[1]
> 
> Here are the counts for man-pages master:
> 
> $ wc -l macro.MR stragglers.MR
>  16193 macro.MR
>    892 stragglers.MR
>  17085 total
> 
> After this patch set:
> 
>  16696 macro.MR
>    389 stragglers.MR
>  17085 total

Hmmm, sounds good.  I'll apply this set.

> 
> This series is not a total solution, but it takes a bite out of the
> problem.  Further, I think I've finally got a reliable stack of sed
> scripts and a procedure that shows my work and is easy to use, so other
> tables with font-based cross references in them should migrate with
> greater ease.
> 
> > BTW, I prefer if the patches from a set are replies to the cover
> > letter.  :)
> 
> That's difficult for me because I generate the series _first_.[2]

That's usual.  Generating them _after_ would be weird.  :)

>  At
> that time I do not know what the MUA will construct for a Message-ID for
> item 0 in the series.  Do you have a suggestion?

Yup.

> 
> Regards,
> Branden
> 
> [1] $ cat find-MR-migrators.sh
> > macro.MR
> > stragglers.MR
> # Deliberately non-migrating: tzfile.5, zdump.8, zic.8
> git grep -n '^\.[BI]R \(\\%\)*\([.@_[:alnum:]\\-]\+\) (\([1-9a-z]\+\))' >> macro.MR
> # 1 false positive: _syscall.2
> git grep -n '^\.R[BI] [^ ]\+ [A-Za-z0-9_]\+.*(.*[1-9].*)' >> macro.MR
> git grep -n '\\f[BI]\(\\%\)*\([.@_[:alnum:]\\-]\+\)\\fP(\([1-9a-z]\+\))' >> stragglers.MR
> 
> [2] $ cat ~/bin/man-pages-format-patch
> #!/bin/sh
> 
> mbox="$HOME/Mail/Drafts/messages"
> 
> if [ $# -lt 1 ]
> then
>     echo "usage: $0 BEGIN..END" >&2
>     exit 2
> fi
> 
> git format-patch --to='Alejandro Colomar <alx@xxxxxxxxxx>' \
>     --cc='linux-man@xxxxxxxxxxxxxxx' --stdout "$@" \
>     >> $HOME/Mail/Drafts/messages

Regarding --to and --cc:

	As described in <man-pages.git/CONTRIBUTING.d/git>:

	   git-format-patch(1)
	       Send patches to the right addresses.

		   $ git config --local format.to 'Alejandro Colomar <alx@xxxxxxxxxx>';
		   $ git config --local format.cc '<linux-man@xxxxxxxxxxxxxxx>';

	The above will remove the need for specifying --to and --cc
	every time.

About --stdout and $HOME/Mail/Drafts/messages:

	I prefer to use ./.patches/, and store one file per patch.
	I have the following in my <~/.config/git/gitignore>:

		# Store patches here:
		/.patches/

	So, I usually generate patches as

		$ git format-patch -o .patches/ master..HEAD -v1 \
			--range-diff=master --cover-letter;

> if [ $? -ne 0 ]
> then
>     echo "$0: problem" >&2
>     exit 1
> fi
> 
> git log --oneline "$@"
> 
> cat <<EOF
> Patches generated in $mbox.
> Will now launch neomutt; use '<ESC>e' to send.
> Delete the draft(s) afterward with 'd'.
> EOF
> 
> read -p 'Ready? ' junk
> neomutt -f "$mbox"

And for sending:

	I use git-send-email(1), with neomutt(1) as a driver.  This is
	also documented in <man-pages.git/CONTRIBUTING.d/git>:

	   git-send-email(1)
	       If mutt(1) or neomutt(1) are configured in the system,
	       git-send-email(1) can be configured to use any of them as a
	       driver.  Recent versions of neomutt(1) can enable crypto with -C.

		   $ git config --global \
			   sendemail.sendmailcmd 'neomutt -C -H - && true';
	       or
		   $ git config --global sendemail.sendmailcmd 'mutt -H - && true';

	Since you sign your mail, and I prefer that you continue doing
	so, you'll need a recent version of neomutt(1) (2024-02-01 or
	later):
	<https://neomutt.org/feature/cli-crypto>
	<https://github.com/neomutt/neomutt/releases/tag/20240201>

	With the above, I can send patches as:

		$ git send-email .patches/v1-*;

	git-send-email(1) defaults to threading all patches so that all
	are a reply to the first one.

> # vim:set ai et sw=4 ts=4 tw=80:


Does that sound good?

Have a lovely night!
Alex


-- 
<https://www.alejandro-colomar.es/>

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Kernel Documentation]     [Netdev]     [Linux Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux