Qt bindings for libdee

Merge lp://staging/~cjwatson/dee-qt/powerpc-pie into lp://staging/dee-qt

  1. powerpc-pie
  2. Merge into trunk
Proposed by Colin Watson
Status: Merged
Approved by: Albert Astals Cid
Approved revision: 88
Merged at revision: 88
Proposed branch: lp://staging/~cjwatson/dee-qt/powerpc-pie
Merge into: lp://staging/dee-qt
Diff against target: 17 lines (+7/-0)
1 file modified
debian/rules (+7/-0)
To merge this branch: bzr merge lp://staging/~cjwatson/dee-qt/powerpc-pie
Reviewer Review Type Date Requested Status
Albert Astals Cid (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+211308@code.staging.launchpad.net

Commit message

Work around pointer equality problem with -Bsymbolic-functions on powerpc/ppc64el.

Description of the change

dee-qt fails to build on powerpc/ppc64el with a signaltest failure. Albert Astals Cid tracked down http://lists.linaro.org/pipermail/linaro-toolchain/2014-January/003942.html which discusses a similar problem, and there's a hint in that thread that building with PIE support works around the problem. I've confirmed that this patch fixes the build on powerpc and ppc64el; for the sake of conservatism I've written it such that it only affects powerpc-based architectures.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:88
http://jenkins.qa.ubuntu.com/job/dee-qt-ci/19/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dee-qt-trusty-amd64-ci/7
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dee-qt-trusty-armhf-ci/7

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/dee-qt-ci/19/rebuild

review: Approve (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

I can confirm that this makes it build on the porter ppc machines, weather or not this is the appropiate/better bug i have no really opinion on since this is well below my knowledge level

review: Approve
Revision history for this message
Albert Astals Cid (aacid) wrote :

below in the stack i mean :D

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

to all changes: