[PATCH] configure.ac: Add mandoc as valid formatter

[vehk at vehk.de]

Hi,

Ingo Schwarze <schwarze at usta.de> wrote:

> Thanks for reporting.  I agree very much with the principle.  Strange
> no one from Void Linux or Alpine Linux reported this issue before...

I had a look at Void's and Alpine's build scripts, and they both
explicitly set --with-mantype=doc. I'm assuming the issue was known but
never reported upstream since it'd imply more work and friction than
committing a quick local change. I'm planning to make this fix known to
the maintainers of the scripts once it's merged.

> However, i'd suggest
> 
> 	AC_PATH_PROGS([NROFF], [nroff mandoc awf], ...
> 
> Otherwise, you get this strange output from ./configure
> when both are installed, needlessly confusing users:
> 
> 	checking for nroff... /usr/local/bin/nroff
> 	checking for mandoc... /usr/bin/mandoc
> 	[...]
> 	checking for mandoc... (cached) /usr/local/bin/nroff
> 
> The reason for this confusing output is that the first and the last of
> these lines both use the variable NROFF, so they should also better
> log "checking for nroff...", hence let's keep nroff first.
> The final value of the NROFF variable doesn't matter, it isn't used
> for anything in the build.

Had not spotted this in my tests and wasn't aware of this behaviour at
all - I agree that we should keep nroff first in this case.

Wolfgang