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
=== modified file 'debian/changelog'
--- debian/changelog 2014-06-24 23:20:39 +0000
+++ debian/changelog 2014-06-27 08:42:05 +0000
@@ -1,3 +1,9 @@
1qtubuntu-sensors (0.6+14.10.20140624.3-0ubuntu2) UNRELEASED; urgency=medium
2
3 * Explicitly select g++-4.9 to prevent from ABI breaks.
4
5 -- Thomas Voß <thomas.voss@canonical.com> Fri, 27 Jun 2014 10:40:48 +0200
6
1qtubuntu-sensors (0.6+14.10.20140624.3-0ubuntu1) utopic; urgency=low7qtubuntu-sensors (0.6+14.10.20140624.3-0ubuntu1) utopic; urgency=low
28
3 [ Ricardo Mendoza ]9 [ Ricardo Mendoza ]
410
=== modified file 'debian/control'
--- debian/control 2014-06-20 12:29:17 +0000
+++ debian/control 2014-06-27 08:42:05 +0000
@@ -4,6 +4,10 @@
4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
5Build-Depends: debhelper (>= 9),5Build-Depends: debhelper (>= 9),
6 cmake,6 cmake,
7# We rely on C++11 features, and to prevent from ABI breaks
8# in libstdc++ causing us issues, we explicitly select a G++
9# version.
10 g++-4.9,
7 libgtest-dev,11 libgtest-dev,
8 libubuntu-application-api-dev (>= 2.1.0),12 libubuntu-application-api-dev (>= 2.1.0),
9 libprocess-cpp-dev (>= 0.0.1+14.04.20131212),13 libprocess-cpp-dev (>= 0.0.1+14.04.20131212),
1014
=== modified file 'debian/rules'
--- debian/rules 2014-06-23 13:43:58 +0000
+++ debian/rules 2014-06-27 08:42:05 +0000
@@ -7,9 +7,19 @@
7export DH_OPTIONS7export DH_OPTIONS
8export QT_QPA_PLATFORM = minimal8export QT_QPA_PLATFORM = minimal
99
10include /usr/share/dpkg/default.mk
11
12# Explicitly selecting a G{CC,++}-version here to avoid accidental
13# ABI breaks introduced by toolchain updates.
14export CC=$(DEB_HOST_GNU_TYPE)-gcc-4.9
15export CXX=$(DEB_HOST_GNU_TYPE)-g++-4.9
16
10%:17%:
11 dh $@ --parallel18 dh $@ --parallel
1219
20override_dh_auto_configure:
21 dh_auto_configure -- -DCMAKE_C_COMPILER=$(CC) -DCMAKE_CXX_COMPILER=$(CXX)
22
13override_dh_install:23override_dh_install:
14 dh_install --fail-missing24 dh_install --fail-missing
1525

Subscribers

People subscribed via source and target branches