Merge lp:~ken-vandine/ido/link-libm into lp:ido/12.10

Proposed by Ken VanDine
Status: Merged
Merged at revision: 111
Proposed branch: lp:~ken-vandine/ido/link-libm
Merge into: lp:ido/12.10
Diff against target: 25 lines (+3/-1)
2 files modified
configure.ac (+2/-0)
src/Makefile.am (+1/-1)
To merge this branch: bzr merge lp:~ken-vandine/ido/link-libm
Reviewer Review Type Date Requested Status
jenkins (community) continuous-integration Needs Fixing
Charles Kerr (community) Approve
Review via email: mp+116498@code.launchpad.net

Description of the change

Use AC_CHECK_LIBM to get -lm now that gtk doesn't include that in it's .pc file

To post a comment you must log in.
Revision history for this message
jenkins (martin-mrazik+qa) wrote :

PASSED: Continuous integration, rev:110
http://jenkins.qa.ubuntu.com/job/ido-ci/4/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ido-ci/./label=quantal/4/console

review: Approve (continuous-integration)
Revision history for this message
Charles Kerr (charlesk) wrote :

Looks good, but one comment/question, does that $(LIBM) need to be added to libido3.pc.in as well?

Revision history for this message
Ken VanDine (ken-vandine) wrote :

> Looks good, but one comment/question, does that $(LIBM) need to be added to
> libido3.pc.in as well?

I don't think so.

Revision history for this message
Charles Kerr (charlesk) :
review: Approve
Revision history for this message
jenkins (martin-mrazik+qa) wrote :

FAILED: Autolanding.
No commit message was specified.
http://jenkins.qa.ubuntu.com/job/ido-autolanding/4/

review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'configure.ac'
--- configure.ac 2012-04-25 20:24:26 +0000
+++ configure.ac 2012-07-24 15:31:19 +0000
@@ -78,6 +78,8 @@
78AC_FUNC_MMAP78AC_FUNC_MMAP
79AC_CHECK_FUNCS([memset munmap strcasecmp strdup])79AC_CHECK_FUNCS([memset munmap strcasecmp strdup])
8080
81AC_CHECK_LIBM
82
81AC_ARG_WITH([gtk],83AC_ARG_WITH([gtk],
82 [AS_HELP_STRING([--with-gtk],84 [AS_HELP_STRING([--with-gtk],
83 [Which version of gtk to use @<:@default=3@:>@])],85 [Which version of gtk to use @<:@default=3@:>@])],
8486
=== modified file 'src/Makefile.am'
--- src/Makefile.am 2012-04-12 15:18:01 +0000
+++ src/Makefile.am 2012-07-24 15:31:19 +0000
@@ -82,7 +82,7 @@
82 idotimeline.h \82 idotimeline.h \
83 libido.h83 libido.h
8484
85libido_0_1_la_LIBADD = $(GTK_LIBS)85libido_0_1_la_LIBADD = $(GTK_LIBS) $(LIBM)
86libido_0_1_la_LDFLAGS = \86libido_0_1_la_LDFLAGS = \
87 $(GTK_LT_LDFLAGS) \87 $(GTK_LT_LDFLAGS) \
88 $(COVERAGE_LDFLAGS)88 $(COVERAGE_LDFLAGS)

Subscribers

People subscribed via source and target branches