Re: [PATCH] man/man2/ptrace.2: Add PTRACE_SET_SYSCALL_INFO, update struct ptrace_syscall_info

[Alejandro Colomar <alx@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1484 bytes --]

On 2026-02-10T23:41:45+0200, Dmitry V. Levin wrote:
> On Tue, Feb 10, 2026 at 10:20:02PM +0100, Alejandro Colomar wrote:
> > Hi Dmitry,
> > 
> > On 2026-02-10T22:53:47+0200, Dmitry V. Levin wrote:
> > > > > Looks good, thanks!
> > > > 
> > > > Should I take that as a Reviewed-by?
> > > 
> > > Feel free to add:
> > > 
> > > Reviewed-by: Dmitry V. Levin <ldv@strace.io>
> > 
> > Thanks!  I'll quote it, if you don't mind, as name-addreess pairs can't
> > contain an unquoted '.' --git(1) misbehaves on the '.', unless they
> > fixed it recently--.
> > 
> > 	Reviewed-by: "Dmitry V. Levin" <ldv@strace.io>
> > 
> > Is that okay?
> 
> I've been using the unquoted format for a while, but the quoted one
> is fine as well.

I've tried to reproduce it now, and they seem to have fixed it.  When
the patch is read by git-send-email(1) and it gets the addresses from
the trailer of the patch, it previously would have transformed
	Dmitry V. Levin <ldv@strace.io>
into
	Dmitry V.  Levin <ldv@strace.io>
Some depdenency (a library) of git(1) interpreted the '.' as ending
a sentence.  I've tried now, and it transforms
	Dmitry V. Levin <ldv@strace.io>
into
	"Dmitry V. Levin" <ldv@strace.io>
thus doing the right thing.  However, I expect some software might still
misbehave.  I remember having reproduced the bug not so long ago (maybe
one year?).


Cheers,
Alex

> 
> Thanks,
> 
> 
> -- 
> ldv
> 

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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]