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 158379 - All binaries linked with against -lperl when configured --with-perl
All binaries linked with against -lperl when configured --with-perl
Status: RESOLVED FIXED
Product: Gnumeric
Classification: Applications
Component: Compilation
git master
Other Linux
: Normal normal
: ---
Assigned To: Jody Goldberg
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2004-11-15 16:40 UTC by Stepan Kasal
Modified: 2005-01-25 17:01 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch (683 bytes, patch)
2004-11-15 16:41 UTC, Stepan Kasal 		none Details | Review
patch with changelog (864 bytes, patch)
2004-11-15 16:46 UTC, Stepan Kasal 		accepted-commit_after_freeze Details | Review

Description Stepan Kasal 2004-11-15 16:40:15 UTC 
I heard the following from JHM Dassen (Ray) on #gnumeric:

when configuring --with-perl (and when that works), _everything_ is linked
-lperl, including the main gnumeric binary.
It'd be better if the stuff for linking against libperl were kept in separate
variables and only used explicitly by the perl plugin's Makefile.

(I guess this experience was made with 1.3.93 or 1.3.92.)
The guilty line of configure.in was commented out just after 1.3.93, so it
cannot bite now.

The problem was caused by AC_CHECK_LIB in configure.in, which adds the library
to global variable LIBS (if the library is found, of course).
To override this default, you have to supply a nonempty third parameter.

The patch below is technically only a comment change atm, but anyway, I
recommend that you accept it for 1.5.
Comment 1 Stepan Kasal 2004-11-15 16:41:03 UTC 
Created attachment 33810 [details] [review]
patch

m4 must see a nonempty parameter
Comment 2 Stepan Kasal 2004-11-15 16:46:50 UTC 
Created attachment 33811 [details] [review]
patch with changelog

I forgot to save it before uploading, sorry.
Comment 3 Stepan Kasal 2005-01-25 17:01:18 UTC 
This patch went to devel branch (1.5), so the issue is closed.