Merge lp:~thomas-voss/net-cpp/fix-1452333 into lp:net-cpp

Proposed by Thomas Voß
Status: Needs review
Proposed branch: lp:~thomas-voss/net-cpp/fix-1452333
Merge into: lp:net-cpp
Diff against target: 32 lines (+0/-12)
2 files modified
debian/control (+0/-4)
debian/rules (+0/-8)
To merge this branch: bzr merge lp:~thomas-voss/net-cpp/fix-1452333
Reviewer Review Type Date Requested Status
Andreas Pokorny (community) Approve
Ubuntu Phablet Team Pending
Review via email: mp+264888@code.launchpad.net

Commit message

Remove explicit dependency on gcc version.

Description of the change

Remove explicit dependency on gcc version.

To post a comment you must log in.
Revision history for this message
Andreas Pokorny (andreas-pokorny) wrote :

package builds fine, and yes stating the CMAKE_CXX_COMPILER like that is not needed.

review: Approve

Unmerged revisions

50. By Thomas Voß

Remove explicit dependency on gcc version.

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 2015-02-25 10:27:29 +0000
3+++ debian/control 2015-07-15 17:52:29 +0000
4@@ -6,10 +6,6 @@
5 doxygen,
6 google-mock,
7 graphviz,
8-# We rely on C++11 features, and to prevent from ABI breaks
9-# in libstdc++ causing us issues, we explicitly select a G++
10-# version.
11- g++-4.9,
12 libboost-dev,
13 libboost-serialization-dev,
14 libboost-system-dev,
15
16=== modified file 'debian/rules'
17--- debian/rules 2014-06-27 08:16:12 +0000
18+++ debian/rules 2015-07-15 17:52:29 +0000
19@@ -5,13 +5,5 @@
20
21 include /usr/share/dpkg/default.mk
22
23-# Explicitly selecting a G{CC,++}-version here to avoid accidental
24-# ABI breaks introduced by toolchain updates.
25-export CC=$(DEB_HOST_GNU_TYPE)-gcc-4.9
26-export CXX=$(DEB_HOST_GNU_TYPE)-g++-4.9
27-
28 %:
29 dh $@ --fail-missing
30-
31-override_dh_auto_configure:
32- dh_auto_configure -- -DCMAKE_C_COMPILER=$(CC) -DCMAKE_CXX_COMPILER=$(CXX)

Subscribers

People subscribed via source and target branches