Merge lp:~unity-api-team/unity-scopes-api/gcc-5-fixes into lp:unity-scopes-api

Proposed by Marcus Tomlinson
Status: Merged
Approved by: Marcus Tomlinson
Approved revision: 620
Merged at revision: 341
Proposed branch: lp:~unity-api-team/unity-scopes-api/gcc-5-fixes
Merge into: lp:unity-scopes-api
Diff against target: 96 lines (+21/-22)
4 files modified
debian/control (+1/-2)
src/scopes/internal/SearchReplyImpl.cpp (+2/-2)
test/gtest/scopes/internal/ScopeConfig/CMakeLists.txt (+16/-16)
test/gtest/scopes/internal/zmq_middleware/RegistryI/RegistryI_test.cpp (+2/-2)
To merge this branch: bzr merge lp:~unity-api-team/unity-scopes-api/gcc-5-fixes
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Marcus Tomlinson (community) Approve
Review via email: mp+265765@code.launchpad.net

This proposal supersedes a proposal from 2015-07-24.

Commit message

Updated standards version to 3.9.6 in debian/control.
Removed copyonly from configure() calls because cmake 3.2.2 doesn't like it.
Removed g++ dependency from debian/control because bzr bd emitted a warning.

Description of the change

Updated standards version to 3.9.6 in debian/control.
Removed copyonly from configure() calls because cmake 3.2.2 doesn't like it.
Removed g++ dependency from debian/control because bzr bd emitted a warning.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
Marcus Tomlinson (marcustomlinson) wrote :

+1

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
620. By Marcus Tomlinson

Reverted symbols removal

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-05-18 13:09:19 +0000
3+++ debian/control 2015-07-24 11:46:27 +0000
4@@ -7,7 +7,6 @@
5 dbus-test-runner,
6 debhelper (>= 9),
7 doxygen,
8- g++,
9 google-mock,
10 graphviz,
11 libaccounts-glib-dev,
12@@ -32,7 +31,7 @@
13 valgrind,
14 libnet-cpp-dev (>=1.1.0),
15 python-tornado
16-Standards-Version: 3.9.5
17+Standards-Version: 3.9.6
18 XS-Testsuite: autopkgtest
19 Section: libs
20 Homepage: https://launchpad.net/unity-scopes-api
21
22=== modified file 'src/scopes/internal/SearchReplyImpl.cpp'
23--- src/scopes/internal/SearchReplyImpl.cpp 2015-02-05 09:35:42 +0000
24+++ src/scopes/internal/SearchReplyImpl.cpp 2015-07-24 11:46:27 +0000
25@@ -223,7 +223,7 @@
26 {
27 // Open a temporary file for writing.
28 tmp_path = mw_proxy_->mw_base()->runtime()->cache_directory() + "/" + cache_file_name + "XXXXXX";
29- auto opener = [tmp_path]()
30+ auto opener = [&tmp_path]()
31 {
32 int tmp_fd = mkstemp(const_cast<char*>(tmp_path.c_str()));
33 if (tmp_fd == -1)
34@@ -232,7 +232,7 @@
35 }
36 return tmp_fd;
37 };
38- auto closer = [tmp_path](int fd)
39+ auto closer = [&tmp_path](int fd)
40 {
41 if (::close(fd) == -1)
42 {
43
44=== modified file 'test/gtest/scopes/internal/ScopeConfig/CMakeLists.txt'
45--- test/gtest/scopes/internal/ScopeConfig/CMakeLists.txt 2015-05-14 02:50:35 +0000
46+++ test/gtest/scopes/internal/ScopeConfig/CMakeLists.txt 2015-07-24 11:46:27 +0000
47@@ -1,19 +1,19 @@
48-configure_file(complete_config.ini.in complete_config.ini copyonly)
49-configure_file(mandatory_config.ini.in mandatory_config.ini copyonly)
50-configure_file(ttl_small.ini.in ttl_small.ini copyonly)
51-configure_file(ttl_medium.ini.in ttl_medium.ini copyonly)
52-configure_file(bad_timeout.ini.in bad_timeout.ini copyonly)
53-configure_file(bad_ttl.ini.in bad_ttl.ini copyonly)
54-configure_file(bad_version.ini.in bad_version.ini copyonly)
55-configure_file(bad_child_ids.ini.in bad_child_ids.ini copyonly)
56-configure_file(empty_child_ids.ini.in empty_child_ids.ini copyonly)
57-configure_file(single_child_id.ini.in single_child_id.ini copyonly)
58-configure_file(bad_keywords.ini.in bad_keywords.ini copyonly)
59-configure_file(empty_keywords.ini.in empty_keywords.ini copyonly)
60-configure_file(single_keyword.ini.in single_keyword.ini copyonly)
61-configure_file(empty_display_name.ini.in empty_display_name.ini copyonly)
62-configure_file(empty_description.ini.in empty_description.ini copyonly)
63-configure_file(empty_author.ini.in empty_author.ini copyonly)
64+configure_file(complete_config.ini.in complete_config.ini)
65+configure_file(mandatory_config.ini.in mandatory_config.ini)
66+configure_file(ttl_small.ini.in ttl_small.ini)
67+configure_file(ttl_medium.ini.in ttl_medium.ini)
68+configure_file(bad_timeout.ini.in bad_timeout.ini)
69+configure_file(bad_ttl.ini.in bad_ttl.ini)
70+configure_file(bad_version.ini.in bad_version.ini)
71+configure_file(bad_child_ids.ini.in bad_child_ids.ini)
72+configure_file(empty_child_ids.ini.in empty_child_ids.ini)
73+configure_file(single_child_id.ini.in single_child_id.ini)
74+configure_file(bad_keywords.ini.in bad_keywords.ini)
75+configure_file(empty_keywords.ini.in empty_keywords.ini)
76+configure_file(single_keyword.ini.in single_keyword.ini)
77+configure_file(empty_display_name.ini.in empty_display_name.ini)
78+configure_file(empty_description.ini.in empty_description.ini)
79+configure_file(empty_author.ini.in empty_author.ini)
80
81 add_definitions(-DCOMPLETE_CONFIG="${CMAKE_CURRENT_BINARY_DIR}/complete_config.ini")
82 add_definitions(-DMANDATORY_CONFIG="${CMAKE_CURRENT_BINARY_DIR}/mandatory_config.ini")
83
84=== modified file 'test/gtest/scopes/internal/zmq_middleware/RegistryI/RegistryI_test.cpp'
85--- test/gtest/scopes/internal/zmq_middleware/RegistryI/RegistryI_test.cpp 2015-02-23 03:35:58 +0000
86+++ test/gtest/scopes/internal/zmq_middleware/RegistryI/RegistryI_test.cpp 2015-07-24 11:46:27 +0000
87@@ -36,8 +36,8 @@
88 #pragma push_macro("BOOST_RANGE_ENABLE_CONCEPT_ASSERT")
89
90 #if BOOST_VERSION / 100000 == 1
91-# if ((BOOST_VERSION / 100) % 1000) == 57
92-# define BOOST_RANGE_ENABLE_CONCEPT_ASSERT 0 // Iterator requirements are too strict with boost 1.57.
93+# if ((BOOST_VERSION / 100) % 1000) == 57 || ((BOOST_VERSION / 100) % 1000) == 58
94+# define BOOST_RANGE_ENABLE_CONCEPT_ASSERT 0 // Iterator requirements are too strict with boost 1.57/1.58.
95 # endif
96 # endif
97

Subscribers

People subscribed via source and target branches

to all changes: