After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 727939 - INTLLIBS are always appended in configure checks
INTLLIBS are always appended in configure checks
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: build
2.40.x
Other OpenBSD
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks:
 
 
Reported: 2014-04-10 07:59 UTC by Antoine Jacoutot
Modified: 2014-04-15 13:12 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
also unset LIBS when we unset LDFLAGS (2.45 KB, patch)
2014-04-10 09:17 UTC, Antoine Jacoutot 		accepted-commit_now Details | Review

Description Antoine Jacoutot 2014-04-10 07:59:59 UTC 
Hi.

After this commit went in:
2b178c762fbc0706031e9890ba190233e243a602
I was wondering why -z,nodelete was not enabled up by configure on OpenBSD.

It turns out INTLLIBS is appended to the configure test:
configure:28871: cc -o conftest -O2 -pipe -I/usr/local/include  -pthread  -Wl,-z,nodelete conftest.c -lintl   >&5
/usr/bin/ld: cannot find -lintl

It cannot find libintl, because I run configure with LDFLAGS="-L/usr/local/lib" which is where libintl is installed and LDFLAGS gets dropped during this particular configure check (same with the -Bsymbolic check), which is expected. But -lintl still gets appended which as seen above makes the check fails.
Comment 1 Antoine Jacoutot 2014-04-10 09:17:51 UTC 
Created attachment 273964 [details] [review]
also unset LIBS when we unset LDFLAGS

This patch does the trick for me. Dunno if it is the best^recommended way.
Comment 2 Allison Karlitskaya (desrt) 2014-04-15 12:56:37 UTC 
Review of attachment 273964 [details] [review]:

Makes sense -- we won't need any LIBS in order to check -Bsymbolic.
Comment 3 Antoine Jacoutot 2014-04-15 13:12:36 UTC 
Thank you Ryan. Pushed as 2d96503.