Hello Axel,
On Thu, Jun 15, 2023 at 01:07:12AM +0200, Axel Beckert wrote:
> thanks for the bug report!

You are welcome.

> Helge Kreutzmann wrote:
> > Finally the issues I'm reporting have accumulated over time and are
> > not always discovered by me, so sometimes my description of the
> > problem my be a bit limited - do not hesitate to ask so we can clarify
> > them.
> 
> Yeah, I think that's the place where things went a bit wrong.
> 
> > Man page: irqtop.1
> > Issue:    ethtool → B<ethtool>(8)
> 
> I'm sorry, but this is a groff-written man page, but your suggested
> change seems POD syntax.

That is correct. At manpages-l10n we do not see the original syntax
but only the "normalized" output as produced by po4a. So kindly
translate this back to groff.

> > "show extra eth stats (from ethtool)"
> 
> That line now renders in man like this:
> 
> "show extra eth stats (from B<ethtool>(8))"
> 
> I don't think that's wanted.

That is exactly as wanted. See man-pages(7) for the traditional
formating conventions. 

> Besides we render htop(1) and friends under "SEE ALSO" also without
> any special formatting.

Then this should be changed there as well. You can check many other
man pages who do.

> So I'll make it simply "ethtool(8)".

Maybe reconsider this (globally)?

Please not that there are tools out there which can identify the
B<ethtool>(8) and create appropriate hyperlinks; moreover it is good
to spot those links in the text.

> > Man page: irqtop.1
> > Issue:    enouth → enough
> 
> "enouth" neither is nor ever was in this Debian package.

That is correct, that is only present in the version from Fedora 38.
Upstream told me, that the man page comes from Debian. Apologies if
Fedora is shipping a different man page. As downstream this is
sometimes hard to spot where a specific text is actually originating
from.

> > "Show per-cpu statistics by specified mode. Available modes are: B<auto>, "
> > "B<enable>, B<disable>. The default option B<auto> detects the width of "
> > "window, then shows the per-cpu statistics if the width of window is large "
> > "enouth to show a full line of statistics."
> 
> Neither is this text.

That's right, this is comming from Fedora again.

> A quick search on codesearch.debian.net reveals that this is part of
> the man page from the _other_ tool named irqtop, the one from
> util-linux:
> https://sources.debian.org/src/util-linux/2.38.1-5/sys-utils/irqtop.1.adoc/?hl=29#L29
> 
> Which is not yet shipped in a binary package in Debian as I didn't
> manage to start the migration between the two irqtops timely before
> the Bookworm freeze.
> 
> Please report that error against src:util-linux.

Thanks for pointing this out. Potentially for Fedora the util-linux
version is (already) shipped. 

I'll forward this appropriately. Thanks for checking!

Greetings

           Helge
-- 
      Dr. Helge Kreutzmann                     deb...@helgefjell.de
           Dipl.-Phys.                   http://www.helgefjell.de/debian.php
        64bit GNU powered                     gpg signed mail preferred
           Help keep free software "libre": http://www.ffii.de/

Attachment: signature.asc
Description: PGP signature

Reply via email to