* [Buildroot] [PATCH 1/1] package/w_scan: fix build with gcc 10
@ 2020-09-20 9:38 Fabrice Fontaine
2020-09-20 13:42 ` Thomas Petazzoni
2020-09-29 15:38 ` Peter Korsgaard
0 siblings, 2 replies; 3+ messages in thread
From: Fabrice Fontaine @ 2020-09-20 9:38 UTC (permalink / raw)
To: buildroot
Fixes:
- http://autobuild.buildroot.org/results/5640442cb4ba06bcbcf31540a64f5665862d39f7
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
...002-si_types-h-fix-build-with-gcc-10.patch | 37 +++++++++++++++++++
1 file changed, 37 insertions(+)
create mode 100644 package/w_scan/0002-si_types-h-fix-build-with-gcc-10.patch
diff --git a/package/w_scan/0002-si_types-h-fix-build-with-gcc-10.patch b/package/w_scan/0002-si_types-h-fix-build-with-gcc-10.patch
new file mode 100644
index 0000000000..50d2488190
--- /dev/null
+++ b/package/w_scan/0002-si_types-h-fix-build-with-gcc-10.patch
@@ -0,0 +1,37 @@
+si_types.h: fix build with gcc 10
+
+Drop service_t, p_service_t, transponder_t and p_transponder_t to avoid
+the following build failure with gcc 10 (which defaults to -fno-common):
+
+/home/giuliobenetti/autobuild/run/instance-3/output-1/host/bin/arm-buildroot-linux-gnueabihf-gcc -Wall -Wextra -Wno-comment -Wswitch-default -Wno-unused-parameter -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -Os -lrt -o w_scan atsc_psip_section.o countries.o descriptors.o diseqc.o dump-dvbscan.o dump-vdr.o dump-xine.o dump-mplayer.o dump-vlc-m3u.o dvbscan.o lnb.o parse-dvbscan.o satellites.o scan.o section.o tools.o emulate.o dump-xml.o iconv_codes.o char-coding.o
+/home/giuliobenetti/autobuild/run/instance-3/output-1/host/lib/gcc/arm-buildroot-linux-gnueabihf/10.2.0/../../../../arm-buildroot-linux-gnueabihf/bin/ld: descriptors.o:(.bss+0x404): multiple definition of `p_transponder_t'; countries.o:(.bss+0x0): first defined here
+/home/giuliobenetti/autobuild/run/instance-3/output-1/host/lib/gcc/arm-buildroot-linux-gnueabihf/10.2.0/../../../../arm-buildroot-linux-gnueabihf/bin/ld: descriptors.o:(.bss+0x408): multiple definition of `transponder_t'; countries.o:(.bss+0x4): first defined here
+/home/giuliobenetti/autobuild/run/instance-3/output-1/host/lib/gcc/arm-buildroot-linux-gnueabihf/10.2.0/../../../../arm-buildroot-linux-gnueabihf/bin/ld: descriptors.o:(.bss+0x480): multiple definition of `p_service_t'; countries.o:(.bss+0x7c): first defined here
+/home/giuliobenetti/autobuild/run/instance-3/output-1/host/lib/gcc/arm-buildroot-linux-gnueabihf/10.2.0/../../../../arm-buildroot-linux-gnueabihf/bin/ld: descriptors.o:(.bss+0x484): multiple definition of `service_t'; countries.o:(.bss+0x80): first defined here
+
+Fixes:
+ - http://autobuild.buildroot.org/results/5640442cb4ba06bcbcf31540a64f5665862d39f7
+
+Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+
+diff -Nura w_scan-20170107.orig/si_types.h w_scan-20170107/si_types.h
+--- w_scan-20170107.orig/si_types.h 2020-09-20 11:09:56.776963441 +0200
++++ w_scan-20170107/si_types.h 2020-09-20 11:13:13.149232478 +0200
+@@ -114,7 +114,7 @@
+ uint32_t logical_channel_number;
+ uint8_t running;
+ void * priv;
+-} service_t, * p_service_t;
++};
+
+ /*******************************************************************************
+ /* transponder type.
+@@ -201,7 +201,7 @@
+ /*----------------------------*/
+ char * network_name;
+ network_change_t network_change;
+-} __attribute__((packed)) transponder_t, * p_transponder_t;
++} __attribute__((packed));
+
+ /*******************************************************************************
+ /* satellite channel routing type.
--
2.28.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH 1/1] package/w_scan: fix build with gcc 10
2020-09-20 9:38 [Buildroot] [PATCH 1/1] package/w_scan: fix build with gcc 10 Fabrice Fontaine
@ 2020-09-20 13:42 ` Thomas Petazzoni
2020-09-29 15:38 ` Peter Korsgaard
1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2020-09-20 13:42 UTC (permalink / raw)
To: buildroot
On Sun, 20 Sep 2020 11:38:46 +0200
Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:
> Fixes:
> - http://autobuild.buildroot.org/results/5640442cb4ba06bcbcf31540a64f5665862d39f7
>
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
> ...002-si_types-h-fix-build-with-gcc-10.patch | 37 +++++++++++++++++++
> 1 file changed, 37 insertions(+)
> create mode 100644 package/w_scan/0002-si_types-h-fix-build-with-gcc-10.patch
I've applied to master, but I'm wondering if we really want to keep
this package. It is not maintained, the homepage is only in German.
Yann, you added it in 2013. Does it still makes sense to have it? Is
there some modern and maintained replacement available?
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH 1/1] package/w_scan: fix build with gcc 10
2020-09-20 9:38 [Buildroot] [PATCH 1/1] package/w_scan: fix build with gcc 10 Fabrice Fontaine
2020-09-20 13:42 ` Thomas Petazzoni
@ 2020-09-29 15:38 ` Peter Korsgaard
1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2020-09-29 15:38 UTC (permalink / raw)
To: buildroot
>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:
> Fixes:
> - http://autobuild.buildroot.org/results/5640442cb4ba06bcbcf31540a64f5665862d39f7
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
Committed to 2020.08.x, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-09-29 15:38 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-20 9:38 [Buildroot] [PATCH 1/1] package/w_scan: fix build with gcc 10 Fabrice Fontaine
2020-09-20 13:42 ` Thomas Petazzoni
2020-09-29 15:38 ` Peter Korsgaard
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.