* [Buildroot] [RESEND PATCH v2 1/1] package/sscep: fix linking error
@ 2023-01-16 18:05 Dario Binacchi
2023-01-16 20:20 ` Yann E. MORIN
0 siblings, 1 reply; 2+ messages in thread
From: Dario Binacchi @ 2023-01-16 18:05 UTC (permalink / raw)
To: buildroot; +Cc: Dario Binacchi, Yann E . MORIN, thomas.petazzoni
Apply the patch to fix the following build failure raised since the
addition of the package in commit
72fa60dc102679f51b228336ec38f5af78ff7646:
uclibc/sysroot/usr/lib/libc.a(getopt.os): in function `__GI_getopt':
getopt.c:(.text+0x598): multiple definition of `getopt'; src/getopt.o:getopt.c:(.text+0x0): first defined here
The patch has been submitted and accepted upstream to address this issue.
Fixes:
- http://autobuild.buildroot.net/results/844c7dd79f69ef2ea8f293bb73a2d4b7533aa97c
Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
---
Changes in v2:
- Add autobuild reference to commit message.
- The applied patch has ben accepted, so update the commit message.
.../sscep/0001-Fix-getopt-linking-error.patch | 55 +++++++++++++++++++
1 file changed, 55 insertions(+)
create mode 100644 package/sscep/0001-Fix-getopt-linking-error.patch
diff --git a/package/sscep/0001-Fix-getopt-linking-error.patch b/package/sscep/0001-Fix-getopt-linking-error.patch
new file mode 100644
index 000000000000..b32613b3cc18
--- /dev/null
+++ b/package/sscep/0001-Fix-getopt-linking-error.patch
@@ -0,0 +1,55 @@
+From c4fc8d9e282e05fcfa0cee9362c60742b26f4b33 Mon Sep 17 00:00:00 2001
+From: Dario Binacchi <dario.binacchi@amarulasolutions.com>
+Date: Fri, 9 Dec 2022 18:18:27 +0100
+Subject: [PATCH] Fix getopt linking error
+
+The buildroot project, to which the sscep application was added, has
+configurations that raise the following linking error:
+buildroot/output/host/lib/gcc/arc-buildroot-linux-uclibc/11.3.0/../../../../arc-buildroot-linux-uclibc/bin/ld: buildroot/output/host/bin/../arc-buildroot-linux-uclibc/sysroot/usr/lib/libc.a(getopt.os):(.data+0x8): multiple definition of `optind'; src/getopt.o:(.data+0x0): first defined here
+buildroot/output/host/lib/gcc/arc-buildroot-linux-uclibc/11.3.0/../../../../arc-buildroot-linux-uclibc/bin/ld: buildroot/output/host/bin/../arc-buildroot-linux-uclibc/sysroot/usr/lib/libc.a(getopt.os): in function `__GI_getopt':
+getopt.c:(.text+0x5a4): multiple definition of `getopt'; src/getopt.o:getopt.c:(.text+0x0): first defined here
+buildroot/output/host/lib/gcc/arc-buildroot-linux-uclibc/11.3.0/../../../../arc-buildroot-linux-uclibc/bin/ld: buildroot/output/host/bin/../arc-buildroot-linux-uclibc/sysroot/usr/lib/libc.a(getopt.os): in function `getopt_long':
+getopt.c:(.text+0x5b0): multiple definition of `getopt_long'; src/getopt.o:getopt.c:(.text+0x128): first defined here
+collect2: error: ld returned 1 exit status
+make[2]: *** [Makefile:507: sscep] Error 1
+make[1]: *** [package/pkg-generic.mk:293: buildroot/output/build/sscep-0.10.0/.stamp_built] Error 2
+
+The patch re-added a check that commit
+81f56f635259b9 ("Replaced GNU getopt by a BSD licensed alternative")
+removed.
+
+Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
+---
+ src/getopt.c | 12 ++++++++++++
+ 1 file changed, 12 insertions(+)
+
+diff --git a/src/getopt.c b/src/getopt.c
+index eae36a64afff..0109406ba4ac 100644
+--- a/src/getopt.c
++++ b/src/getopt.c
+@@ -31,6 +31,16 @@
+ #include <stddef.h>
+ #include <string.h>
+
++#define GETOPT_INTERFACE_VERSION 2
++#if !defined _LIBC && defined __GLIBC__ && __GLIBC__ >= 2
++# include <gnu-versions.h>
++# if _GNU_GETOPT_INTERFACE_VERSION == GETOPT_INTERFACE_VERSION
++# define ELIDE_CODE
++# endif
++#endif
++
++#ifndef ELIDE_CODE
++
+ char* optarg;
+ int optopt;
+ /* The variable optind [...] shall be initialized to 1 by the system. */
+@@ -226,3 +236,5 @@ int getopt_long(int argc, char* const argv[], const char* optstring,
+ ++optind;
+ return retval;
+ }
++
++#endif /* Not ELIDE_CODE. */
+--
+2.32.0
+
--
2.32.0
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [Buildroot] [RESEND PATCH v2 1/1] package/sscep: fix linking error
2023-01-16 18:05 [Buildroot] [RESEND PATCH v2 1/1] package/sscep: fix linking error Dario Binacchi
@ 2023-01-16 20:20 ` Yann E. MORIN
0 siblings, 0 replies; 2+ messages in thread
From: Yann E. MORIN @ 2023-01-16 20:20 UTC (permalink / raw)
To: Dario Binacchi; +Cc: thomas.petazzoni, buildroot
Dario, All,
On 2023-01-16 19:05 +0100, Dario Binacchi spake thusly:
> Apply the patch to fix the following build failure raised since the
> addition of the package in commit
> 72fa60dc102679f51b228336ec38f5af78ff7646:
>
> uclibc/sysroot/usr/lib/libc.a(getopt.os): in function `__GI_getopt':
> getopt.c:(.text+0x598): multiple definition of `getopt'; src/getopt.o:getopt.c:(.text+0x0): first defined here
>
> The patch has been submitted and accepted upstream to address this issue.
>
> Fixes:
> - http://autobuild.buildroot.net/results/844c7dd79f69ef2ea8f293bb73a2d4b7533aa97c
>
> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
Now that it is applied upstream, I made that an actual backport (to have
the proper sha1).
Applied to master, thanks.
Regards,
Yann E. MORIN.
> ---
> Changes in v2:
> - Add autobuild reference to commit message.
> - The applied patch has ben accepted, so update the commit message.
>
> .../sscep/0001-Fix-getopt-linking-error.patch | 55 +++++++++++++++++++
> 1 file changed, 55 insertions(+)
> create mode 100644 package/sscep/0001-Fix-getopt-linking-error.patch
>
> diff --git a/package/sscep/0001-Fix-getopt-linking-error.patch b/package/sscep/0001-Fix-getopt-linking-error.patch
> new file mode 100644
> index 000000000000..b32613b3cc18
> --- /dev/null
> +++ b/package/sscep/0001-Fix-getopt-linking-error.patch
> @@ -0,0 +1,55 @@
> +From c4fc8d9e282e05fcfa0cee9362c60742b26f4b33 Mon Sep 17 00:00:00 2001
> +From: Dario Binacchi <dario.binacchi@amarulasolutions.com>
> +Date: Fri, 9 Dec 2022 18:18:27 +0100
> +Subject: [PATCH] Fix getopt linking error
> +
> +The buildroot project, to which the sscep application was added, has
> +configurations that raise the following linking error:
> +buildroot/output/host/lib/gcc/arc-buildroot-linux-uclibc/11.3.0/../../../../arc-buildroot-linux-uclibc/bin/ld: buildroot/output/host/bin/../arc-buildroot-linux-uclibc/sysroot/usr/lib/libc.a(getopt.os):(.data+0x8): multiple definition of `optind'; src/getopt.o:(.data+0x0): first defined here
> +buildroot/output/host/lib/gcc/arc-buildroot-linux-uclibc/11.3.0/../../../../arc-buildroot-linux-uclibc/bin/ld: buildroot/output/host/bin/../arc-buildroot-linux-uclibc/sysroot/usr/lib/libc.a(getopt.os): in function `__GI_getopt':
> +getopt.c:(.text+0x5a4): multiple definition of `getopt'; src/getopt.o:getopt.c:(.text+0x0): first defined here
> +buildroot/output/host/lib/gcc/arc-buildroot-linux-uclibc/11.3.0/../../../../arc-buildroot-linux-uclibc/bin/ld: buildroot/output/host/bin/../arc-buildroot-linux-uclibc/sysroot/usr/lib/libc.a(getopt.os): in function `getopt_long':
> +getopt.c:(.text+0x5b0): multiple definition of `getopt_long'; src/getopt.o:getopt.c:(.text+0x128): first defined here
> +collect2: error: ld returned 1 exit status
> +make[2]: *** [Makefile:507: sscep] Error 1
> +make[1]: *** [package/pkg-generic.mk:293: buildroot/output/build/sscep-0.10.0/.stamp_built] Error 2
> +
> +The patch re-added a check that commit
> +81f56f635259b9 ("Replaced GNU getopt by a BSD licensed alternative")
> +removed.
> +
> +Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
> +---
> + src/getopt.c | 12 ++++++++++++
> + 1 file changed, 12 insertions(+)
> +
> +diff --git a/src/getopt.c b/src/getopt.c
> +index eae36a64afff..0109406ba4ac 100644
> +--- a/src/getopt.c
> ++++ b/src/getopt.c
> +@@ -31,6 +31,16 @@
> + #include <stddef.h>
> + #include <string.h>
> +
> ++#define GETOPT_INTERFACE_VERSION 2
> ++#if !defined _LIBC && defined __GLIBC__ && __GLIBC__ >= 2
> ++# include <gnu-versions.h>
> ++# if _GNU_GETOPT_INTERFACE_VERSION == GETOPT_INTERFACE_VERSION
> ++# define ELIDE_CODE
> ++# endif
> ++#endif
> ++
> ++#ifndef ELIDE_CODE
> ++
> + char* optarg;
> + int optopt;
> + /* The variable optind [...] shall be initialized to 1 by the system. */
> +@@ -226,3 +236,5 @@ int getopt_long(int argc, char* const argv[], const char* optstring,
> + ++optind;
> + return retval;
> + }
> ++
> ++#endif /* Not ELIDE_CODE. */
> +--
> +2.32.0
> +
> --
> 2.32.0
>
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-01-16 20:20 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-01-16 18:05 [Buildroot] [RESEND PATCH v2 1/1] package/sscep: fix linking error Dario Binacchi
2023-01-16 20:20 ` Yann E. MORIN
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox