* [Buildroot] [PATCH 1/1] package/elfutils: fix build with uclibc @ 2021-03-18 7:42 Fabrice Fontaine 2021-03-18 17:33 ` Peter Seiderer 2021-03-20 21:24 ` Thomas Petazzoni 0 siblings, 2 replies; 6+ messages in thread From: Fabrice Fontaine @ 2021-03-18 7:42 UTC (permalink / raw) To: buildroot Update first patch to fix a build failure with uclibc and elfutils in version 0.183.0 which errors out if obstack is not found since https://sourceware.org/git/?p=elfutils.git;a=commit;h=da855fc9cd415c288bfcb9de1f3d5eb329de0557 Indeed, obstack has been removed from uclibc-ng since version 1.0.21 and https://github.com/wbx-github/uclibc-ng/commit/0bd6bfb2b643ea2b4b1440dfd917ba752f0c0d15 Fixes: - http://autobuild.buildroot.org/results/003ae73c7b48ad6b9837fb19ca197061d20eaf6d Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> --- ...-Add-a-enable-disable-progs-configure-option.patch | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch b/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch index e51f749cf6..bac9bc1245 100644 --- a/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch +++ b/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch @@ -20,6 +20,8 @@ Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> [Rebased on 0.183] Signed-off-by: Peter Seiderer <ps.report@gmx.net> +[Fabrice: don't error out if obstack is not found] +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> --- Makefile.am | 6 +++++- configure.ac | 6 ++++++ @@ -60,6 +62,15 @@ index d345495..0dd4b1e 100644 dnl zlib is mandatory. save_LIBS="$LIBS" LIBS= +@@ -579,7 +585,7 @@ saved_LIBS="$LIBS" + AC_SEARCH_LIBS([_obstack_free], [obstack]) + LIBS="$saved_LIBS" + case "$ac_cv_search__obstack_free" in +- no) AC_MSG_FAILURE([failed to find _obstack_free]) ;; ++ no) obstack_LIBS= ;; + -l*) obstack_LIBS="$ac_cv_search__obstack_free" ;; + *) obstack_LIBS= ;; + esac -- 2.30.1 -- 2.30.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/1] package/elfutils: fix build with uclibc 2021-03-18 7:42 [Buildroot] [PATCH 1/1] package/elfutils: fix build with uclibc Fabrice Fontaine @ 2021-03-18 17:33 ` Peter Seiderer 2021-03-18 17:48 ` Fabrice Fontaine 2021-03-20 21:24 ` Thomas Petazzoni 1 sibling, 1 reply; 6+ messages in thread From: Peter Seiderer @ 2021-03-18 17:33 UTC (permalink / raw) To: buildroot Hello Fabrice, On Thu, 18 Mar 2021 08:42:19 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > Update first patch to fix a build failure with uclibc and elfutils in > version 0.183.0 which errors out if obstack is not found since > https://sourceware.org/git/?p=elfutils.git;a=commit;h=da855fc9cd415c288bfcb9de1f3d5eb329de0557 > > Indeed, obstack has been removed from uclibc-ng since version 1.0.21 and > https://github.com/wbx-github/uclibc-ng/commit/0bd6bfb2b643ea2b4b1440dfd917ba752f0c0d15 > > Fixes: > - http://autobuild.buildroot.org/results/003ae73c7b48ad6b9837fb19ca197061d20eaf6d > Thanks for taking care of the build failure, but... > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > --- > ...-Add-a-enable-disable-progs-configure-option.patch | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch b/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch > index e51f749cf6..bac9bc1245 100644 > --- a/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch > +++ b/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch > @@ -20,6 +20,8 @@ Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com> > Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> > [Rebased on 0.183] > Signed-off-by: Peter Seiderer <ps.report@gmx.net> > +[Fabrice: don't error out if obstack is not found] > +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > --- > Makefile.am | 6 +++++- > configure.ac | 6 ++++++ > @@ -60,6 +62,15 @@ index d345495..0dd4b1e 100644 > dnl zlib is mandatory. > save_LIBS="$LIBS" > LIBS= > +@@ -579,7 +585,7 @@ saved_LIBS="$LIBS" > + AC_SEARCH_LIBS([_obstack_free], [obstack]) > + LIBS="$saved_LIBS" > + case "$ac_cv_search__obstack_free" in > +- no) AC_MSG_FAILURE([failed to find _obstack_free]) ;; > ++ no) obstack_LIBS= ;; > + -l*) obstack_LIBS="$ac_cv_search__obstack_free" ;; > + *) obstack_LIBS= ;; > + esac > -- > 2.30.1 > Is this not a complete unrelated/new matter (and so belongs to an extra patch)? Regards, Peter ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/1] package/elfutils: fix build with uclibc 2021-03-18 17:33 ` Peter Seiderer @ 2021-03-18 17:48 ` Fabrice Fontaine 2021-03-18 19:13 ` Peter Seiderer 0 siblings, 1 reply; 6+ messages in thread From: Fabrice Fontaine @ 2021-03-18 17:48 UTC (permalink / raw) To: buildroot Hello Peter, Le jeu. 18 mars 2021 ? 18:33, Peter Seiderer <ps.report@gmx.net> a ?crit : > > Hello Fabrice, > > On Thu, 18 Mar 2021 08:42:19 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > Update first patch to fix a build failure with uclibc and elfutils in > > version 0.183.0 which errors out if obstack is not found since > > https://sourceware.org/git/?p=elfutils.git;a=commit;h=da855fc9cd415c288bfcb9de1f3d5eb329de0557 > > > > Indeed, obstack has been removed from uclibc-ng since version 1.0.21 and > > https://github.com/wbx-github/uclibc-ng/commit/0bd6bfb2b643ea2b4b1440dfd917ba752f0c0d15 > > > > Fixes: > > - http://autobuild.buildroot.org/results/003ae73c7b48ad6b9837fb19ca197061d20eaf6d > > > > Thanks for taking care of the build failure, but... > > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > > --- > > ...-Add-a-enable-disable-progs-configure-option.patch | 11 +++++++++++ > > 1 file changed, 11 insertions(+) > > > > diff --git a/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch b/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch > > index e51f749cf6..bac9bc1245 100644 > > --- a/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch > > +++ b/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch > > @@ -20,6 +20,8 @@ Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com> > > Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> > > [Rebased on 0.183] > > Signed-off-by: Peter Seiderer <ps.report@gmx.net> > > +[Fabrice: don't error out if obstack is not found] > > +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > > --- > > Makefile.am | 6 +++++- > > configure.ac | 6 ++++++ > > @@ -60,6 +62,15 @@ index d345495..0dd4b1e 100644 > > dnl zlib is mandatory. > > save_LIBS="$LIBS" > > LIBS= > > +@@ -579,7 +585,7 @@ saved_LIBS="$LIBS" > > + AC_SEARCH_LIBS([_obstack_free], [obstack]) > > + LIBS="$saved_LIBS" > > + case "$ac_cv_search__obstack_free" in > > +- no) AC_MSG_FAILURE([failed to find _obstack_free]) ;; > > ++ no) obstack_LIBS= ;; > > + -l*) obstack_LIBS="$ac_cv_search__obstack_free" ;; > > + *) obstack_LIBS= ;; > > + esac > > -- > > 2.30.1 > > > > Is this not a complete unrelated/new matter (and so belongs to an extra patch)? From my perspective, this is not unrelated as obstack has "always" been used by some of the elfutils program but I'm open to discussion. I can create a dedicated patch but it doesn't make sense to send it upstream without first patch. I don't know if first patch was sent and if so, what was upstream reaction? An other option would be to disable elfutils on uclibc as I don't think we should maintain not upstreamable patch forever. > > Regards, > Peter > > > Best Regards, Fabrice ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/1] package/elfutils: fix build with uclibc 2021-03-18 17:48 ` Fabrice Fontaine @ 2021-03-18 19:13 ` Peter Seiderer 2021-03-19 6:56 ` Fabrice Fontaine 0 siblings, 1 reply; 6+ messages in thread From: Peter Seiderer @ 2021-03-18 19:13 UTC (permalink / raw) To: buildroot Hello Fabrice, On Thu, 18 Mar 2021 18:48:24 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > Hello Peter, > > Le jeu. 18 mars 2021 ? 18:33, Peter Seiderer <ps.report@gmx.net> a ?crit : > > > > Hello Fabrice, > > > > On Thu, 18 Mar 2021 08:42:19 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > > > Update first patch to fix a build failure with uclibc and elfutils in > > > version 0.183.0 which errors out if obstack is not found since > > > https://sourceware.org/git/?p=elfutils.git;a=commit;h=da855fc9cd415c288bfcb9de1f3d5eb329de0557 > > > > > > Indeed, obstack has been removed from uclibc-ng since version 1.0.21 and > > > https://github.com/wbx-github/uclibc-ng/commit/0bd6bfb2b643ea2b4b1440dfd917ba752f0c0d15 > > > > > > Fixes: > > > - http://autobuild.buildroot.org/results/003ae73c7b48ad6b9837fb19ca197061d20eaf6d > > > > > > > Thanks for taking care of the build failure, but... > > > > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > > > --- > > > ...-Add-a-enable-disable-progs-configure-option.patch | 11 +++++++++++ > > > 1 file changed, 11 insertions(+) > > > > > > diff --git a/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch b/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch > > > index e51f749cf6..bac9bc1245 100644 > > > --- a/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch > > > +++ b/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch > > > @@ -20,6 +20,8 @@ Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com> > > > Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> > > > [Rebased on 0.183] > > > Signed-off-by: Peter Seiderer <ps.report@gmx.net> > > > +[Fabrice: don't error out if obstack is not found] > > > +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > > > --- > > > Makefile.am | 6 +++++- > > > configure.ac | 6 ++++++ > > > @@ -60,6 +62,15 @@ index d345495..0dd4b1e 100644 > > > dnl zlib is mandatory. > > > save_LIBS="$LIBS" > > > LIBS= > > > +@@ -579,7 +585,7 @@ saved_LIBS="$LIBS" > > > + AC_SEARCH_LIBS([_obstack_free], [obstack]) > > > + LIBS="$saved_LIBS" > > > + case "$ac_cv_search__obstack_free" in > > > +- no) AC_MSG_FAILURE([failed to find _obstack_free]) ;; > > > ++ no) obstack_LIBS= ;; > > > + -l*) obstack_LIBS="$ac_cv_search__obstack_free" ;; > > > + *) obstack_LIBS= ;; > > > + esac > > > -- > > > 2.30.1 > > > > > > > Is this not a complete unrelated/new matter (and so belongs to an extra patch)? > From my perspective, this is not unrelated as obstack has "always" > been used by some of the elfutils program but I'm open to discussion. O.k, good point ;-), maybe this reasoning is worth to be more emphasized in the commit message/patch-commit-message? > I can create a dedicated patch but it doesn't make sense to send it > upstream without first patch. For upstream the check/error-out on obstack should depend on the --{enable,disable}-progs option... > I don't know if first patch was sent and if so, what was upstream reaction? > An other option would be to disable elfutils on uclibc as I don't > think we should maintain not upstreamable patch forever. Give it a upstream chance... ;-) Nevertheless you can add my: Reviewed-by: Peter Seiderer <ps.report@gmx.net> Regards, Peter > > > > Regards, > > Peter > > > > > > > Best Regards, > > Fabrice ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/1] package/elfutils: fix build with uclibc 2021-03-18 19:13 ` Peter Seiderer @ 2021-03-19 6:56 ` Fabrice Fontaine 0 siblings, 0 replies; 6+ messages in thread From: Fabrice Fontaine @ 2021-03-19 6:56 UTC (permalink / raw) To: buildroot Hi Peter, Le jeu. 18 mars 2021 ? 20:13, Peter Seiderer <ps.report@gmx.net> a ?crit : > > Hello Fabrice, > > On Thu, 18 Mar 2021 18:48:24 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > Hello Peter, > > > > Le jeu. 18 mars 2021 ? 18:33, Peter Seiderer <ps.report@gmx.net> a ?crit : > > > > > > Hello Fabrice, > > > > > > On Thu, 18 Mar 2021 08:42:19 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > > > > > Update first patch to fix a build failure with uclibc and elfutils in > > > > version 0.183.0 which errors out if obstack is not found since > > > > https://sourceware.org/git/?p=elfutils.git;a=commit;h=da855fc9cd415c288bfcb9de1f3d5eb329de0557 > > > > > > > > Indeed, obstack has been removed from uclibc-ng since version 1.0.21 and > > > > https://github.com/wbx-github/uclibc-ng/commit/0bd6bfb2b643ea2b4b1440dfd917ba752f0c0d15 > > > > > > > > Fixes: > > > > - http://autobuild.buildroot.org/results/003ae73c7b48ad6b9837fb19ca197061d20eaf6d > > > > > > > > > > Thanks for taking care of the build failure, but... > > > > > > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > > > > --- > > > > ...-Add-a-enable-disable-progs-configure-option.patch | 11 +++++++++++ > > > > 1 file changed, 11 insertions(+) > > > > > > > > diff --git a/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch b/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch > > > > index e51f749cf6..bac9bc1245 100644 > > > > --- a/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch > > > > +++ b/package/elfutils/0001-Add-a-enable-disable-progs-configure-option.patch > > > > @@ -20,6 +20,8 @@ Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com> > > > > Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> > > > > [Rebased on 0.183] > > > > Signed-off-by: Peter Seiderer <ps.report@gmx.net> > > > > +[Fabrice: don't error out if obstack is not found] > > > > +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > > > > --- > > > > Makefile.am | 6 +++++- > > > > configure.ac | 6 ++++++ > > > > @@ -60,6 +62,15 @@ index d345495..0dd4b1e 100644 > > > > dnl zlib is mandatory. > > > > save_LIBS="$LIBS" > > > > LIBS= > > > > +@@ -579,7 +585,7 @@ saved_LIBS="$LIBS" > > > > + AC_SEARCH_LIBS([_obstack_free], [obstack]) > > > > + LIBS="$saved_LIBS" > > > > + case "$ac_cv_search__obstack_free" in > > > > +- no) AC_MSG_FAILURE([failed to find _obstack_free]) ;; > > > > ++ no) obstack_LIBS= ;; > > > > + -l*) obstack_LIBS="$ac_cv_search__obstack_free" ;; > > > > + *) obstack_LIBS= ;; > > > > + esac > > > > -- > > > > 2.30.1 > > > > > > > > > > Is this not a complete unrelated/new matter (and so belongs to an extra patch)? > > From my perspective, this is not unrelated as obstack has "always" > > been used by some of the elfutils program but I'm open to discussion. > > O.k, good point ;-), maybe this reasoning is worth to be more emphasized in > the commit message/patch-commit-message? > > > I can create a dedicated patch but it doesn't make sense to send it > > upstream without first patch. > > For upstream the check/error-out on obstack should depend on the > --{enable,disable}-progs option... > > > I don't know if first patch was sent and if so, what was upstream reaction? > > An other option would be to disable elfutils on uclibc as I don't > > think we should maintain not upstreamable patch forever. > > Give it a upstream chance... ;-) I had forgotten that I already sent them this patch 6 months ago and they didn't want to merge it: https://sourceware.org/pipermail/elfutils-devel/2020q4/002913.html I'll let others decide if we should continue to maintain this patch forever or if we should drop it and disable elfutils on uclibc. > > Nevertheless you can add my: > > Reviewed-by: Peter Seiderer <ps.report@gmx.net> > > Regards, > Peter > > > > > > > Regards, > > > Peter > > > > > > > > > > > Best Regards, > > > > Fabrice > Best Regards, Fabrice ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/1] package/elfutils: fix build with uclibc 2021-03-18 7:42 [Buildroot] [PATCH 1/1] package/elfutils: fix build with uclibc Fabrice Fontaine 2021-03-18 17:33 ` Peter Seiderer @ 2021-03-20 21:24 ` Thomas Petazzoni 1 sibling, 0 replies; 6+ messages in thread From: Thomas Petazzoni @ 2021-03-20 21:24 UTC (permalink / raw) To: buildroot On Thu, 18 Mar 2021 08:42:19 +0100 Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > Update first patch to fix a build failure with uclibc and elfutils in > version 0.183.0 which errors out if obstack is not found since > https://sourceware.org/git/?p=elfutils.git;a=commit;h=da855fc9cd415c288bfcb9de1f3d5eb329de0557 > > Indeed, obstack has been removed from uclibc-ng since version 1.0.21 and > https://github.com/wbx-github/uclibc-ng/commit/0bd6bfb2b643ea2b4b1440dfd917ba752f0c0d15 > > Fixes: > - http://autobuild.buildroot.org/results/003ae73c7b48ad6b9837fb19ca197061d20eaf6d > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > --- > ...-Add-a-enable-disable-progs-configure-option.patch | 11 +++++++++++ > 1 file changed, 11 insertions(+) Applied to master, thanks. Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-03-20 21:24 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2021-03-18 7:42 [Buildroot] [PATCH 1/1] package/elfutils: fix build with uclibc Fabrice Fontaine 2021-03-18 17:33 ` Peter Seiderer 2021-03-18 17:48 ` Fabrice Fontaine 2021-03-18 19:13 ` Peter Seiderer 2021-03-19 6:56 ` Fabrice Fontaine 2021-03-20 21:24 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox