Merge lp://staging/~laney/libindicator/libm-libadd into lp://staging/libindicator/15.10

Proposed by Iain Lane
Status: Merged
Approved by: Lars Karlitski
Approved revision: 532
Merged at revision: 532
Proposed branch: lp://staging/~laney/libindicator/libm-libadd
Merge into: lp://staging/libindicator/15.10
Diff against target: 26 lines (+3/-2)
2 files modified
configure.ac (+1/-1)
libindicator/Makefile.am (+2/-1)
To merge this branch: bzr merge lp://staging/~laney/libindicator/libm-libadd
Reviewer Review Type Date Requested Status
Lars Karlitski (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Sebastien Bacher Pending
Review via email: mp+279871@code.staging.launchpad.net

Commit message

Add -lm via LDADD to resolve FTBFS in Xenial

Description of the change

libindicator FTBFS in xenial atm

> libtool: link: gcc -shared -fPIC -DPIC .libs/libindicator_la-gen-indicator-service.xml.o .libs/libindicator_la-indicator-object.o .libs/libindicator_la-indicator-object-enum-types.o .libs/libindicator_la-indicator-desktop-shortcuts.o .libs/libindicator_la-indicator-image-helper.o .libs/libindicator_la-indicator-object-marshal.o .libs/libindicator_la-indicator-service.o .libs/libindicator_la-indicator-service-manager.o -lgtk-x11-2.0 -lgdk-x11-2.0 -lpangocairo-1.0 -latk-1.0 -lcairo -lgdk_pixbuf-2.0 -lpangoft2-1.0 -lpango-1.0 -lfontconfig /usr/lib/s390x-linux-gnu/libfreetype.so -lgmodule-2.0 -lgio-2.0 -lgobject-2.0 -lglib-2.0-lm -pthread -O2 -Wl,-Bsymbolic-functions -Wl,-z -Wl,relro -Wl,-z -Wl,defs -Wl,--as-needed -Wl,--export-dynamic -pthread -pthread -Wl,-soname -Wl,libindicator.so.7 -Wl,-version-script -Wl,.libs/libindicator.ver -o .libs/libindicator.so.7.0.0
> /usr/bin/ld: cannot find -lglib-2.0-lm
> collect2: error: ld returned 1 exit status

Adding the LIBM flags like this fixes it.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Lars Karlitski (larsu) wrote :

Yes. Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches