Merge lp:~ricmm/platform-api/gcc-4.7-abi into lp:platform-api

Proposed by Ricardo Mendoza
Status: Merged
Approved by: Ricardo Salveti
Approved revision: 75
Merged at revision: 75
Proposed branch: lp:~ricmm/platform-api/gcc-4.7-abi
Merge into: lp:platform-api
Diff against target: 24 lines (+3/-0)
2 files modified
debian/control (+1/-0)
debian/rules (+2/-0)
To merge this branch: bzr merge lp:~ricmm/platform-api/gcc-4.7-abi
Reviewer Review Type Date Requested Status
Ricardo Salveti (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+171643@code.launchpad.net

Commit message

GCC 4.8 introduces a change for ARM AAPCS calling conventions, this breaks ABI for us because the Android library is built with GCC 4.7.

Revert to building platform-api with GCC 4.7 until we can update the Android build.

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
Ricardo Salveti (rsalveti) wrote :

Good, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/control'
2--- debian/control 2013-06-14 21:06:57 +0000
3+++ debian/control 2013-06-26 19:23:26 +0000
4@@ -3,6 +3,7 @@
5 Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
6 Build-Depends: cmake,
7 debhelper (>= 9),
8+ g++-4.7,
9 libgl1-mesa-dev[!armhf],
10 libgles2-mesa-dev[armhf],
11 libhybris-dev,
12
13=== modified file 'debian/rules'
14--- debian/rules 2013-06-14 21:06:57 +0000
15+++ debian/rules 2013-06-26 19:23:26 +0000
16@@ -2,6 +2,8 @@
17
18 DPKG_GENSYMBOLS_CHECK_LEVEL = 4
19
20+export CXX=g++-4.7
21+
22 gles2_architectures := armhf
23 ifeq ($(DEB_HOST_ARCH),$(findstring $(DEB_HOST_ARCH), $(gles2_architectures)))
24 use_gles=1

Subscribers

People subscribed via source and target branches