Re: [PATCH] execveat.2: srcfix

Michael Kerrisk (man-pages)
2021-01-01T11:44:44Z

Hi Alex,

On 1/1/21 12:29 AM, Alejandro Colomar (man-pages) wrote:
>
>
> On 12/31/20 7:55 PM, Alejandro Colomar (man-pages) wrote:
>> Hi Michael,
>>
[...]

> Hi Michael,
>
> I've seen some of the 'ffix!!' problems are caused by a yet missing
> [.nf/.fi].

Yes.

> So when you finish the patch I guess they'll vanish.
> Still have a look at some of them, though, as there might be other causes.

Yes, thanks for catching those.

>> BTW, Why didn't you include man2?
> [...]
>
>>>
>>> What do you think about the work-in-progess patch below? The following
>>> may be useful for review:
>>>
>>> for p in $(git grep -l 'SYN' man[3-8]); do
>>> echo "===================== $p"; man -l $p 2> /dev/null |
>>> sed -n '/^SYNOP/,/DESCR/p' | sed '/Feat/,$d';
>>> done | less
>
> I see now that the patch has changed man2, but the script above only
> shows man3 and above. I guess those were the ones you had pending,
> right? :)

Yes.

>
> Send me the next revision when you have it to have another look at it.

I pushed something now. Maybe we tweak it still a bit.

> Although the render looks weird on some cases because it's not yet
> finished, the diff looks good to me.

Okay.

Thanks,

Michael

--
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/