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 740963 - build: Add -lm dependency to clutter-gst-3.0.la
build: Add -lm dependency to clutter-gst-3.0.la
Status: RESOLVED FIXED
Product: clutter-gst
Classification: Other
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: clutter-gst-maint
clutter-gst-maint
Depends on:
Blocks:
 
 
Reported: 2014-12-01 11:48 UTC by Philip Withnall
Modified: 2014-12-01 14:08 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
build: Add -lm dependency to clutter-gst-3.0.la (1.27 KB, patch)
2014-12-01 11:48 UTC, Philip Withnall 		reviewed Details | Review
build: Add -lm dependency to clutter-gst-3.0.la (1.73 KB, patch)
2014-12-01 12:22 UTC, Philip Withnall 		committed Details | Review

Description Philip Withnall 2014-12-01 11:48:15 UTC 
Otherwise clutter-gst’s introspection build fails with:
    ./.libs/libclutter-gst-3.0.so: error: undefined reference to 'rint'
    ./.libs/libclutter-gst-3.0.so: error: undefined reference to 'cos'
    ./.libs/libclutter-gst-3.0.so: error: undefined reference to 'sin'
Comment 1 Philip Withnall 2014-12-01 11:48:17 UTC 
Created attachment 291881 [details] [review]
build: Add -lm dependency to clutter-gst-3.0.la

It uses rint(), cos() and sin().
Comment 2 Emmanuele Bassi (:ebassi) 2014-12-01 11:50:21 UTC 
Review of attachment 291881 [details] [review]:

I would use `LT_LIB_M` in configure.ac, and then use `$(LIBM)` in the various Makefile.am, instead.
Comment 3 Philip Withnall 2014-12-01 12:22:05 UTC 
Created attachment 291883 [details] [review]
build: Add -lm dependency to clutter-gst-3.0.la

It uses rint(), cos() and sin().
Comment 4 Philip Withnall 2014-12-01 14:08:18 UTC 
Merged. Thanks for the super-fast review!

Attachment 291883 [details] pushed as b9de8d4 - build: Add -lm dependency to clutter-gst-3.0.la