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 707804 - geocode-glib should link against libm
geocode-glib should link against libm
Status: RESOLVED FIXED
Product: geocode-glib
Classification: Other
Component: general
0.99.x
Other Linux
: Normal normal
: ---
Assigned To: geocode-glib maintainer(s)
geocode-glib maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2013-09-09 20:25 UTC by Laurent Bigonville
Modified: 2013-09-16 17:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
build: Check for libm in configure (again) (797 bytes, patch)
2013-09-09 23:04 UTC, Bastien Nocera
committed Details | Review

Description Laurent Bigonville 2013-09-09 20:25:25 UTC
Hi,

With version 0.99.2, some debian tool is complaining about missing symbols:

dpkg-shlibdeps: warning: symbol atan2 used by debian/libgeocode-glib0/usr/lib/libgeocode-glib.so.0.0.0 found in none of the libraries
dpkg-shlibdeps: warning: symbol cos used by debian/libgeocode-glib0/usr/lib/libgeocode-glib.so.0.0.0 found in none of the libraries
dpkg-shlibdeps: warning: symbol sin used by debian/libgeocode-glib0/usr/lib/libgeocode-glib.so.0.0.0 found in none of the libraries
dpkg-shlibdeps: warning: symbol sqrt used by debian/libgeocode-glib0/usr/lib/libgeocode-glib.so.0.0.0 found in none of the libraries

It seems that this has been removed from the configure.ac:
-AC_CHECK_LIB([m],[sin])

I guess it should be added back
Comment 1 Bastien Nocera 2013-09-09 23:04:24 UTC
Created attachment 254530 [details] [review]
build: Check for libm in configure (again)

As we did in 0e0e387356b340d2f539ad20696237fc94259b08 which
got broken by 4d78c0ecc8c0a6a92cd29da0d0a55fc546e5463d.
Comment 2 Zeeshan Ali 2013-09-09 23:19:42 UTC
Review of attachment 254530 [details] [review]:

Yeah, sorry about that.
Comment 3 Bastien Nocera 2013-09-16 17:46:57 UTC
Attachment 254530 [details] pushed as d1a2a7e - build: Check for libm in configure (again)