Hi Michael,

Michael Reed wrote on Fri, Sep 16, 2016 at 01:05:39PM -0400:

> The attached diff cleans up all mandoc(1) warnings and most igor(1)
> warnings from the ports man pages; most of the changes should be
> trivial.  While I was removing a stale reference to a.out(5) in
> check-lib-depends.1, I noticed this sentence in that same man page:
> 
>      -o       Scan binaries using ldd(1), which also works for old a.out
>               binaries, instead of objdump(1), which only works for elf(5)
>               binaries.
> 
> Given the a.out(5) manual page seems to have been removed [1], is the
> above sentence still correct?

I think it is still correct, but i doubt that using it is still useful
in any way.  Maybe espie@ could garbage collect the -o option, maybe
even the whole OpenBSD::BinaryScan::Ldd package - but there may be
tentacles, so i dare not touch it.  And i don't know how urgent that
cleanup is.

For now, i committed your diff as you sent it.

Thanks,
  Ingo


> Index: man1/check-lib-depends.1
> ===================================================================
> RCS file: /cvs/ports/infrastructure/man/man1/check-lib-depends.1,v
> retrieving revision 1.5
> diff -u -p -r1.5 check-lib-depends.1
> --- man1/check-lib-depends.1  27 Nov 2011 16:50:34 -0000      1.5
> +++ man1/check-lib-depends.1  16 Sep 2016 16:55:29 -0000
> @@ -81,9 +81,7 @@ Always show progress-meter, even if not 
>  .It Fl o
>  Scan binaries using
>  .Xr ldd 1 ,
> -which also works for old
> -.Xr a.out 5
> -binaries, instead of
> +which also works for old a.out binaries, instead of
>  .Xr objdump 1 ,
>  which only works for
>  .Xr elf 5
> @@ -113,6 +111,5 @@ to other options.
>  .Sh SEE ALSO
>  .Xr ldd 1 ,
>  .Xr objdump 1 ,
> -.Xr a.out 5 ,
>  .Xr bsd.port.mk 5 ,
>  .Xr elf 5
> Index: man1/dpb.1
> ===================================================================
> RCS file: /cvs/ports/infrastructure/man/man1/dpb.1,v
> retrieving revision 1.119
> diff -u -p -r1.119 dpb.1
> --- man1/dpb.1        30 Aug 2016 14:00:46 -0000      1.119
> +++ man1/dpb.1        16 Sep 2016 16:55:29 -0000
> @@ -809,7 +809,7 @@ When
>  .Nm
>  is run as root, it uses a privilege drop model instead of the
>  dangerous privilege elevation model of
> -.Xr doas 8 .
> +.Xr doas 1 .
>  When run as root, by default,
>  .Ar _pbuild
>  is used as the build and log user, and
> @@ -1461,8 +1461,8 @@ and restart it.
>  .Sh SEE ALSO
>  .Xr clean-old-distfiles 1 ,
>  .Xr dpb-replay 1 ,
> -.Xr pkgpath 7 ,
> -.Xr proot 1
> +.Xr proot 1 ,
> +.Xr pkgpath 7
>  .Sh AUTHOR
>  Marc Espie
>  .Sh HISTORY
> Index: man8/bulk.8
> ===================================================================
> RCS file: /cvs/ports/infrastructure/man/man8/bulk.8,v
> retrieving revision 1.9
> diff -u -p -r1.9 bulk.8
> --- man8/bulk.8       22 Jun 2016 23:25:57 -0000      1.9
> +++ man8/bulk.8       16 Sep 2016 16:55:29 -0000
> @@ -36,7 +36,6 @@ Clean up and do subsequent bulks.
>  .It
>  Perform additional maintenance.
>  .El
> -
>  .Ss 1. Choose master machine setup and create partitions
>  Setup a master machine with enough room for a chroot, say
>  .Pa /build .
> @@ -97,7 +96,7 @@ block out quick proto {tcp,udp} from sel
>  Recent
>  .Ox
>  systems do not need any kind of
> -.Xr doas 8
> +.Xr doas 1
>  setup for bulk ports builds, as
>  .Xr dpb 1
>  is run as root and drops permissions appropriately.
> @@ -183,7 +182,6 @@ again each time, most things will get cl
>  Note that known directories such as WRKOBJDIR do not get cleaned up
>  automatically, so you may want to set up a STARTUP_SCRIPT in
>  .Xr dpb 1 .
> -.Pp
>  .Ss 5. Perform additional maintenance
>  .Xr clean-old-distfiles 1
>  should be run occasionally since the DISTDIR will continue growing.

Reply via email to