Re: [PATCH v2 1/3] proc_pid_fdinfo.5: Reduce indent for most of the page

Hi Alex, At 2024-11-02T11:39:37+0100, Alejandro Colomar wrote: > And diffs are a real win for text. Thus, semantic newlines are a real > win for text. "Write poems, not prose." (Any chance we may get that > warning added to groff(1)? :D) Yes, but I've kicked it out to groff 1.25 because a gif

Re: [PATCH v2 1/3] proc_pid_fdinfo.5: Reduce indent for most of the page

Hi Colin, At 2024-11-03T00:47:23+, Colin Watson wrote: > (That's leaving aside things like localized man pages, which I know > some folks on the groff list tend to sniff I can think of only one, the maintainer of a rival formatter. ;-) > at but I think they're important, Me too. I agree w

Re: [PATCH v2 1/3] proc_pid_fdinfo.5: Reduce indent for most of the page

Hi Colin, At 2024-11-02T19:06:53+, Colin Watson wrote: > How embarrassing. Could somebody please file a bug on > https://gitlab.com/man-db/man-db/-/issues to remind me to fix that? Done; . > lexgrog(1) is a useful (if oddly-named, sorry) debuggi

Re: [PATCH v3 2/4] proc_pid_fdinfo.5: Make pid clearer in the name and 1st paragraph

Hi Ian, At 2024-11-01T12:11:54-0700, Ian Rogers wrote: > diff --git a/man/man5/proc_pid_fdinfo.5 b/man/man5/proc_pid_fdinfo.5 > index 87e6dbe56..935b54b4c 100644 > --- a/man/man5/proc_pid_fdinfo.5 > +++ b/man/man5/proc_pid_fdinfo.5 [...] > -this is a subdirectory containing one entry for each file

Re: [PATCH v4 3/4] proc_pid_fdinfo.5: Add subsection headers for different fd types

Hi Ian, At 2024-11-01T14:18:29-0700, Ian Rogers wrote: > Make the sections about eventfd, epoll, signalfd, inotify, fanotify, > timerfd better separated with a clearer subsection header. > > Signed-off-by: Ian Rogers [...] > +.SS eventfd > .P [...] > +.SS epoll > .P [...] > +.SS signalfd > .P

Re: [PATCH v2 1/3] proc_pid_fdinfo.5: Reduce indent for most of the page

[adding Colin Watson to CC; and the groff list because I started musing] Hi Alex, At 2024-11-01T21:07:29+0100, Alejandro Colomar wrote: > > > > -/proc/pid/fdinfo/ \- information about file descriptors > > > > +.IR /proc/ pid /fdinfo " \- information about file descriptors" > > > > > > I wouldn't

Re: [PATCH v3 4/4] proc_pid_fdinfo.5: Add DRM subsection

Hi Ian, At 2024-11-01T12:11:56-0700, Ian Rogers wrote: [...] > +.EX > +pos:0 > +flags: 0212 > +mnt_id: 26 > +ino:284 > +drm-driver: i915 > +drm-client-id: 39 > +drm-pdev: :00:02.0 > +drm-total-system0: 6044 KiB > +drm-shared-system0: 0 > +drm-active-system0:

Re: [PATCH v1 1/3] proc_pid_fdinfo.5: Reduce indent for most of the page

At 2024-10-15T11:38:22-0700, Ian Rogers wrote: > When /proc/pid/fdinfo was part of proc.5 man page the indentation made > sense. As a standalone man page the indentation doesn't need to be so > far over to the right. > > Signed-off-by: Ian Rogers > --- > man/man5/proc_pid_fdinfo.5 | 50 +