diff mbox series

[nft] configure.ac: docbook2man invalid syntax error

Message ID 20180618121747.22158-1-eric@regit.org
State Changes Requested
Delegated to: Pablo Neira
Headers show
Series [nft] configure.ac: docbook2man invalid syntax error | expand

Commit Message

Eric Leblond June 18, 2018, 12:17 p.m. UTC
docbook2man can not be used with the same option so let's remove
it from the alternative. Fedora and debian seems to be fine with
that.
---
 configure.ac | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

Comments

Máté Eckl June 28, 2018, 10:20 a.m. UTC | #1
Hi,

It seems that I might be a bit late on this, but could you provide some details
on what error occured with docbook2man being an alternative?
I checked out the commit before yours (291f9a3) and I don't get any errors with
doc generation (nor had I before my last pull).

On the other hand, ArchLinux seems not to have the the executables of the other
alternatives (at least not with the same names), so I cannot compile nft now.

I attached the output of the compilation commands run at 291f9a3, I don't see any errors
regarding man page generation so I don't understand why this patch is necessary.

Maybe if you point out the bug, I can look after it.

Regards,
Máté

On Mon, Jun 18, 2018 at 02:17:47PM +0200, Eric Leblond wrote:
> docbook2man can not be used with the same option so let's remove
> it from the alternative. Fedora and debian seems to be fine with
> that.
> ---
>  configure.ac | 9 +++------
>  1 file changed, 3 insertions(+), 6 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 2a19733..88f0d74 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -56,14 +56,11 @@ AM_PROG_LIBTOOL
>  
>  AM_COND_IF([BUILD_MAN], [
>  	AC_CHECK_PROG(DOCBOOK2X_MAN, [docbook2x-man], [docbook2x-man], [no])
> -	AC_CHECK_PROG(DOCBOOK2MAN, [docbook2man], [docbook2man], [no])
>  	AC_CHECK_PROG(DB2X_DOCBOOK2MAN, [db2x_docbook2man], [db2x_docbook2man], [no])
>  	AS_IF([test "$DOCBOOK2X_MAN" != "no"], [DB2MAN="$DOCBOOK2X_MAN"],
> -	      [AS_IF([test "$DOCBOOK2MAN" != "no"], [DB2MAN="$DOCBOOK2MAN"],
> -		     [AS_IF([test "$DB2X_DOCBOOK2MAN" != "no"],
> -			    [DB2MAN="$DB2X_DOCBOOK2MAN"],
> -			    [AC_MSG_ERROR([docbook2x-man/db2x_docbook2man not found, required for building man pages])]
> -		     )]
> +	     [AS_IF([test "$DB2X_DOCBOOK2MAN" != "no"],
> +		    [DB2MAN="$DB2X_DOCBOOK2MAN --xinclude"],
> +		    [AC_MSG_ERROR([docbook2x-man/db2x_docbook2man not found, required for building man pages])]
>  	     )]
>  	)
>  	AC_SUBST(DB2MAN)
> -- 
> 2.17.1
> 
> --
> To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
$ sh autogen.sh && ./configure --prefix=/usr --sbindir=/usr/bin --sysconfdir=/etc && make -j8
libtoolize: putting auxiliary files in AC_CONFIG_AUX_DIR, 'build-aux'.
libtoolize: copying file 'build-aux/ltmain.sh'
libtoolize: putting macros in AC_CONFIG_MACRO_DIRS, 'm4'.
libtoolize: copying file 'm4/libtool.m4'
libtoolize: copying file 'm4/ltoptions.m4'
libtoolize: copying file 'm4/ltsugar.m4'
libtoolize: copying file 'm4/ltversion.m4'
libtoolize: copying file 'm4/lt~obsolete.m4'
configure.ac:54: installing 'build-aux/ar-lib'
configure.ac:34: installing 'build-aux/compile'
configure.ac:55: installing 'build-aux/config.guess'
configure.ac:55: installing 'build-aux/config.sub'
configure.ac:6: installing 'build-aux/install-sh'
configure.ac:6: installing 'build-aux/missing'
src/Makefile.am: installing 'build-aux/depcomp'
configure.ac: installing 'build-aux/ylwrap'
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /usr/bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking whether make supports nested variables... yes
checking how to create a pax tar archive... gnutar
checking whether make supports nested variables... (cached) yes
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables... 
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking whether gcc understands -c and -o together... yes
checking for style of include used by make... GNU
checking dependency style of gcc... gcc3
checking for a sed that does not truncate output... /usr/bin/sed
checking for flex... flex
checking lex output file root... lex.yy
checking lex library... -lfl
checking whether yytext is a pointer... yes
checking for bison... bison -y
checking for ar... ar
checking the archiver (ar) interface... ar
checking build system type... x86_64-pc-linux-gnu
checking host system type... x86_64-pc-linux-gnu
checking how to print strings... printf
checking for a sed that does not truncate output... (cached) /usr/bin/sed
checking for grep that handles long lines and -e... /usr/bin/grep
checking for egrep... /usr/bin/grep -E
checking for fgrep... /usr/bin/grep -F
checking for ld used by gcc... /usr/bin/ld
checking if the linker (/usr/bin/ld) is GNU ld... yes
checking for BSD- or MS-compatible name lister (nm)... /usr/bin/nm -B
checking the name lister (/usr/bin/nm -B) interface... BSD nm
checking whether ln -s works... yes
checking the maximum length of command line arguments... 1572864
checking how to convert x86_64-pc-linux-gnu file names to x86_64-pc-linux-gnu format... func_convert_file_noop
checking how to convert x86_64-pc-linux-gnu file names to toolchain format... func_convert_file_noop
checking for /usr/bin/ld option to reload object files... -r
checking for objdump... objdump
checking how to recognize dependent libraries... pass_all
checking for dlltool... no
checking how to associate runtime and link libraries... printf %s\n
checking for archiver @FILE support... @
checking for strip... strip
checking for ranlib... ranlib
checking command to parse /usr/bin/nm -B output from gcc object... ok
checking for sysroot... no
checking for a working dd... /usr/bin/dd
checking how to truncate binary pipes... /usr/bin/dd bs=4096 count=1
checking for mt... no
checking if : is a manifest tool... no
checking how to run the C preprocessor... gcc -E
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking for dlfcn.h... yes
checking for objdir... .libs
checking if gcc supports -fno-rtti -fno-exceptions... no
checking for gcc option to produce PIC... -fPIC -DPIC
checking if gcc PIC flag -fPIC -DPIC works... yes
checking if gcc static flag -static works... yes
checking if gcc supports -c -o file.o... yes
checking if gcc supports -c -o file.o... (cached) yes
checking whether the gcc linker (/usr/bin/ld -m elf_x86_64) supports shared libraries... yes
checking whether -lc should be explicitly linked in... no
checking dynamic linker characteristics... GNU/Linux ld.so
checking how to hardcode library paths into programs... immediate
checking whether stripping libraries is possible... yes
checking if libtool supports shared libraries... yes
checking whether to build shared libraries... yes
checking whether to build static libraries... yes
checking for docbook2x-man... no
checking for docbook2man... docbook2man
checking for db2x_docbook2man... no
checking for a2x... found
checking for pkg-config... /usr/bin/pkg-config
checking pkg-config is at least version 0.9.0... yes
checking for LIBMNL... yes
checking for LIBNFTNL... yes
checking for __gmpz_init in -lgmp... yes
checking for readline in -lreadline... yes
checking that generated files are newer than configure... done
configure: creating ./config.status
config.status: creating Makefile
config.status: creating libnftables.pc
config.status: creating src/Makefile
config.status: creating include/Makefile
config.status: creating include/nftables/Makefile
config.status: creating include/linux/Makefile
config.status: creating include/linux/netfilter/Makefile
config.status: creating include/linux/netfilter_arp/Makefile
config.status: creating include/linux/netfilter_bridge/Makefile
config.status: creating include/linux/netfilter_ipv4/Makefile
config.status: creating include/linux/netfilter_ipv6/Makefile
config.status: creating files/Makefile
config.status: creating files/nftables/Makefile
config.status: creating doc/Makefile
config.status: creating config.h
config.status: executing depfiles commands
config.status: executing libtool commands

nft configuration:
  cli support:			yes
  enable debugging symbols:	yes
  use mini-gmp:			no
  enable man page:              yes
  enable pdf documentation:	no
  libxtables support:		no
  json output support:          no
make  all-recursive
make[1]: Entering directory '/tmp/nftables'
Making all in src
make[2]: Entering directory '/tmp/nftables/src'
  YACC     parser_bison.c
updating parser_bison.h
make  all-am
make[3]: Entering directory '/tmp/nftables/src'
  CC       rule.lo
  CC       statement.lo
  CC       datatype.lo
  CC       expression.lo
  CC       evaluate.lo
  CC       proto.lo
  CC       payload.lo
  CC       exthdr.lo
  CC       fib.lo
  CC       hash.lo
  CC       meta.lo
  CC       rt.lo
  CC       numgen.lo
  CC       ct.lo
  CC       netlink.lo
  CC       netlink_linearize.lo
  CC       netlink_delinearize.lo
  CC       monitor.lo
  CC       segtree.lo
  CC       rbtree.lo
  CC       gmputil.lo
  CC       utils.lo
  CC       erec.lo
  CC       mnl.lo
  CC       iface.lo
  CC       services.lo
  CC       mergesort.lo
  CC       tcpopt.lo
  CC       socket.lo
  CC       libnftables.lo
  CC       libparser_la-parser_bison.lo
  LEX      scanner.c
  CC       main.o
  CC       cli.o
  CC       libparser_la-scanner.lo
  CCLD     libparser.la
  CCLD     libnftables.la
  CCLD     nft
make[3]: Leaving directory '/tmp/nftables/src'
make[2]: Leaving directory '/tmp/nftables/src'
Making all in include
make[2]: Entering directory '/tmp/nftables/include'
Making all in linux
make[3]: Entering directory '/tmp/nftables/include/linux'
Making all in netfilter
make[4]: Entering directory '/tmp/nftables/include/linux/netfilter'
make[4]: Nothing to be done for 'all'.
make[4]: Leaving directory '/tmp/nftables/include/linux/netfilter'
Making all in netfilter_arp
make[4]: Entering directory '/tmp/nftables/include/linux/netfilter_arp'
make[4]: Nothing to be done for 'all'.
make[4]: Leaving directory '/tmp/nftables/include/linux/netfilter_arp'
Making all in netfilter_bridge
make[4]: Entering directory '/tmp/nftables/include/linux/netfilter_bridge'
make[4]: Nothing to be done for 'all'.
make[4]: Leaving directory '/tmp/nftables/include/linux/netfilter_bridge'
Making all in netfilter_ipv4
make[4]: Entering directory '/tmp/nftables/include/linux/netfilter_ipv4'
make[4]: Nothing to be done for 'all'.
make[4]: Leaving directory '/tmp/nftables/include/linux/netfilter_ipv4'
Making all in netfilter_ipv6
make[4]: Entering directory '/tmp/nftables/include/linux/netfilter_ipv6'
make[4]: Nothing to be done for 'all'.
make[4]: Leaving directory '/tmp/nftables/include/linux/netfilter_ipv6'
make[4]: Entering directory '/tmp/nftables/include/linux'
make[4]: Nothing to be done for 'all-am'.
make[4]: Leaving directory '/tmp/nftables/include/linux'
make[3]: Leaving directory '/tmp/nftables/include/linux'
Making all in nftables
make[3]: Entering directory '/tmp/nftables/include/nftables'
make[3]: Nothing to be done for 'all'.
make[3]: Leaving directory '/tmp/nftables/include/nftables'
make[3]: Entering directory '/tmp/nftables/include'
make[3]: Nothing to be done for 'all-am'.
make[3]: Leaving directory '/tmp/nftables/include'
make[2]: Leaving directory '/tmp/nftables/include'
Making all in files
make[2]: Entering directory '/tmp/nftables/files'
Making all in nftables
make[3]: Entering directory '/tmp/nftables/files/nftables'
make[3]: Nothing to be done for 'all'.
make[3]: Leaving directory '/tmp/nftables/files/nftables'
make[3]: Entering directory '/tmp/nftables/files'
make[3]: Nothing to be done for 'all-am'.
make[3]: Leaving directory '/tmp/nftables/files'
make[2]: Leaving directory '/tmp/nftables/files'
Making all in doc
make[2]: Entering directory '/tmp/nftables/doc'
a2x --doctype manpage --format manpage libnftables-json.adoc
a2x --doctype manpage --format manpage libnftables.adoc
  GEN      nft.8
make[2]: Leaving directory '/tmp/nftables/doc'
make[2]: Entering directory '/tmp/nftables'
make[2]: Leaving directory '/tmp/nftables'
make[1]: Leaving directory '/tmp/nftables'
Eric Leblond June 28, 2018, 12:36 p.m. UTC | #2
Hello,

On Thu, 2018-06-28 at 12:20 +0200, Máté Eckl wrote:
> Hi,
> 
> It seems that I might be a bit late on this, but could you provide
> some details
> on what error occured with docbook2man being an alternative?
> I checked out the commit before yours (291f9a3) and I don't get any
> errors with
> doc generation (nor had I before my last pull).
> 
> On the other hand, ArchLinux seems not to have the the executables of
> the other
> alternatives (at least not with the same names), so I cannot compile
> nft now.

Ok, so this was done for ArchLinux. I did not manage to find an
operating system where it was working.

> 
> I attached the output of the compilation commands run at 291f9a3, I
> don't see any errors
> regarding man page generation so I don't understand why this patch is
> necessary.

On Fedora and Debian, docbook2man is wrapper and has a different syntax
so the command fails.

> 
> Maybe if you point out the bug, I can look after it.

A possible solution is to update configure.ac to check that docbook2man
(if found) accept the correct option.

BR,
--
Eric

> 
> Regards,
> Máté
> 
> On Mon, Jun 18, 2018 at 02:17:47PM +0200, Eric Leblond wrote:
> > docbook2man can not be used with the same option so let's remove
> > it from the alternative. Fedora and debian seems to be fine with
> > that.
> > ---
> >  configure.ac | 9 +++------
> >  1 file changed, 3 insertions(+), 6 deletions(-)
> > 
> > diff --git a/configure.ac b/configure.ac
> > index 2a19733..88f0d74 100644
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -56,14 +56,11 @@ AM_PROG_LIBTOOL
> >  
> >  AM_COND_IF([BUILD_MAN], [
> >  	AC_CHECK_PROG(DOCBOOK2X_MAN, [docbook2x-man], [docbook2x-
> > man], [no])
> > -	AC_CHECK_PROG(DOCBOOK2MAN, [docbook2man], [docbook2man],
> > [no])
> >  	AC_CHECK_PROG(DB2X_DOCBOOK2MAN, [db2x_docbook2man],
> > [db2x_docbook2man], [no])
> >  	AS_IF([test "$DOCBOOK2X_MAN" != "no"],
> > [DB2MAN="$DOCBOOK2X_MAN"],
> > -	      [AS_IF([test "$DOCBOOK2MAN" != "no"],
> > [DB2MAN="$DOCBOOK2MAN"],
> > -		     [AS_IF([test "$DB2X_DOCBOOK2MAN" != "no"],
> > -			    [DB2MAN="$DB2X_DOCBOOK2MAN"],
> > -			    [AC_MSG_ERROR([docbook2x-
> > man/db2x_docbook2man not found, required for building man pages])]
> > -		     )]
> > +	     [AS_IF([test "$DB2X_DOCBOOK2MAN" != "no"],
> > +		    [DB2MAN="$DB2X_DOCBOOK2MAN --xinclude"],
> > +		    [AC_MSG_ERROR([docbook2x-man/db2x_docbook2man
> > not found, required for building man pages])]
> >  	     )]
> >  	)
> >  	AC_SUBST(DB2MAN)
> > -- 
> > 2.17.1
> > 
> > --
> > To unsubscribe from this list: send the line "unsubscribe
> > netfilter-devel" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
Máté Eckl June 28, 2018, 4:28 p.m. UTC | #3
On Thu, Jun 28, 2018 at 02:36:28PM +0200, Eric Leblond wrote:
> Hello,
> 
> On Thu, 2018-06-28 at 12:20 +0200, Máté Eckl wrote:
> > Hi,
> > 
> > It seems that I might be a bit late on this, but could you provide
> > some details
> > on what error occured with docbook2man being an alternative?
> > I checked out the commit before yours (291f9a3) and I don't get any
> > errors with
> > doc generation (nor had I before my last pull).
> > 
> > On the other hand, ArchLinux seems not to have the the executables of
> > the other
> > alternatives (at least not with the same names), so I cannot compile
> > nft now.
> 
> Ok, so this was done for ArchLinux. I did not manage to find an
> operating system where it was working.
> 
> > 
> > I attached the output of the compilation commands run at 291f9a3, I
> > don't see any errors
> > regarding man page generation so I don't understand why this patch is
> > necessary.
> 
> On Fedora and Debian, docbook2man is wrapper and has a different syntax
> so the command fails.
> 
> > 
> > Maybe if you point out the bug, I can look after it.
> 
> A possible solution is to update configure.ac to check that docbook2man
> (if found) accept the correct option.

I don't really know autoconfig, but isn't that possible to prioritize the
alternatives?  For example, use docbook2x-man if available, if it is not, use
db2x-docbook2man, if neither of them is present, use docbook2man? And of course
fail, if none of them is present.

> 
> BR,
> --
> Eric
> 
> > 
> > Regards,
> > Máté
> > 
> > On Mon, Jun 18, 2018 at 02:17:47PM +0200, Eric Leblond wrote:
> > > docbook2man can not be used with the same option so let's remove
> > > it from the alternative. Fedora and debian seems to be fine with
> > > that.
> > > ---
> > >  configure.ac | 9 +++------
> > >  1 file changed, 3 insertions(+), 6 deletions(-)
> > > 
> > > diff --git a/configure.ac b/configure.ac
> > > index 2a19733..88f0d74 100644
> > > --- a/configure.ac
> > > +++ b/configure.ac
> > > @@ -56,14 +56,11 @@ AM_PROG_LIBTOOL
> > >  
> > >  AM_COND_IF([BUILD_MAN], [
> > >  	AC_CHECK_PROG(DOCBOOK2X_MAN, [docbook2x-man], [docbook2x-
> > > man], [no])
> > > -	AC_CHECK_PROG(DOCBOOK2MAN, [docbook2man], [docbook2man],
> > > [no])
> > >  	AC_CHECK_PROG(DB2X_DOCBOOK2MAN, [db2x_docbook2man],
> > > [db2x_docbook2man], [no])
> > >  	AS_IF([test "$DOCBOOK2X_MAN" != "no"],
> > > [DB2MAN="$DOCBOOK2X_MAN"],
> > > -	      [AS_IF([test "$DOCBOOK2MAN" != "no"],
> > > [DB2MAN="$DOCBOOK2MAN"],
> > > -		     [AS_IF([test "$DB2X_DOCBOOK2MAN" != "no"],
> > > -			    [DB2MAN="$DB2X_DOCBOOK2MAN"],
> > > -			    [AC_MSG_ERROR([docbook2x-
> > > man/db2x_docbook2man not found, required for building man pages])]
> > > -		     )]
> > > +	     [AS_IF([test "$DB2X_DOCBOOK2MAN" != "no"],
> > > +		    [DB2MAN="$DB2X_DOCBOOK2MAN --xinclude"],
> > > +		    [AC_MSG_ERROR([docbook2x-man/db2x_docbook2man
> > > not found, required for building man pages])]
> > >  	     )]
> > >  	)
> > >  	AC_SUBST(DB2MAN)
> > > -- 
> > > 2.17.1
> > > 
> > > --
> > > To unsubscribe from this list: send the line "unsubscribe
> > > netfilter-devel" in
> > > the body of a message to majordomo@vger.kernel.org
> > > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> -- 
> Eric Leblond <eric@regit.org>
> --
> To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Florian Westphal June 28, 2018, 4:30 p.m. UTC | #4
Máté Eckl <ecklm94@gmail.com> wrote:
> I don't really know autoconfig, but isn't that possible to prioritize the
> alternatives?  For example, use docbook2x-man if available, if it is not, use
> db2x-docbook2man, if neither of them is present, use docbook2man? And of course
> fail, if none of them is present.

I think it would be much better to display an error and just continue
without building documentation.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Pablo Neira Ayuso June 28, 2018, 4:43 p.m. UTC | #5
On Thu, Jun 28, 2018 at 06:30:15PM +0200, Florian Westphal wrote:
> Máté Eckl <ecklm94@gmail.com> wrote:
> > I don't really know autoconfig, but isn't that possible to prioritize the
> > alternatives?  For example, use docbook2x-man if available, if it is not, use
> > db2x-docbook2man, if neither of them is present, use docbook2man? And of course
> > fail, if none of them is present.
> 
> I think it would be much better to display an error and just continue
> without building documentation.

We can also fallback and make a second search for the package in
ArchLinux too, right?
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Máté Eckl June 28, 2018, 9:15 p.m. UTC | #6
On Thu, Jun 28, 2018 at 06:43:40PM +0200, Pablo Neira Ayuso wrote:
> On Thu, Jun 28, 2018 at 06:30:15PM +0200, Florian Westphal wrote:
> > Máté Eckl <ecklm94@gmail.com> wrote:
> > > I don't really know autoconfig, but isn't that possible to prioritize the
> > > alternatives?  For example, use docbook2x-man if available, if it is not, use
> > > db2x-docbook2man, if neither of them is present, use docbook2man? And of course
> > > fail, if none of them is present.
> > 
> > I think it would be much better to display an error and just continue
> > without building documentation.
> 
> We can also fallback and make a second search for the package in
> ArchLinux too, right?

So we are looking for docbook2x-man or db2x_docbook2man binaries, right?

Btw, which of these are present on Debian/fedora?

For ArchLinux:
In the official repo there is no binary (nor package) like this.

docbook2x provides the following binaries:
 - usr/bin/db2x_manxml
 - usr/bin/db2x_texixml
 - usr/bin/db2x_xsltproc
 - usr/bin/docbook2man
 - usr/bin/docbook2texi
 - usr/bin/sgml2xml-isoent
 - usr/bin/utf8trans

docbook-utils provide these:
 - usr/bin/db2dvi
 - usr/bin/db2html
 - usr/bin/db2man
 - usr/bin/db2pdf
 - usr/bin/db2ps
 - usr/bin/db2rtf
 - usr/bin/db2tex
 - usr/bin/db2texi
 - usr/bin/db2txt
 - usr/bin/docbook2dvi
 - usr/bin/docbook2html
 - usr/bin/docbook2man
 - usr/bin/docbook2pdf
 - usr/bin/docbook2ps
 - usr/bin/docbook2rtf
 - usr/bin/docbook2tex
 - usr/bin/docbook2texi
 - usr/bin/docbook2txt
 - usr/bin/jw
 - usr/bin/sgmldiff

For other packages related to docbook, see [1], but the rest seems irrelevant.
There is no *db2* package here.

In the Arch User Repository docbook-to-man provides:
./usr/bin
./usr/bin/instant
./usr/bin/docbook-to-man

No other packages seem to be relevant. There are some *db2* packages, but they
are not related to man generation.

[1] https://www.archlinux.org/packages/?sort=&q=docbook
[2] https://aur.archlinux.org/packages/?O=0&K=docbook
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Duncan Roe June 29, 2018, 1:04 a.m. UTC | #7
On Thu, Jun 28, 2018 at 11:15:58PM +0200, Máté Eckl wrote:
> On Thu, Jun 28, 2018 at 06:43:40PM +0200, Pablo Neira Ayuso wrote:
> > On Thu, Jun 28, 2018 at 06:30:15PM +0200, Florian Westphal wrote:
> > > Máté Eckl <ecklm94@gmail.com> wrote:
> > > > I don't really know autoconfig, but isn't that possible to prioritize the
> > > > alternatives?  For example, use docbook2x-man if available, if it is not, use
> > > > db2x-docbook2man, if neither of them is present, use docbook2man? And of course
> > > > fail, if none of them is present.
> > >
> > > I think it would be much better to display an error and just continue
> > > without building documentation.
> >
> > We can also fallback and make a second search for the package in
> > ArchLinux too, right?
>
> So we are looking for docbook2x-man or db2x_docbook2man binaries, right?
>
> Btw, which of these are present on Debian/fedora?
>
> For ArchLinux:
> In the official repo there is no binary (nor package) like this.
>
> docbook2x provides the following binaries:
>  - usr/bin/db2x_manxml
>  - usr/bin/db2x_texixml
>  - usr/bin/db2x_xsltproc
>  - usr/bin/docbook2man
>  - usr/bin/docbook2texi
>  - usr/bin/sgml2xml-isoent
>  - usr/bin/utf8trans
>
> docbook-utils provide these:
>  - usr/bin/db2dvi
>  - usr/bin/db2html
>  - usr/bin/db2man
>  - usr/bin/db2pdf
>  - usr/bin/db2ps
>  - usr/bin/db2rtf
>  - usr/bin/db2tex
>  - usr/bin/db2texi
>  - usr/bin/db2txt
>  - usr/bin/docbook2dvi
>  - usr/bin/docbook2html
>  - usr/bin/docbook2man
>  - usr/bin/docbook2pdf
>  - usr/bin/docbook2ps
>  - usr/bin/docbook2rtf
>  - usr/bin/docbook2tex
>  - usr/bin/docbook2texi
>  - usr/bin/docbook2txt
>  - usr/bin/jw
>  - usr/bin/sgmldiff
>
> For other packages related to docbook, see [1], but the rest seems irrelevant.
> There is no *db2* package here.
>
> In the Arch User Repository docbook-to-man provides:
> ./usr/bin
> ./usr/bin/instant
> ./usr/bin/docbook-to-man
>
> No other packages seem to be relevant. There are some *db2* packages, but they
> are not related to man generation.
>
> [1] https://www.archlinux.org/packages/?sort=&q=docbook
> [2] https://aur.archlinux.org/packages/?O=0&K=docbook
> --
docbook2man has always worked flawlessly for me under Slackware.

Whatever you do for other distributions, please don't break that.

Cheers ... Duncan.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox series

Patch

diff --git a/configure.ac b/configure.ac
index 2a19733..88f0d74 100644
--- a/configure.ac
+++ b/configure.ac
@@ -56,14 +56,11 @@  AM_PROG_LIBTOOL
 
 AM_COND_IF([BUILD_MAN], [
 	AC_CHECK_PROG(DOCBOOK2X_MAN, [docbook2x-man], [docbook2x-man], [no])
-	AC_CHECK_PROG(DOCBOOK2MAN, [docbook2man], [docbook2man], [no])
 	AC_CHECK_PROG(DB2X_DOCBOOK2MAN, [db2x_docbook2man], [db2x_docbook2man], [no])
 	AS_IF([test "$DOCBOOK2X_MAN" != "no"], [DB2MAN="$DOCBOOK2X_MAN"],
-	      [AS_IF([test "$DOCBOOK2MAN" != "no"], [DB2MAN="$DOCBOOK2MAN"],
-		     [AS_IF([test "$DB2X_DOCBOOK2MAN" != "no"],
-			    [DB2MAN="$DB2X_DOCBOOK2MAN"],
-			    [AC_MSG_ERROR([docbook2x-man/db2x_docbook2man not found, required for building man pages])]
-		     )]
+	     [AS_IF([test "$DB2X_DOCBOOK2MAN" != "no"],
+		    [DB2MAN="$DB2X_DOCBOOK2MAN --xinclude"],
+		    [AC_MSG_ERROR([docbook2x-man/db2x_docbook2man not found, required for building man pages])]
 	     )]
 	)
 	AC_SUBST(DB2MAN)