* [PATCH] elfutils: Fix GNU_HASH QA Issue
@ 2020-09-02 3:22 Khem Raj
2020-09-02 9:41 ` [OE-core] " Ross Burton
0 siblings, 1 reply; 3+ messages in thread
From: Khem Raj @ 2020-09-02 3:22 UTC (permalink / raw)
To: openembedded-core; +Cc: Khem Raj
Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
.../recipes-devtools/elfutils/elfutils_0.180.bb | 1 +
.../elfutils/files/ldflags.patch | 17 +++++++++++++++++
2 files changed, 18 insertions(+)
create mode 100644 meta/recipes-devtools/elfutils/files/ldflags.patch
diff --git a/meta/recipes-devtools/elfutils/elfutils_0.180.bb b/meta/recipes-devtools/elfutils/elfutils_0.180.bb
index 9f8bfc24f3..d3ee750725 100644
--- a/meta/recipes-devtools/elfutils/elfutils_0.180.bb
+++ b/meta/recipes-devtools/elfutils/elfutils_0.180.bb
@@ -20,6 +20,7 @@ SRC_URI = "https://sourceware.org/elfutils/ftp/${PV}/${BP}.tar.bz2 \
file://run-ptest \
file://ptest.patch \
file://0001-tests-Makefile.am-compile-test_nlist-with-standard-C.patch \
+ file://ldflags.patch \
"
SRC_URI_append_libc-musl = " \
file://0001-musl-obstack-fts.patch \
diff --git a/meta/recipes-devtools/elfutils/files/ldflags.patch b/meta/recipes-devtools/elfutils/files/ldflags.patch
new file mode 100644
index 0000000000..ef1d65e2c3
--- /dev/null
+++ b/meta/recipes-devtools/elfutils/files/ldflags.patch
@@ -0,0 +1,17 @@
+Pass linker flags along to CCLD, this ensures proper linking
+w.r.t. hash_style selection during cross-compile
+
+Upstream-Status: Pending
+Signed-off-by: Khem Raj <raj.khem@gmail.com>
+
+--- a/tests/Makefile.am
++++ b/tests/Makefile.am
+@@ -86,7 +86,7 @@ backtrace-child-biarch$(EXEEXT): backtra
+ test-nlist$(EXEEXT): test-nlist.c
+ $(AM_V_CC)$(CC) $(DEFS) $(DEFAULT_INCLUDES) $(INCLUDES) \
+ $(AM_CPPFLAGS) $(CPPFLAGS) \
+- $(CFLAGS) $(test_nlist_LDADD) -o $@ $<
++ $(CFLAGS) $(test_nlist_LDADD) $(LDFLAGS) -o $@ $<
+
+ TESTS = run-arextract.sh run-arsymtest.sh run-ar.sh newfile test-nlist \
+ update1 update2 update3 update4 \
--
2.28.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [OE-core] [PATCH] elfutils: Fix GNU_HASH QA Issue
2020-09-02 3:22 [PATCH] elfutils: Fix GNU_HASH QA Issue Khem Raj
@ 2020-09-02 9:41 ` Ross Burton
2020-09-02 15:22 ` Khem Raj
0 siblings, 1 reply; 3+ messages in thread
From: Ross Burton @ 2020-09-02 9:41 UTC (permalink / raw)
To: Khem Raj, openembedded-core
On 02/09/2020 04:22, Khem Raj wrote:
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ---
> .../recipes-devtools/elfutils/elfutils_0.180.bb | 1 +
> .../elfutils/files/ldflags.patch | 17 +++++++++++++++++
> 2 files changed, 18 insertions(+)
> create mode 100644 meta/recipes-devtools/elfutils/files/ldflags.patch
>
> diff --git a/meta/recipes-devtools/elfutils/elfutils_0.180.bb b/meta/recipes-devtools/elfutils/elfutils_0.180.bb
> index 9f8bfc24f3..d3ee750725 100644
> --- a/meta/recipes-devtools/elfutils/elfutils_0.180.bb
> +++ b/meta/recipes-devtools/elfutils/elfutils_0.180.bb
> @@ -20,6 +20,7 @@ SRC_URI = "https://sourceware.org/elfutils/ftp/${PV}/${BP}.tar.bz2 \
> file://run-ptest \
> file://ptest.patch \
> file://0001-tests-Makefile.am-compile-test_nlist-with-standard-C.patch \
> + file://ldflags.patch \
> "
> SRC_URI_append_libc-musl = " \
> file://0001-musl-obstack-fts.patch \
> diff --git a/meta/recipes-devtools/elfutils/files/ldflags.patch b/meta/recipes-devtools/elfutils/files/ldflags.patch
> new file mode 100644
> index 0000000000..ef1d65e2c3
> --- /dev/null
> +++ b/meta/recipes-devtools/elfutils/files/ldflags.patch
> @@ -0,0 +1,17 @@
> +Pass linker flags along to CCLD, this ensures proper linking
> +w.r.t. hash_style selection during cross-compile
Did you re-run the test suite? That binary is explicitly compiled with
minimal flags, so adding the LDFLAGS might actually break the test. For
the same reason, you can set Upstream-Status to Inappropriate.
I've a fix here for the same issue that uses INSANE_SKIP instead.
Ross
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [OE-core] [PATCH] elfutils: Fix GNU_HASH QA Issue
2020-09-02 9:41 ` [OE-core] " Ross Burton
@ 2020-09-02 15:22 ` Khem Raj
0 siblings, 0 replies; 3+ messages in thread
From: Khem Raj @ 2020-09-02 15:22 UTC (permalink / raw)
To: Ross Burton; +Cc: Patches and discussions about the oe-core layer
On Wed, Sep 2, 2020 at 2:41 AM Ross Burton <ross@burtonini.com> wrote:
>
> On 02/09/2020 04:22, Khem Raj wrote:
> > Signed-off-by: Khem Raj <raj.khem@gmail.com>
> > ---
> > .../recipes-devtools/elfutils/elfutils_0.180.bb | 1 +
> > .../elfutils/files/ldflags.patch | 17 +++++++++++++++++
> > 2 files changed, 18 insertions(+)
> > create mode 100644 meta/recipes-devtools/elfutils/files/ldflags.patch
> >
> > diff --git a/meta/recipes-devtools/elfutils/elfutils_0.180.bb b/meta/recipes-devtools/elfutils/elfutils_0.180.bb
> > index 9f8bfc24f3..d3ee750725 100644
> > --- a/meta/recipes-devtools/elfutils/elfutils_0.180.bb
> > +++ b/meta/recipes-devtools/elfutils/elfutils_0.180.bb
> > @@ -20,6 +20,7 @@ SRC_URI = "https://sourceware.org/elfutils/ftp/${PV}/${BP}.tar.bz2 \
> > file://run-ptest \
> > file://ptest.patch \
> > file://0001-tests-Makefile.am-compile-test_nlist-with-standard-C.patch \
> > + file://ldflags.patch \
> > "
> > SRC_URI_append_libc-musl = " \
> > file://0001-musl-obstack-fts.patch \
> > diff --git a/meta/recipes-devtools/elfutils/files/ldflags.patch b/meta/recipes-devtools/elfutils/files/ldflags.patch
> > new file mode 100644
> > index 0000000000..ef1d65e2c3
> > --- /dev/null
> > +++ b/meta/recipes-devtools/elfutils/files/ldflags.patch
> > @@ -0,0 +1,17 @@
> > +Pass linker flags along to CCLD, this ensures proper linking
> > +w.r.t. hash_style selection during cross-compile
>
> Did you re-run the test suite? That binary is explicitly compiled with
> minimal flags, so adding the LDFLAGS might actually break the test. For
> the same reason, you can set Upstream-Status to Inappropriate.
>
I have executed on qemu and it worked ok.
> I've a fix here for the same issue that uses INSANE_SKIP instead.
>
> Ross
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-09-02 15:22 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-02 3:22 [PATCH] elfutils: Fix GNU_HASH QA Issue Khem Raj
2020-09-02 9:41 ` [OE-core] " Ross Burton
2020-09-02 15:22 ` Khem Raj
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.