Re: [PATCH v2] man*/: ffix (migrate to `MR`)

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

 



Hi Alex,

At 2023-08-01T01:15:02+0200, Alejandro Colomar wrote:
> Nah, I eyeballed random samples the diff and it looked good.  That,
> and your extensive tests, make me confident enough.  If we screwed
> anything, we can fix it.
> 
> The only concern I had some time ago was with code like exit(1), but
> that should be using italics today, so it shouldn't be a problem.  I
> can't imagine big issues.

Well, we'll see how it sits in your 'MR' branch for a while.  Maybe we
can recruit some beta testers.

> > Please do verify it yourself with the tools above (or better ones).
> > I'm well aware that this is a huge change that can make people
> > nervous.
> 
> I applied the patch, amended the message with a quote from this email,
> and pushed to the MR branch in my private git repo at
> <http://www.alejandro-colomar.es/src/alx/linux/man-pages/man-pages.git/log/?h=MR>.

I saw.  Thanks!

> Oh, and I also removed a few pages from your patch, per CONTRIBUTING
> guidelines:
> 
> Notes
>    External and autogenerated pages
>        A few pages come from external sources.  Fixes to the pages
>        should really go to the upstream source.
> 
>        tzfile(5), tzselect(8), zdump(8), and zic(8) come from the tz
>        project <https://www.iana.org/time-zones>.
> 
>        bpf-helpers(7) is autogenerated from the Linux kernel sources
>        using scripts.  See man-pages commits 53666f6c3 and 19c7f7839 for
>        details.

Right, thank you.  I completely zoned about those.

Unfortunately none of the time zone pages offer any hint I can see that
they're autogenerated, so (short of hard-coding a list), they can't be
automatically excluded from global changes.

bpf-helpers does announce that it's produced by rst2man (without naming
the tool explicitly).

The good news is that I've started to have a look at how rst2man works.
Dmitry Schachnev of the Debian Python team was really helpful in this
area.  I've poked and prodded it a bit.  Found some gaps.

I'm afraid I might horrify Engelbert Gruber, the upstream maintainer.

Regards,
Branden

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