On Tue, Sep 13, 2016 at 03:32:05AM -0600, Anthony J. Bentley wrote:
> Jiri B writes:
> > Hi,
> >
> > lftp man pages is odd, see section for 'mirror' command.
>
> The lftp(1) manual uses tbl(7) macros, which need extra preprocessing.
> pkg_create(1) doesn't do any preprocessing... although that's easy
> enough to fix. (Anyone have an idea of how many USE_GROFF manuals use
> troff preprocessors? Is it worth it?)
>
> USE_GROFF could also be removed from lftp. It's got some spacing and
> line wrapping issues in mandoc though.
>
> Index: OpenBSD/PackingElement.pm
> ===================================================================
> RCS file: /cvs/src/usr.sbin/pkg_add/OpenBSD/PackingElement.pm,v
> retrieving revision 1.244
> diff -u -p -r1.244 PackingElement.pm
> --- OpenBSD/PackingElement.pm 25 Jun 2016 18:02:59 -0000 1.244
> +++ OpenBSD/PackingElement.pm 13 Sep 2016 09:22:17 -0000
> @@ -655,7 +655,7 @@ sub format
> chdir($dir) or die "Can't chdir to $dir";
> },
> OpenBSD::Paths->groff,
> - qw(-mandoc -mtty-char -E -Ww -Tascii -P -c),
> + qw(-mandoc -mtty-char -E -Ww -Tascii -t -P -c),
> @extra, '--', $file);
> } else {
> die "Can't parse source name $fname";
Nope, wrong location. See how @extra is defined. lftp manpage is bogus.
It should have the preprocessing letters in a comment.
