package/open-plc-utils: fix build with gcc 10
authorFabrice Fontaine <fontaine.fabrice@gmail.com>
Wed, 23 Sep 2020 17:16:01 +0000 (19:16 +0200)
committerThomas Petazzoni <thomas.petazzoni@bootlin.com>
Wed, 23 Sep 2020 18:44:39 +0000 (20:44 +0200)
Fixes:
 - http://autobuild.buildroot.org/results/6b3064b64dea3d4aaf219f787429c25918101483

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
package/open-plc-utils/0002-plc-plc.h-fix-build-with-gcc-10.patch [new file with mode: 0644]

diff --git a/package/open-plc-utils/0002-plc-plc.h-fix-build-with-gcc-10.patch b/package/open-plc-utils/0002-plc-plc.h-fix-build-with-gcc-10.patch
new file mode 100644 (file)
index 0000000..063bfc5
--- /dev/null
@@ -0,0 +1,37 @@
+From 27222be0b8db59cae30a5fcca93255daea147bb6 Mon Sep 17 00:00:00 2001
+From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+Date: Wed, 23 Sep 2020 19:05:23 +0200
+Subject: [PATCH] plc/plc.h: fix build with gcc 10
+
+Drop plcproperty from plc/plc.h to avoid the following build failure
+with gcc 10 (which defaults to -fno-common):
+
+/home/buildroot/autobuild/run/instance-3/output-1/host/lib/gcc/arm-buildroot-linux-gnueabihf/10.2.0/../../../../arm-buildroot-linux-gnueabihf/bin/ld: rules.o:(.bss+0x0): multiple definition of `plcproperty'; pibruin.o:(.bss+0x0): first defined here
+
+Fixes:
+ - http://autobuild.buildroot.org/results/6b3064b64dea3d4aaf219f787429c25918101483
+
+Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+[Upstream status: https://github.com/qca/open-plc-utils/pull/141]
+---
+ plc/plc.h | 4 +---
+ 1 file changed, 1 insertion(+), 3 deletions(-)
+
+diff --git a/plc/plc.h b/plc/plc.h
+index 1fa32b53..208cf91a 100644
+--- a/plc/plc.h
++++ b/plc/plc.h
+@@ -496,9 +496,7 @@ struct __packed plcproperty
+       uint8_t DATA_FORMAT;
+       uint32_t DATA_LENGTH;
+       uint8_t DATA_BUFFER [128];
+-}
+-
+-plcproperty;
++};
+ #ifndef __GNUC__
+ #pragma pack (pop)
+-- 
+2.28.0
+