Bug 433565 – dev-libs/libnl-3.2.12 fails with USE="-doc", configure: error: conditional "LINK_DOC" was never defined

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Ben Kohler 2012-08-31 19:47:14 UTC]

Created attachment 322657 [details]
build.log

dev-libs/libnl-3.2.12.ebuild introduces $(use_enable doc) that causes a ./configure failure when USE="-doc" is set.  I looked into this a bit, and it seems that --disable-doc was introduced upstream in libnl-3.2.10, and it has been broken always.

3.2.9 ./configure --disable-doc:
configure: WARNING: unrecognized options: --disable-doc

3.2.10 ./configure --disable-doc:
configure: error: conditional "LINK_DOC" was never defined.
Usually this means the macro was only invoked conditionally.


Anyway, 3.2.12 in portage today currently fails with the same error as 3.2.10.  build.log attached, config.log incoming.

[Comment 1 Ben Kohler 2012-08-31 19:48:15 UTC]

Created attachment 322659 [details]
config.log

[Comment 2 Ulrich Müller 2012-09-01 10:00:15 UTC]

*** Bug 433589 has been marked as a duplicate of this bug. ***

[Comment 3 Ulrich Müller 2012-09-01 10:00:37 UTC]

*** Bug 433617 has been marked as a duplicate of this bug. ***

[Comment 4 Ulrich Müller 2012-09-01 13:14:22 UTC]

*** Bug 433555 has been marked as a duplicate of this bug. ***

[Comment 5 Ulrich Müller 2012-09-01 13:14:59 UTC]

*** Bug 433639 has been marked as a duplicate of this bug. ***

[Comment 6 Ulrich Müller 2012-09-01 13:15:37 UTC]

*** Bug 433623 has been marked as a duplicate of this bug. ***

[Comment 7 Ulrich Müller 2012-09-01 13:16:22 UTC]

*** Bug 433575 has been marked as a duplicate of this bug. ***

[Comment 8 Ulrich Müller 2012-09-01 13:17:02 UTC]

*** Bug 433567 has been marked as a duplicate of this bug. ***

[Comment 9 Ulrich Müller 2012-09-01 13:19:25 UTC]

*** Bug 433643 has been marked as a duplicate of this bug. ***

[Comment 10 Ulrich Müller 2012-09-01 13:22:46 UTC]

*** Bug 433641 has been marked as a duplicate of this bug. ***

[Comment 11 Ulrich Müller 2012-09-01 13:26:32 UTC]

*** Bug 433643 has been marked as a duplicate of this bug. ***

[Comment 12 Ulrich Müller 2012-09-01 13:50:07 UTC]

*** Bug 433659 has been marked as a duplicate of this bug. ***

[Comment 13 Martin Jansa 2012-09-01 15:22:33 UTC]

Easiest solution for this is to move those conditionals after "fi" from --disable-docs in configure.in and reexecute autoconf.

[Comment 14 Ulrich Müller 2012-09-01 15:46:28 UTC]

*** Bug 433661 has been marked as a duplicate of this bug. ***

[Comment 15 Ulrich Müller 2012-09-01 15:48:55 UTC]

*** Bug 433625 has been marked as a duplicate of this bug. ***

[Comment 16 Ulrich Müller 2012-09-01 15:50:43 UTC]

*** Bug 433597 has been marked as a duplicate of this bug. ***

[Comment 17 Tom Wijsman (TomWij) (RETIRED) 2012-09-01 17:08:40 UTC]

Someone on the forums (http://forums.gentoo.org/viewtopic-p-7129184.html) reported the following sequence commands to work, while we have USE flag -doc:

  sed -i 's/use_enable\ doc/use_disable doc/'  /usr/portage/dev-libs/libnl/libnl-3.2.12.ebuild 
  ebuild /usr/portage/dev-libs/libnl/libnl-3.2.12.ebuild digest 
  sudo emerge libnl -1

So, I feel that it's just that the ebuild needs to correctly respond to the USE flag doc. No need to change ./configure at all, as far as I can see.

[Comment 18 Diego Elio Pettenò (RETIRED) 2012-09-01 17:10:05 UTC]

There is no use_disable. The change needs to be to the configure.

[Comment 19 Martin Jansa 2012-09-01 17:13:26 UTC]

jama libnl-3.2.12 # diff -uNr configure.in.orig configure.in
--- configure.in.orig   2012-08-30 13:41:22.000000000 +0200
+++ configure.in        2012-09-01 19:12:44.481322735 +0200
@@ -148,11 +148,12 @@
                link_doc=no
        fi
 
-       AM_CONDITIONAL([LINK_DOC], [test "x$link_doc" = "xyes"])
-       AM_CONDITIONAL([HAVE_DOXYGEN], [test "x$HAVE_DOXYGEN" = "xyes"])
-       AM_CONDITIONAL([HAVE_ASCIIDOC], [test "x$HAVE_ASCIIDOC" = "xyes"])
 fi
 
+AM_CONDITIONAL([LINK_DOC], [test "x$link_doc" = "xyes"])
+AM_CONDITIONAL([HAVE_DOXYGEN], [test "x$HAVE_DOXYGEN" = "xyes"])
+AM_CONDITIONAL([HAVE_ASCIIDOC], [test "x$HAVE_ASCIIDOC" = "xyes"])
+
 AM_CONDITIONAL([GENERATE_DOC], [test "x$generate_doc" != "xno"])
 
 AC_ARG_WITH([pkgconfigdir], AS_HELP_STRING([--with-pkgconfigdir=PATH],

$ autoconf

and it builds fine

[Comment 20 Jeroen Roovers (RETIRED) 2012-09-01 17:35:13 UTC]

Ah, this is libnl breaking every other minor release.

[Comment 21 Jeroen Roovers (RETIRED) 2012-09-01 17:47:08 UTC]

  01 Sep 2012; Jeroen Roovers <[e-mail]> libnl-3.2.12.ebuild,
  +files/libnl-3.2.12-doc.patch:
  Fix building with USE=-doc by Ben Kohler (bug #433565), patch by Martin
  Jansa.

[Comment 22 Ulrich Müller 2012-09-01 18:14:38 UTC]

*** Bug 433671 has been marked as a duplicate of this bug. ***

[Comment 23 Jeroen Roovers (RETIRED) 2012-09-06 14:28:25 UTC]

*** Bug 434068 has been marked as a duplicate of this bug. ***

[Comment 24 Jeroen Roovers (RETIRED) 2012-09-06 14:29:11 UTC]

This is fixed upstream and should be OK in 3.2.13.