Merge lp:~thomas-voss/qtubuntu-sensors/explicit-gcc-version into lp:qtubuntu-sensors

Proposed by Thomas Voß
Status: Needs review
Proposed branch: lp:~thomas-voss/qtubuntu-sensors/explicit-gcc-version
Merge into: lp:qtubuntu-sensors
Diff against target: 51 lines (+20/-0)
3 files modified
debian/changelog (+6/-0)
debian/control (+4/-0)
debian/rules (+10/-0)
To merge this branch: bzr merge lp:~thomas-voss/qtubuntu-sensors/explicit-gcc-version
Reviewer Review Type Date Requested Status
Colin Watson Approve
PS Jenkins bot continuous-integration Approve
Matthias Klose Pending
Ricardo Mendoza Pending
Review via email: mp+224767@code.launchpad.net

Commit message

Explicitly select g++-4.9 to prevent from ABI breaks.

Description of the change

Explicitly select g++-4.9 to prevent from ABI breaks.

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
Colin Watson (cjwatson) wrote :

I'd suggest bumping the build-dependency version on libprocess-cpp-dev, but otherwise fine.

review: Approve

Unmerged revisions

69. By Thomas Voß

Explicitly select g++-4.9 to prevent from ABI breaks.

68. By Thomas Voß

Explicitly select g++-4.9 to prevent from ABI breaks.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2014-06-24 23:20:39 +0000
3+++ debian/changelog 2014-06-27 08:42:05 +0000
4@@ -1,3 +1,9 @@
5+qtubuntu-sensors (0.6+14.10.20140624.3-0ubuntu2) UNRELEASED; urgency=medium
6+
7+ * Explicitly select g++-4.9 to prevent from ABI breaks.
8+
9+ -- Thomas Voß <thomas.voss@canonical.com> Fri, 27 Jun 2014 10:40:48 +0200
10+
11 qtubuntu-sensors (0.6+14.10.20140624.3-0ubuntu1) utopic; urgency=low
12
13 [ Ricardo Mendoza ]
14
15=== modified file 'debian/control'
16--- debian/control 2014-06-20 12:29:17 +0000
17+++ debian/control 2014-06-27 08:42:05 +0000
18@@ -4,6 +4,10 @@
19 Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
20 Build-Depends: debhelper (>= 9),
21 cmake,
22+# We rely on C++11 features, and to prevent from ABI breaks
23+# in libstdc++ causing us issues, we explicitly select a G++
24+# version.
25+ g++-4.9,
26 libgtest-dev,
27 libubuntu-application-api-dev (>= 2.1.0),
28 libprocess-cpp-dev (>= 0.0.1+14.04.20131212),
29
30=== modified file 'debian/rules'
31--- debian/rules 2014-06-23 13:43:58 +0000
32+++ debian/rules 2014-06-27 08:42:05 +0000
33@@ -7,9 +7,19 @@
34 export DH_OPTIONS
35 export QT_QPA_PLATFORM = minimal
36
37+include /usr/share/dpkg/default.mk
38+
39+# Explicitly selecting a G{CC,++}-version here to avoid accidental
40+# ABI breaks introduced by toolchain updates.
41+export CC=$(DEB_HOST_GNU_TYPE)-gcc-4.9
42+export CXX=$(DEB_HOST_GNU_TYPE)-g++-4.9
43+
44 %:
45 dh $@ --parallel
46
47+override_dh_auto_configure:
48+ dh_auto_configure -- -DCMAKE_C_COMPILER=$(CC) -DCMAKE_CXX_COMPILER=$(CXX)
49+
50 override_dh_install:
51 dh_install --fail-missing
52

Subscribers

People subscribed via source and target branches