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 722080 - Use -ldl is not portable
Use -ldl is not portable
Status: RESOLVED FIXED
Product: seed
Classification: Bindings
Component: libseed
git master
Other FreeBSD
: Normal normal
: ---
Assigned To: seed-maint
seed-maint
Depends on:
Blocks:
 
 
Reported: 2014-01-13 07:22 UTC by Ting-Wei Lan
Modified: 2014-01-14 04:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Check whether we actually needs libdl (843 bytes, patch)
2014-01-13 07:25 UTC, Ting-Wei Lan
none Details | Review

Description Ting-Wei Lan 2014-01-13 07:22:34 UTC
Some systems do not have libdl, and functions such as dlsym() may already exist in libc. We should check whether it is required to link to libdl, or it will cause error on systems which does not have libdl.
Comment 1 Ting-Wei Lan 2014-01-13 07:25:22 UTC
Created attachment 266130 [details] [review]
Check whether we actually needs libdl
Comment 2 Alan Knowles 2014-01-14 03:22:49 UTC
Thanks

Committed to git
Comment 3 Ting-Wei Lan 2014-01-14 04:06:02 UTC
It seems the git commit is wrong. -ldl still exists in libseed/Makefile.am, and it does not works on systems which do not have libdl.
Comment 4 Alan Knowles 2014-01-14 04:19:21 UTC
patch was playing up here...

should be commit now
Thanks