Bug 1523239

Summary: rebuild broken when docbook-utils is present
Product: Red Hat Enterprise Linux 7 Reporter: Tomas Dolezal <todoleza>
Component: nftablesAssignee: Phil Sutter <psutter>
Status: CLOSED ERRATA QA Contact: Tomas Dolezal <todoleza>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 7.5CC: lmiksik
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: nftables-0.8-7.el7 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-04-10 16:09:09 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Tomas Dolezal 2017-12-07 13:42:09 UTC
Description of problem:
./configure script finds docbook-utils and tries to use it to build manpages. that fails on `jw` error. the package is successfully built if docbook-utils are not present (or disabled via configure?)

If missing, then log reads:
checking for docbook2x-man... no
checking for docbook2man... no
checking for db2x_docbook2man... no
configure: WARNING: docbookx2-man/db2x_docbook2man not found, no manpages will be built
....

As found later in the log, manpage is prepackaged.

Version-Release number of selected component (if applicable):
nftables-0.8-4.el7

How reproducible:
always, environment

Steps to Reproduce:
have docbook-utils installed

Actual results:Making all in doc
make[2]: Entering directory `/home/test/rpmbuild/BUILD/nftables-0.8/doc'
docbook2man --xinclude nft.xml
Usage: jw [<options>] <sgml_file>
where <options> are:
  -f|--frontend <frontend>: 	 Specify the frontend (source format)
  				 (default is docbook)
  -b|--backend <backend>: 	 Specify the backend (destination format)
  				 (default is html)
  -c|--cat <file>: 		 Specify an extra SGML open catalog
  -n|--nostd: 			 Do not use the standard SGML open catalogs
  -d|--dsl <file>|default|none:	 Specify an alternate style sheet
  				 (default is to use the default stylesheet)
  -l|--dcl <file>: 		 Specify an alternate SGML declaration
  				 (usual ones like xml.dcl get detected automatically)
  -s|--sgmlbase <path>: 	 Change base directory for SGML distribution
  				 (usually /usr/share/sgml)
  -p|--parser <program>: 	 Specify the parser if several are installed
  				  (jade or openjade)
  -o|--output <directory>: 	 Set output directory
  -u|--nochunks: 		 Output only one big file
  				 (overrides the stylesheet settings)
  -i|--include <section>: 	 Specify a SGML marked section to include
  				 (should be marked as "ignore" in the SGML text)
  -w|--warning <warning_type>|list: Control warnings or display the allowed warning types
  -e|--errors <error_type>|list: Control errors or display the allowed error types
  -h|--help: 			 Print this help message and exit
  -V <variable[=value]>: 	 Set a variable
  -v|--version: 		 Print the version and exit
make[2]: *** [nft.8] Error 1
make[2]: Leaving directory `/home/test/rpmbuild/BUILD/nftables-0.8/doc'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/home/test/rpmbuild/BUILD/nftables-0.8'
make: *** [all] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.eXSg5t (%build)

Expected results:


Additional info:

Comment 3 Phil Sutter 2017-12-07 16:47:34 UTC
When initially importing the package from Fedora, docbook2X wasn't available in RHEL buildsystems. Therefore I included a prebuilt nft.8 into the RPM.

An easy way to disable man page building is to pass DOCBOOK2X_MAN=/bin/true to configure script.

Comment 5 Phil Sutter 2017-12-09 15:26:44 UTC
I had to tweak the solution a bit. The previous approach caused issues since 'make install' still tried to install the man page despite it wasn't built. A proper way to disable it completely is to pass ac_cv_prog_DOCBOOK2X_MAN="no" to configure.

Comment 9 Phil Sutter 2017-12-18 18:53:28 UTC
Hi Tomas,

I missed that there are multiple options for docbook2man implementations being searched, current release should disable tests for all of them.

Sorry, Phil

Comment 13 errata-xmlrpc 2018-04-10 16:09:09 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHEA-2018:0882