Merge lp:~jbicha/syncevolution/fix-gcc6-ftbfs into lp:~phablet-team/syncevolution/ubuntu

Proposed by Jeremy Bícha
Status: Merged
Merged at revision: 4643
Proposed branch: lp:~jbicha/syncevolution/fix-gcc6-ftbfs
Merge into: lp:~phablet-team/syncevolution/ubuntu
Diff against target: 105 lines (+72/-0)
4 files modified
debian/changelog (+10/-0)
debian/patches/0003-Add-missing-casts-from-shared_ptr-to-bool-to-fix-FTB.patch (+60/-0)
debian/patches/series (+1/-0)
debian/rules (+1/-0)
To merge this branch: bzr merge lp:~jbicha/syncevolution/fix-gcc6-ftbfs
Reviewer Review Type Date Requested Status
Ubuntu Phablet Team Pending
Review via email: mp+302980@code.launchpad.net

Description of the change

To post a comment you must log in.
4643. By Jeremy Bícha

* Add 0003-Add-missing-casts-from-shared_ptr-to-bool-to-fix-FTB.patch
  to add missing casts from shared_ptr to bool, fixes FTBFS with GCC 6
* Fix remaining FTBFS with GCC 6 by downgrading the C++ dialect to gnu++98
  (Closes: #811624) (LP: #1613405)

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 2016-07-06 14:15:27 +0000
3+++ debian/changelog 2016-08-19 16:49:31 +0000
4@@ -1,3 +1,13 @@
5+syncevolution (1.5.1+16.10.20160706-0ubuntu2) yakkety; urgency=medium
6+
7+ [ Tino Mettler ]
8+ * Add 0003-Add-missing-casts-from-shared_ptr-to-bool-to-fix-FTB.patch
9+ to add missing casts from shared_ptr to bool, fixes FTBFS with GCC 6
10+ * Fix remaining FTBFS with GCC 6 by downgrading the C++ dialect to gnu++98
11+ (Closes: #811624) (LP: #1613405)
12+
13+ -- Jeremy Bicha <jbicha@ubuntu.com> Fri, 19 Aug 2016 12:48:59 -0400
14+
15 syncevolution (1.5.1+16.10.20160706-0ubuntu1) yakkety; urgency=medium
16
17 * New upstream release
18
19=== added file 'debian/patches/0003-Add-missing-casts-from-shared_ptr-to-bool-to-fix-FTB.patch'
20--- debian/patches/0003-Add-missing-casts-from-shared_ptr-to-bool-to-fix-FTB.patch 1970-01-01 00:00:00 +0000
21+++ debian/patches/0003-Add-missing-casts-from-shared_ptr-to-bool-to-fix-FTB.patch 2016-08-19 16:49:31 +0000
22@@ -0,0 +1,60 @@
23+From 9286b8851524a85f5c9d7352f5a89c550c13f315 Mon Sep 17 00:00:00 2001
24+From: Tino Mettler <tino+debian@tikei.de>
25+Date: Wed, 6 Jul 2016 14:33:19 +0200
26+Subject: [PATCH] Add missing casts from shared_ptr to bool to fix FTBFS with
27+ GCC 6
28+
29+---
30+ src/dbus/server/session.h | 2 +-
31+ src/syncevo/Logging.h | 4 ++--
32+ src/syncevo/StringDataBlob.h | 2 +-
33+ 3 files changed, 4 insertions(+), 4 deletions(-)
34+
35+diff --git a/src/dbus/server/session.h b/src/dbus/server/session.h
36+index af46f5d..b762bab 100644
37+--- a/src/dbus/server/session.h
38++++ b/src/dbus/server/session.h
39+@@ -425,7 +425,7 @@ public:
40+ void setServerAlerted(bool serverAlerted) { m_serverAlerted = serverAlerted; }
41+
42+ void initServer(SharedBuffer data, const std::string &messageType);
43+- void setStubConnection(const boost::shared_ptr<Connection> c) { m_connection = c; m_useConnection = c; }
44++ void setStubConnection(const boost::shared_ptr<Connection> c) { m_connection = c; m_useConnection = static_cast<bool> (c); }
45+ boost::weak_ptr<Connection> getStubConnection() { return m_connection; }
46+ bool useStubConnection() { return m_useConnection; }
47+
48+diff --git a/src/syncevo/Logging.h b/src/syncevo/Logging.h
49+index ce9a249..5deb01b 100644
50+--- a/src/syncevo/Logging.h
51++++ b/src/syncevo/Logging.h
52+@@ -255,7 +255,7 @@ class Logger
53+ Handle &operator = (const Handle &other) throw ();
54+ ~Handle() throw ();
55+
56+- operator bool () const { return m_logger; }
57++ operator bool () const { return static_cast<bool> (m_logger); }
58+ bool operator == (Logger *logger) const { return m_logger.get() == logger; }
59+ Logger *get() const { return m_logger.get(); }
60+
61+@@ -398,7 +398,7 @@ template<class L> class PushLogger : boost::noncopyable
62+ }
63+ }
64+
65+- operator bool () const { return m_logger; }
66++ operator bool () const { return static_cast<bool> (m_logger); }
67+
68+ void reset(const Logger::Handle &logger)
69+ {
70+diff --git a/src/syncevo/StringDataBlob.h b/src/syncevo/StringDataBlob.h
71+index 62bc4c7..14f0d61 100644
72+--- a/src/syncevo/StringDataBlob.h
73++++ b/src/syncevo/StringDataBlob.h
74+@@ -53,7 +53,7 @@ class StringDataBlob : public DataBlob
75+
76+ virtual boost::shared_ptr<std::string> getData() { return m_data; }
77+ virtual std::string getName() const { return m_name; }
78+- virtual bool exists() const { return m_data; }
79++ virtual bool exists() const { return static_cast<bool> (m_data); }
80+ virtual bool isReadonly() const { return m_readonly; }
81+ };
82+
83
84=== modified file 'debian/patches/series'
85--- debian/patches/series 2016-07-05 17:27:00 +0000
86+++ debian/patches/series 2016-08-19 16:49:31 +0000
87@@ -1,5 +1,6 @@
88 0001-Fix-FTBFS-on-kfreebsd-due-to-missing-SOCK_CLOEXEC.patch
89 0002-Avoid-register-unecessary-timezones.patch
90+0003-Add-missing-casts-from-shared_ptr-to-bool-to-fix-FTB.patch
91 fix-photo-merging.patch
92 calendar-limit-date-sync.diff
93 Use-90-days-as-default-value-for-syncInterval.patch
94
95=== modified file 'debian/rules'
96--- debian/rules 2016-07-05 17:27:00 +0000
97+++ debian/rules 2016-08-19 16:49:31 +0000
98@@ -6,6 +6,7 @@
99 export DH_VERBOSE=1
100 export DEB_BUILD_MAINT_OPTIONS = hardening=+all
101 export DEB_CFLAGS_MAINT_APPEND = -Wall -pedantic
102+export DEB_CXXFLAGS_MAINT_APPEND = -std=gnu++98
103 export DEB_LDFLAGS_MAINT_APPEND = -Wl,--as-needed
104
105 %:

Subscribers

People subscribed via source and target branches

to all changes: