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

Proposed by Ken VanDine
Status: Merged
Merged at revision: 111
Proposed branch: lp://staging/~ken-vandine/ido/link-libm
Merge into: lp://staging/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://staging/~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.staging.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
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches