Skip to contentSkip to conversation

dbus-cpp: fix build with gcc 7.x

PATCH New Thomas Petazzoni 2 participants 3 messages Aug 21, 2017 – Aug 23, 2017

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 dbus-cpp: fix build with gcc 7.x

This commit adds a patch to dbus-cpp to make it build with gcc 7.x.

Fixes:

http://autobuild.buildroot.net/results/07a7559c0efeeda16c239e0fa06259d4cd48c71b/

---
 ...3-src-pipe.c-fix-build-error-with-gcc-7.x.patch | 36 ++++++++++++++++++++++
 1 file changed, 36 insertions(+)
 create mode 100644 package/dbus-cpp/0003-src-pipe.c-fix-build-error-with-gcc-7.x.patch
package/dbus-cpp/0003-src-pipe.c-fix-build-error-with-gcc-7.x.patch new file +36 -0
new file mode 100644
index 0000000..6f3e624
@@ -0,0 +1,36 @@
+From 167042116f523f90911f3a28c4ec065918d56cc9 Mon Sep 17 00:00:00 2001
+From: Thomas Petazzoni <[email protected]>
+Date: Tue, 22 Aug 2017 00:02:52 +0200
+Subject: [PATCH] src/pipe.c: fix build error with gcc 7.x
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+The write function takes a char* as argument, not a char.
+
+Fixes:
+
+pipe.cpp: In member function ?void DBus::Pipe::signal()?:
+pipe.cpp:86:29: error: invalid conversion from ?char? to ?const void*? [-fpermissive]
+   ::write(_fd_write, '\0', 1);
+                             ^
+
+Signed-off-by: Thomas Petazzoni <[email protected]>
+---
+ src/pipe.cpp | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/pipe.cpp b/src/pipe.cpp
+index 01211b3..d0572c7 100644
+--- a/src/pipe.cpp
++++ b/src/pipe.cpp
+@@ -83,5 +83,5 @@ ssize_t Pipe::read(void *buffer, unsigned int &nbytes)
+ void Pipe::signal()
+ {
+   // TODO: ignoring return of read/write generates warning; maybe relevant for eventloop work...
+-  ::write(_fd_write, '\0', 1);
++  ::write(_fd_write, "\0", 1);
+ }
+-- 
+2.9.4
+
Signature · 1 line
2.9.4
#1 lore raw

PATCH 1/1 dbus-cpp: fix build with gcc 7.x

Am Tue, 22 Aug 2017 00:04:42 +0200 schrieb Thomas Petazzoni:

Quoted 5 lines

This commit adds a patch to dbus-cpp to make it build with gcc 7.x.

Fixes:

http://autobuild.buildroot.net/

results/07a7559c0efeeda16c239e0fa06259d4cd48c71b/

Quoted 2 lines

[ build-tested using http://autobuild.buildroot.net/
results/07a/07a7559c0efeeda16c239e0fa06259d4cd48c71b/defconfig ]

author Applied
#2 lore raw

PATCH 1/1 dbus-cpp: fix build with gcc 7.x

In reply to Thomas Petazzoni

Hello,

Quoted 11 lines · Thomas Petazzoni

This commit adds a patch to dbus-cpp to make it build with gcc 7.x.

Fixes:

http://autobuild.buildroot.net/results/07a7559c0efeeda16c239e0fa06259d4cd48c71b/

---
 ...3-src-pipe.c-fix-build-error-with-gcc-7.x.patch | 36 ++++++++++++++++++++++
 1 file changed, 36 insertions(+)
 create mode 100644 package/dbus-cpp/0003-src-pipe.c-fix-build-error-with-gcc-7.x.patch

Applied to master, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com