Skip to contentSkip to conversation

package/nftables: do not build docs

PATCH New Yann E. MORIN 2 participants 2 messages Dec 27, 2014

lore
Share & tools
lore
#0 lore raw

PATCH v2 1/1 package/nftables: do not build docs

Building docs requires dblatex with a working Jade Wrapper (jw) which may well be missing on the build machine, and we do not build our own.

Since docs are anyway removed from the target, just do not build them.

Tell ./configure that we do not have the required tools:
- DBLATEX=no to disable PDF generation
- DOCBOOK2X_MAN=no, DOCBOOK2MAN=no and DB2X_DOCBOOK2MAN=no to disable manpage generation

Cc Thomas Petazzoni <[email protected]>

---
Changes v1 -> v2:
- do not touch Makefile.am, do not autoreconf, just pre-seed appropriate variables so ./configure concludes it does not have the required tools (Thomas)

---
 package/nftables/nftables.mk | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
package/nftables/nftables.mk +5 -1
index 9f18df7..6be05ed 100644
@@ -22,6 +22,10 @@ endif
 
 NFTABLES_CONF_ENV = \
 	ac_cv_prog_CONFIG_PDF=no \
-	LIBS="$(NFTABLES_LIBS)"
+	LIBS="$(NFTABLES_LIBS)" \
+	DBLATEX=no \
+	DOCBOOK2X_MAN=no \
+	DOCBOOK2MAN=no \
+	DB2X_DOCBOOK2MAN=no
 
 $(eval $(autotools-package))
Signature · 1 line
1.9.1
#1 lore raw

PATCH v2 1/1 package/nftables: do not build docs

Dear Yann E. MORIN,

Quoted 5 lines · Yann E. MORIN

Building docs requires dblatex with a working Jade Wrapper (jw) which may well be missing on the build machine, and we do not build our own.

Since docs are anyway removed from the target, just do not build them.

Quoted 7 lines

Tell ./configure that we do not have the required tools:
- DBLATEX=no to disable PDF generation
- DOCBOOK2X_MAN=no, DOCBOOK2MAN=no and DB2X_DOCBOOK2MAN=no to disable manpage generation

Cc Thomas Petazzoni <[email protected]>

Applied, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com