Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 222999 - net-libs/gnutls-2.2.x configure options problem
Summary: net-libs/gnutls-2.2.x configure options problem
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Linux bug wranglers
URL: http://www.gnutls.org
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-05-20 19:27 UTC by franky
Modified: 2008-05-20 20:02 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
gnutls-2.2.5.ebuild.diff (gnutls-2.2.5.ebuild.diff,375 bytes, patch)
2008-05-20 19:31 UTC, franky
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description franky 2008-05-20 19:27:05 UTC
Sorry for the noise in bug 222823, wasn't my intention to interrupt the stabilisation process :)

There's somthing wrong with the configure options in gnutls-2.2.x!

---snip----
local myconf
        use bindist && myconf="--disable-lzo" || myconf="$(use_enable lzo)"
---snip----

--disable-lzo should be --without-lzo and (use_enable lzo) should be (use_with lzo) otherwise it's an UNRECOGNIZED option, with the result that USE="bindist lzo" enables lzo-compression support.

Made a little patch, which solves "configure WARNING: Unrecognized options" and handles lzo support correctly :)

Reproducible: Always

Steps to Reproduce:
1.USE="bindist lzo" emerge net-libs/gnutls-2.2.x
2.watch configure for unrecognized options
3.configure activates lzo compression support

Actual Results:  
gnutls gets build with lzo compression support, although it's disabled because of licensing issues

Expected Results:  
USE="bindist lzo" emerge gnutls ends without lzo compression support

* lzo support was disabled for binary distribution of gnutls
 * due to licensing issues. See Bug 202381 for details.
>>> Unpacking source...
>>> Unpacking gnutls-2.2.5.tar.bz2 to /var/tmp/portage/net-libs/gnutls-2.2.5/work
 * Running elibtoolize in: gnutls-2.2.5/build-aux
 *   Applying portage-1.5.10.patch ...
 *   Applying sed-1.5.6.patch ...
>>> Source unpacked.
>>> Compiling source in /var/tmp/portage/net-libs/gnutls-2.2.5/work/gnutls-2.2.5 ...
 * econf: updating gnutls-2.2.5/build-aux/config.guess with /usr/share/gnuconfig/config.guess
 * econf: updating gnutls-2.2.5/build-aux/config.sub with /usr/share/gnuconfig/config.sub
./configure --prefix=/usr --host=i686-pc-linux-gnu --mandir=/usr/share/man --infodir=/usr/share/info --datadir=/usr/share --sysconfdir=/etc --localstatedir=/var/lib --without-included-opencdk --with-zlib --with-lzo --enable-nls --disable-guile --disable-gtk-doc --disable-lzo --build=i686-pc-linux-gnu
configure: WARNING: Unrecognized options: --disable-lzo


*** Libcfg+ was not found. Will use the included one.
checking whether to use the included libcfg... yes
checking whether to include zlib compression support... yes
checking for libz... yes
checking how to link with libz... /usr/lib/libz.so
checking whether to include lzo compression support... yes
checking whether to use the included lzo compression library... no
checking for lzo1x_1_compress in -llzo2... yes
checking lzo/lzo1x.h usability... yes
checking lzo/lzo1x.h presence... yes
checking for lzo/lzo1x.h... yes
Comment 1 franky 2008-05-20 19:31:06 UTC
Created attachment 153787 [details, diff]
gnutls-2.2.5.ebuild.diff

patch file to get rid of unrecognized options warning and correctly handle USE="bindist lzo"
Comment 2 franky 2008-05-20 19:43:23 UTC
corsair fixed this already! :)
My approach was a bit tame, forgot to get rid of the second lzo option!

./configure --prefix=/usr --host=powerpc64-unknown-linux-gnu
--mandir=/usr/share/man --infodir=/usr/share/info --datadir=/usr/share
--sysconfdir=/etc --localstatedir=/var/lib --without-included-opencdk
--with-zlib --with-lzo --enable-nls --disable-guile --disable-gtk-doc
--enable-lzo --libdir=/usr/lib64 --build=powerpc64-unknown-linux-gnu
configure: WARNING: Unrecognized options: --enable-lzo

the error is from the redundant entrys --enable-lzo and --with-lzo.

src_compile() logic is broken. it does first "use bindist &&
myconf="--disable-lzo" || myconf="$(use_enable lzo)"" and then "econf [...]
$(use_with lzo)"

I removed the redundant one after econf and changed use_enable to use_with in
the bindist line. I also changed --disable-lzo to --without-lzo.
Comment 3 franky 2008-05-20 20:02:49 UTC
More @ bug 222823