Message ID | 20240603133938.3552266-2-richard.purdie@linuxfoundation.org |
---|---|
State | Accepted, archived |
Commit | 7c8553f81bccc3e8c2bb1116ee1e89f5f8af4c9e |
Headers | show |
Series | [1/4] libtool: 2.4.7 -> 2.5.0 | expand |
Thank you for your submission. Patchtest identified one or more issues with the patch. Please see the log below for more information: --- Testing patch /home/patchtest/share/mboxes/2-4-autotools-libtool-Drop-libtool-sysroot-patch-as-not-needed.patch FAIL: test commit message presence: Please include a commit message on your patch explaining the change (test_mbox.TestMbox.test_commit_message_presence) PASS: test CVE check ignore (test_metadata.TestMetadata.test_cve_check_ignore) PASS: test Signed-off-by presence (test_mbox.TestMbox.test_signed_off_by_presence) PASS: test author valid (test_mbox.TestMbox.test_author_valid) PASS: test lic files chksum modified not mentioned (test_metadata.TestMetadata.test_lic_files_chksum_modified_not_mentioned) PASS: test max line length (test_metadata.TestMetadata.test_max_line_length) PASS: test mbox format (test_mbox.TestMbox.test_mbox_format) PASS: test non-AUH upgrade (test_mbox.TestMbox.test_non_auh_upgrade) PASS: test shortlog format (test_mbox.TestMbox.test_shortlog_format) PASS: test shortlog length (test_mbox.TestMbox.test_shortlog_length) PASS: test target mailing list (test_mbox.TestMbox.test_target_mailing_list) SKIP: pretest pylint: No python related patches, skipping test (test_python_pylint.PyLint.pretest_pylint) SKIP: pretest src uri left files: Patch cannot be merged (test_metadata.TestMetadata.pretest_src_uri_left_files) SKIP: test CVE tag format: No new CVE patches introduced (test_patch.TestPatch.test_cve_tag_format) SKIP: test Signed-off-by presence: No new CVE patches introduced (test_patch.TestPatch.test_signed_off_by_presence) SKIP: test Upstream-Status presence: No new CVE patches introduced (test_patch.TestPatch.test_upstream_status_presence_format) SKIP: test bugzilla entry format: No bug ID found (test_mbox.TestMbox.test_bugzilla_entry_format) SKIP: test lic files chksum presence: No added recipes, skipping test (test_metadata.TestMetadata.test_lic_files_chksum_presence) SKIP: test license presence: No added recipes, skipping test (test_metadata.TestMetadata.test_license_presence) SKIP: test pylint: No python related patches, skipping test (test_python_pylint.PyLint.test_pylint) SKIP: test series merge on head: Merge test is disabled for now (test_mbox.TestMbox.test_series_merge_on_head) SKIP: test src uri left files: Patch cannot be merged (test_metadata.TestMetadata.test_src_uri_left_files) SKIP: test summary presence: No added recipes, skipping test (test_metadata.TestMetadata.test_summary_presence) --- Please address the issues identified and submit a new revision of the patch, or alternatively, reply to this email with an explanation of why the patch should be accepted. If you believe these results are due to an error in patchtest, please submit a bug at https://bugzilla.yoctoproject.org/ (use the 'Patchtest' category under 'Yocto Project Subprojects'). For more information on specific failures, see: https://wiki.yoctoproject.org/wiki/Patchtest. Thank you!
Always lovely to see a libtool patch go away, nice! On Mon, Jun 3, 2024 at 1:39 PM Richard Purdie via lists.openembedded.org <richard.purdie=linuxfoundation.org@lists.openembedded.org> wrote: > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> > --- > meta/classes-global/insane.bbclass | 2 +- > meta/classes-recipe/autotools.bbclass | 9 +- > .../libtool/libtool-2.5.0.inc | 1 - > .../libtool/libtool-native_2.5.0.bb | 2 +- > ...e-the-with-sysroot-option-to-avoid-c.patch | 171 ------------------ > ...ool.m4-Handle-as-a-sysroot-correctly.patch | 6 +- > 6 files changed, 6 insertions(+), 185 deletions(-) > delete mode 100644 > meta/recipes-devtools/libtool/libtool/0002-libtool.m4-Rename-the-with-sysroot-option-to-avoid-c.patch > > diff --git a/meta/classes-global/insane.bbclass > b/meta/classes-global/insane.bbclass > index 151cf0c4eac..822d190a304 100644 > --- a/meta/classes-global/insane.bbclass > +++ b/meta/classes-global/insane.bbclass > @@ -58,7 +58,7 @@ enabled tests are listed here, the do_package_qa task > will run under fakeroot." > > ALL_QA = "${WARN_QA} ${ERROR_QA}" > > -UNKNOWN_CONFIGURE_OPT_IGNORE ?= "--enable-nls --disable-nls > --disable-silent-rules --disable-dependency-tracking --with-libtool-sysroot > --disable-static" > +UNKNOWN_CONFIGURE_OPT_IGNORE ?= "--enable-nls --disable-nls > --disable-silent-rules --disable-dependency-tracking --disable-static" > > # This is a list of directories that are expected to be empty. > QA_EMPTY_DIRS ?= " \ > diff --git a/meta/classes-recipe/autotools.bbclass > b/meta/classes-recipe/autotools.bbclass > index 7ee1b0d9c74..9ebdca47680 100644 > --- a/meta/classes-recipe/autotools.bbclass > +++ b/meta/classes-recipe/autotools.bbclass > @@ -57,12 +57,6 @@ export CXXFLAGS_FOR_BUILD="${BUILD_CXXFLAGS}" > export LD_FOR_BUILD = "${BUILD_LD}" > export LDFLAGS_FOR_BUILD = "${BUILD_LDFLAGS}" > > -def append_libtool_sysroot(d): > - # Only supply libtool sysroot option for non-native packages > - if not bb.data.inherits_class('native', d): > - return '--with-libtool-sysroot=${STAGING_DIR_HOST}' > - return "" > - > CONFIGUREOPTS = " --build=${BUILD_SYS} \ > --host=${HOST_SYS} \ > --target=${TARGET_SYS} \ > @@ -81,8 +75,7 @@ CONFIGUREOPTS = " --build=${BUILD_SYS} \ > --infodir=${infodir} \ > --mandir=${mandir} \ > --disable-silent-rules \ > - ${CONFIGUREOPT_DEPTRACK} \ > - ${@append_libtool_sysroot(d)}" > + ${CONFIGUREOPT_DEPTRACK}" > CONFIGUREOPT_DEPTRACK ?= "--disable-dependency-tracking" > > CACHED_CONFIGUREVARS ?= "" > diff --git a/meta/recipes-devtools/libtool/libtool-2.5.0.inc > b/meta/recipes-devtools/libtool/libtool-2.5.0.inc > index dd6215476e8..a609de6369b 100644 > --- a/meta/recipes-devtools/libtool/libtool-2.5.0.inc > +++ b/meta/recipes-devtools/libtool/libtool-2.5.0.inc > @@ -10,7 +10,6 @@ LIC_FILES_CHKSUM = > "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \ > > SRC_URI = "https://alpha.gnu.org/gnu/libtool/libtool-${PV}.tar.gz \ > > file://0001-ltmain.in-Handle-trailing-slashes-on-install-command.patch \ > - > file://0002-libtool.m4-Rename-the-with-sysroot-option-to-avoid-c.patch \ > > file://0003-ltmain.in-Add-missing-sysroot-to-library-path.patch \ > > file://0004-ltmain.sh-Fix-sysroot-paths-being-encoded-into-RPATH.patch \ > > file://0005-ltmain.in-Don-t-encode-RATHS-which-match-default-lin.patch \ > diff --git a/meta/recipes-devtools/libtool/libtool-native_2.5.0.bb > b/meta/recipes-devtools/libtool/libtool-native_2.5.0.bb > index 19024f7830c..90c06f0ec3f 100644 > --- a/meta/recipes-devtools/libtool/libtool-native_2.5.0.bb > +++ b/meta/recipes-devtools/libtool/libtool-native_2.5.0.bb > @@ -4,7 +4,7 @@ DEPENDS = "" > > inherit native > > -EXTRA_OECONF = " --with-libtool-sysroot=${STAGING_DIR_NATIVE}" > +EXTRA_OECONF = " --with-sysroot=${STAGING_DIR_NATIVE}" > > do_configure:prepend () { > # Remove any existing libtool m4 since old stale versions would > break > diff --git > a/meta/recipes-devtools/libtool/libtool/0002-libtool.m4-Rename-the-with-sysroot-option-to-avoid-c.patch > b/meta/recipes-devtools/libtool/libtool/0002-libtool.m4-Rename-the-with-sysroot-option-to-avoid-c.patch > deleted file mode 100644 > index 156622bb54b..00000000000 > --- > a/meta/recipes-devtools/libtool/libtool/0002-libtool.m4-Rename-the-with-sysroot-option-to-avoid-c.patch > +++ /dev/null > @@ -1,171 +0,0 @@ > -From: Khem Raj <raj.khem@gmail.com> > -Subject: [PATCH 02/12] libtool.m4: Rename the --with-sysroot option to > avoid conflict with gcc/binutils > - > -This patch renames the --with-sysroot option to --with-libtool-sysroot > -to avoid namespace conflict with binutils, gcc and other toolchain > -components since these componets also add that option to configure > -and this becomes confusing and conflicting otherwise. > - > -Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> > - > -Upstream report: > -http://lists.gnu.org/archive/html/libtool/2010-10/msg00048.html > - > -Upstream-Status: Submitted [ > https://lists.gnu.org/archive/html/libtool-patches/2021-10/msg00014.html] > - > -Index: libtool-2.5.0/m4/libtool.m4 > -=================================================================== > ---- libtool-2.5.0.orig/m4/libtool.m4 > -+++ libtool-2.5.0/m4/libtool.m4 > -@@ -1243,16 +1243,16 @@ _LT_DECL([], [ECHO], [1], [An echo progr > - AC_DEFUN([_LT_WITH_SYSROOT], > - [m4_require([_LT_DECL_SED])dnl > - AC_MSG_CHECKING([for sysroot]) > --AC_ARG_WITH([sysroot], > --[AS_HELP_STRING([--with-sysroot@<:@=DIR@:>@], > -+AC_ARG_WITH([libtool-sysroot], > -+[AS_HELP_STRING([--with-libtool-sysroot@<:@=DIR@:>@], > - [Search for dependent libraries within DIR (or the compiler's sysroot > - if not specified).])], > --[], [with_sysroot=no]) > -+[], [with_libtool_sysroot=no]) > - > - dnl lt_sysroot will always be passed unquoted. We quote it here > - dnl in case the user passed a directory name. > - lt_sysroot= > --case $with_sysroot in #( > -+case $with_libtool_sysroot in #( > - yes) > - if test yes = "$GCC"; then > - # Trim trailing / since we'll always append absolute paths and we > want > -@@ -1261,12 +1261,12 @@ case $with_sysroot in #( > - fi > - ;; #( > - /*) > -- lt_sysroot=`echo "$with_sysroot" | $SED -e "$sed_quote_subst"` > -+ lt_sysroot=`echo "$with_libtool_sysroot" | $SED -e "$sed_quote_subst"` > - ;; #( > - no|'') > - ;; #( > - *) > -- AC_MSG_RESULT([$with_sysroot]) > -+ AC_MSG_RESULT([$with_libtool_sysroot]) > - AC_MSG_ERROR([The sysroot must be an absolute path.]) > - ;; > - esac > -Index: libtool-2.5.0/tests/sysroot.at > -=================================================================== > ---- libtool-2.5.0.orig/tests/sysroot.at > -+++ libtool-2.5.0/tests/sysroot.at > -@@ -65,7 +65,7 @@ while read file; do > - done]) > - > - LDFLAGS="$LDFLAGS --sysroot=$sysroot -no-undefined" > --configure_options="$configure_options --with-sysroot=$sysroot > --prefix=$prefix" > -+configure_options="$configure_options --with-libtool-sysroot=$sysroot > --prefix=$prefix" > - > - #??? > - if test PATH = "$shlibpath_var"; then > -@@ -115,7 +115,7 @@ AM_INIT_AUTOMAKE([foreign]) > - AC_PROG_CC > - AC_CONFIG_SRCDIR([lib2.c]) > - LT_INIT > --sysroot=$with_sysroot > -+sysroot=$with_libtool_sysroot > - AC_SUBST([sysroot]) > - AC_OUTPUT(Makefile) > - ]]) > -@@ -156,7 +156,7 @@ AM_INIT_AUTOMAKE([foreign]) > - AC_PROG_CC > - AC_CONFIG_SRCDIR([prog.c]) > - LT_INIT > --sysroot=$with_sysroot > -+sysroot=$with_libtool_sysroot > - AC_SUBST([sysroot]) > - AC_OUTPUT(Makefile) > - ]]) > -Index: libtool-2.5.0/tests/testsuite > -=================================================================== > ---- libtool-2.5.0.orig/tests/testsuite > -+++ libtool-2.5.0/tests/testsuite > -@@ -49054,7 +49054,7 @@ $at_traceon; } > - > - > - LDFLAGS="$LDFLAGS --sysroot=$sysroot -no-undefined" > --configure_options="$configure_options --with-sysroot=$sysroot > --prefix=$prefix" > -+configure_options="$configure_options --with-libtool-sysroot=$sysroot > --prefix=$prefix" > - > - #??? > - if test PATH = "$shlibpath_var"; then > -@@ -49269,7 +49269,7 @@ AM_INIT_AUTOMAKE([foreign]) > - AC_PROG_CC > - AC_CONFIG_SRCDIR([lib2.c]) > - LT_INIT > --sysroot=$with_sysroot > -+sysroot=$with_libtool_sysroot > - AC_SUBST([sysroot]) > - AC_OUTPUT(Makefile) > - _ATEOF > -@@ -49463,7 +49463,7 @@ AM_INIT_AUTOMAKE([foreign]) > - AC_PROG_CC > - AC_CONFIG_SRCDIR([prog.c]) > - LT_INIT > --sysroot=$with_sysroot > -+sysroot=$with_libtool_sysroot > - AC_SUBST([sysroot]) > - AC_OUTPUT(Makefile) > - _ATEOF > -@@ -49821,7 +49821,7 @@ $at_traceon; } > - > - > - LDFLAGS="$LDFLAGS --sysroot=$sysroot -no-undefined" > --configure_options="$configure_options --with-sysroot=$sysroot > --prefix=$prefix" > -+configure_options="$configure_options --with-libtool-sysroot=$sysroot > --prefix=$prefix" > - > - #??? > - if test PATH = "$shlibpath_var"; then > -@@ -50036,7 +50036,7 @@ AM_INIT_AUTOMAKE([foreign]) > - AC_PROG_CC > - AC_CONFIG_SRCDIR([lib2.c]) > - LT_INIT > --sysroot=$with_sysroot > -+sysroot=$with_libtool_sysroot > - AC_SUBST([sysroot]) > - AC_OUTPUT(Makefile) > - _ATEOF > -@@ -50230,7 +50230,7 @@ AM_INIT_AUTOMAKE([foreign]) > - AC_PROG_CC > - AC_CONFIG_SRCDIR([prog.c]) > - LT_INIT > --sysroot=$with_sysroot > -+sysroot=$with_libtool_sysroot > - AC_SUBST([sysroot]) > - AC_OUTPUT(Makefile) > - _ATEOF > -@@ -50588,7 +50588,7 @@ $at_traceon; } > - > - > - LDFLAGS="$LDFLAGS --sysroot=$sysroot -no-undefined" > --configure_options="$configure_options --with-sysroot=$sysroot > --prefix=$prefix" > -+configure_options="$configure_options --with-libtool-sysroot=$sysroot > --prefix=$prefix" > - > - #??? > - if test PATH = "$shlibpath_var"; then > -@@ -50803,7 +50803,7 @@ AM_INIT_AUTOMAKE([foreign]) > - AC_PROG_CC > - AC_CONFIG_SRCDIR([lib2.c]) > - LT_INIT > --sysroot=$with_sysroot > -+sysroot=$with_libtool_sysroot > - AC_SUBST([sysroot]) > - AC_OUTPUT(Makefile) > - _ATEOF > -@@ -50997,7 +50997,7 @@ AM_INIT_AUTOMAKE([foreign]) > - AC_PROG_CC > - AC_CONFIG_SRCDIR([prog.c]) > - LT_INIT > --sysroot=$with_sysroot > -+sysroot=$with_libtool_sysroot > - AC_SUBST([sysroot]) > - AC_OUTPUT(Makefile) > - _ATEOF > diff --git > a/meta/recipes-devtools/libtool/libtool/0006-libtool.m4-Handle-as-a-sysroot-correctly.patch > b/meta/recipes-devtools/libtool/libtool/0006-libtool.m4-Handle-as-a-sysroot-correctly.patch > index 435c52c7301..feb1048b554 100644 > --- > a/meta/recipes-devtools/libtool/libtool/0006-libtool.m4-Handle-as-a-sysroot-correctly.patch > +++ > b/meta/recipes-devtools/libtool/libtool/0006-libtool.m4-Handle-as-a-sysroot-correctly.patch > @@ -17,7 +17,7 @@ Index: libtool-2.5.0/m4/libtool.m4 > @@ -1253,18 +1253,18 @@ dnl lt_sysroot will always be passed unq > dnl in case the user passed a directory name. > lt_sysroot= > - case $with_libtool_sysroot in #( > + case $with_sysroot in #( > - yes) > + no) > if test yes = "$GCC"; then > @@ -29,10 +29,10 @@ Index: libtool-2.5.0/m4/libtool.m4 > + yes|''|/) > + ;; #( > /*) > - lt_sysroot=`echo "$with_libtool_sysroot" | $SED -e "$sed_quote_subst"` > + lt_sysroot=`echo "$with_sysroot" | $SED -e "$sed_quote_subst"` > ;; #( > - no|'') > - ;; #( > *) > - AC_MSG_RESULT([$with_libtool_sysroot]) > + AC_MSG_RESULT([$with_sysroot]) > AC_MSG_ERROR([The sysroot must be an absolute path.]) > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#200250): > https://lists.openembedded.org/g/openembedded-core/message/200250 > Mute This Topic: https://lists.openembedded.org/mt/106461688/3617123 > Group Owner: openembedded-core+owner@lists.openembedded.org > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [ > kergoth@gmail.com] > -=-=-=-=-=-=-=-=-=-=-=- > >
On Mon, 2024-06-03 at 15:59 +0000, Christopher Larson wrote:
> Always lovely to see a libtool patch go away, nice!
Yes!
We've had reasonable success at getting quite a few merged upstream so
the upgrade removes a load too.
https://savannah.gnu.org/news/?id=10631
5 commits into 2.5.0 and we had more into 2.4.7 and 2.4.6.
Its taken far too long but things do look/feel cleaner now.
Cheers,
Richard
diff --git a/meta/classes-global/insane.bbclass b/meta/classes-global/insane.bbclass index 151cf0c4eac..822d190a304 100644 --- a/meta/classes-global/insane.bbclass +++ b/meta/classes-global/insane.bbclass @@ -58,7 +58,7 @@ enabled tests are listed here, the do_package_qa task will run under fakeroot." ALL_QA = "${WARN_QA} ${ERROR_QA}" -UNKNOWN_CONFIGURE_OPT_IGNORE ?= "--enable-nls --disable-nls --disable-silent-rules --disable-dependency-tracking --with-libtool-sysroot --disable-static" +UNKNOWN_CONFIGURE_OPT_IGNORE ?= "--enable-nls --disable-nls --disable-silent-rules --disable-dependency-tracking --disable-static" # This is a list of directories that are expected to be empty. QA_EMPTY_DIRS ?= " \ diff --git a/meta/classes-recipe/autotools.bbclass b/meta/classes-recipe/autotools.bbclass index 7ee1b0d9c74..9ebdca47680 100644 --- a/meta/classes-recipe/autotools.bbclass +++ b/meta/classes-recipe/autotools.bbclass @@ -57,12 +57,6 @@ export CXXFLAGS_FOR_BUILD="${BUILD_CXXFLAGS}" export LD_FOR_BUILD = "${BUILD_LD}" export LDFLAGS_FOR_BUILD = "${BUILD_LDFLAGS}" -def append_libtool_sysroot(d): - # Only supply libtool sysroot option for non-native packages - if not bb.data.inherits_class('native', d): - return '--with-libtool-sysroot=${STAGING_DIR_HOST}' - return "" - CONFIGUREOPTS = " --build=${BUILD_SYS} \ --host=${HOST_SYS} \ --target=${TARGET_SYS} \ @@ -81,8 +75,7 @@ CONFIGUREOPTS = " --build=${BUILD_SYS} \ --infodir=${infodir} \ --mandir=${mandir} \ --disable-silent-rules \ - ${CONFIGUREOPT_DEPTRACK} \ - ${@append_libtool_sysroot(d)}" + ${CONFIGUREOPT_DEPTRACK}" CONFIGUREOPT_DEPTRACK ?= "--disable-dependency-tracking" CACHED_CONFIGUREVARS ?= "" diff --git a/meta/recipes-devtools/libtool/libtool-2.5.0.inc b/meta/recipes-devtools/libtool/libtool-2.5.0.inc index dd6215476e8..a609de6369b 100644 --- a/meta/recipes-devtools/libtool/libtool-2.5.0.inc +++ b/meta/recipes-devtools/libtool/libtool-2.5.0.inc @@ -10,7 +10,6 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \ SRC_URI = "https://alpha.gnu.org/gnu/libtool/libtool-${PV}.tar.gz \ file://0001-ltmain.in-Handle-trailing-slashes-on-install-command.patch \ - file://0002-libtool.m4-Rename-the-with-sysroot-option-to-avoid-c.patch \ file://0003-ltmain.in-Add-missing-sysroot-to-library-path.patch \ file://0004-ltmain.sh-Fix-sysroot-paths-being-encoded-into-RPATH.patch \ file://0005-ltmain.in-Don-t-encode-RATHS-which-match-default-lin.patch \ diff --git a/meta/recipes-devtools/libtool/libtool-native_2.5.0.bb b/meta/recipes-devtools/libtool/libtool-native_2.5.0.bb index 19024f7830c..90c06f0ec3f 100644 --- a/meta/recipes-devtools/libtool/libtool-native_2.5.0.bb +++ b/meta/recipes-devtools/libtool/libtool-native_2.5.0.bb @@ -4,7 +4,7 @@ DEPENDS = "" inherit native -EXTRA_OECONF = " --with-libtool-sysroot=${STAGING_DIR_NATIVE}" +EXTRA_OECONF = " --with-sysroot=${STAGING_DIR_NATIVE}" do_configure:prepend () { # Remove any existing libtool m4 since old stale versions would break diff --git a/meta/recipes-devtools/libtool/libtool/0002-libtool.m4-Rename-the-with-sysroot-option-to-avoid-c.patch b/meta/recipes-devtools/libtool/libtool/0002-libtool.m4-Rename-the-with-sysroot-option-to-avoid-c.patch deleted file mode 100644 index 156622bb54b..00000000000 --- a/meta/recipes-devtools/libtool/libtool/0002-libtool.m4-Rename-the-with-sysroot-option-to-avoid-c.patch +++ /dev/null @@ -1,171 +0,0 @@ -From: Khem Raj <raj.khem@gmail.com> -Subject: [PATCH 02/12] libtool.m4: Rename the --with-sysroot option to avoid conflict with gcc/binutils - -This patch renames the --with-sysroot option to --with-libtool-sysroot -to avoid namespace conflict with binutils, gcc and other toolchain -components since these componets also add that option to configure -and this becomes confusing and conflicting otherwise. - -Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> - -Upstream report: -http://lists.gnu.org/archive/html/libtool/2010-10/msg00048.html - -Upstream-Status: Submitted [https://lists.gnu.org/archive/html/libtool-patches/2021-10/msg00014.html] - -Index: libtool-2.5.0/m4/libtool.m4 -=================================================================== ---- libtool-2.5.0.orig/m4/libtool.m4 -+++ libtool-2.5.0/m4/libtool.m4 -@@ -1243,16 +1243,16 @@ _LT_DECL([], [ECHO], [1], [An echo progr - AC_DEFUN([_LT_WITH_SYSROOT], - [m4_require([_LT_DECL_SED])dnl - AC_MSG_CHECKING([for sysroot]) --AC_ARG_WITH([sysroot], --[AS_HELP_STRING([--with-sysroot@<:@=DIR@:>@], -+AC_ARG_WITH([libtool-sysroot], -+[AS_HELP_STRING([--with-libtool-sysroot@<:@=DIR@:>@], - [Search for dependent libraries within DIR (or the compiler's sysroot - if not specified).])], --[], [with_sysroot=no]) -+[], [with_libtool_sysroot=no]) - - dnl lt_sysroot will always be passed unquoted. We quote it here - dnl in case the user passed a directory name. - lt_sysroot= --case $with_sysroot in #( -+case $with_libtool_sysroot in #( - yes) - if test yes = "$GCC"; then - # Trim trailing / since we'll always append absolute paths and we want -@@ -1261,12 +1261,12 @@ case $with_sysroot in #( - fi - ;; #( - /*) -- lt_sysroot=`echo "$with_sysroot" | $SED -e "$sed_quote_subst"` -+ lt_sysroot=`echo "$with_libtool_sysroot" | $SED -e "$sed_quote_subst"` - ;; #( - no|'') - ;; #( - *) -- AC_MSG_RESULT([$with_sysroot]) -+ AC_MSG_RESULT([$with_libtool_sysroot]) - AC_MSG_ERROR([The sysroot must be an absolute path.]) - ;; - esac -Index: libtool-2.5.0/tests/sysroot.at -=================================================================== ---- libtool-2.5.0.orig/tests/sysroot.at -+++ libtool-2.5.0/tests/sysroot.at -@@ -65,7 +65,7 @@ while read file; do - done]) - - LDFLAGS="$LDFLAGS --sysroot=$sysroot -no-undefined" --configure_options="$configure_options --with-sysroot=$sysroot --prefix=$prefix" -+configure_options="$configure_options --with-libtool-sysroot=$sysroot --prefix=$prefix" - - #??? - if test PATH = "$shlibpath_var"; then -@@ -115,7 +115,7 @@ AM_INIT_AUTOMAKE([foreign]) - AC_PROG_CC - AC_CONFIG_SRCDIR([lib2.c]) - LT_INIT --sysroot=$with_sysroot -+sysroot=$with_libtool_sysroot - AC_SUBST([sysroot]) - AC_OUTPUT(Makefile) - ]]) -@@ -156,7 +156,7 @@ AM_INIT_AUTOMAKE([foreign]) - AC_PROG_CC - AC_CONFIG_SRCDIR([prog.c]) - LT_INIT --sysroot=$with_sysroot -+sysroot=$with_libtool_sysroot - AC_SUBST([sysroot]) - AC_OUTPUT(Makefile) - ]]) -Index: libtool-2.5.0/tests/testsuite -=================================================================== ---- libtool-2.5.0.orig/tests/testsuite -+++ libtool-2.5.0/tests/testsuite -@@ -49054,7 +49054,7 @@ $at_traceon; } - - - LDFLAGS="$LDFLAGS --sysroot=$sysroot -no-undefined" --configure_options="$configure_options --with-sysroot=$sysroot --prefix=$prefix" -+configure_options="$configure_options --with-libtool-sysroot=$sysroot --prefix=$prefix" - - #??? - if test PATH = "$shlibpath_var"; then -@@ -49269,7 +49269,7 @@ AM_INIT_AUTOMAKE([foreign]) - AC_PROG_CC - AC_CONFIG_SRCDIR([lib2.c]) - LT_INIT --sysroot=$with_sysroot -+sysroot=$with_libtool_sysroot - AC_SUBST([sysroot]) - AC_OUTPUT(Makefile) - _ATEOF -@@ -49463,7 +49463,7 @@ AM_INIT_AUTOMAKE([foreign]) - AC_PROG_CC - AC_CONFIG_SRCDIR([prog.c]) - LT_INIT --sysroot=$with_sysroot -+sysroot=$with_libtool_sysroot - AC_SUBST([sysroot]) - AC_OUTPUT(Makefile) - _ATEOF -@@ -49821,7 +49821,7 @@ $at_traceon; } - - - LDFLAGS="$LDFLAGS --sysroot=$sysroot -no-undefined" --configure_options="$configure_options --with-sysroot=$sysroot --prefix=$prefix" -+configure_options="$configure_options --with-libtool-sysroot=$sysroot --prefix=$prefix" - - #??? - if test PATH = "$shlibpath_var"; then -@@ -50036,7 +50036,7 @@ AM_INIT_AUTOMAKE([foreign]) - AC_PROG_CC - AC_CONFIG_SRCDIR([lib2.c]) - LT_INIT --sysroot=$with_sysroot -+sysroot=$with_libtool_sysroot - AC_SUBST([sysroot]) - AC_OUTPUT(Makefile) - _ATEOF -@@ -50230,7 +50230,7 @@ AM_INIT_AUTOMAKE([foreign]) - AC_PROG_CC - AC_CONFIG_SRCDIR([prog.c]) - LT_INIT --sysroot=$with_sysroot -+sysroot=$with_libtool_sysroot - AC_SUBST([sysroot]) - AC_OUTPUT(Makefile) - _ATEOF -@@ -50588,7 +50588,7 @@ $at_traceon; } - - - LDFLAGS="$LDFLAGS --sysroot=$sysroot -no-undefined" --configure_options="$configure_options --with-sysroot=$sysroot --prefix=$prefix" -+configure_options="$configure_options --with-libtool-sysroot=$sysroot --prefix=$prefix" - - #??? - if test PATH = "$shlibpath_var"; then -@@ -50803,7 +50803,7 @@ AM_INIT_AUTOMAKE([foreign]) - AC_PROG_CC - AC_CONFIG_SRCDIR([lib2.c]) - LT_INIT --sysroot=$with_sysroot -+sysroot=$with_libtool_sysroot - AC_SUBST([sysroot]) - AC_OUTPUT(Makefile) - _ATEOF -@@ -50997,7 +50997,7 @@ AM_INIT_AUTOMAKE([foreign]) - AC_PROG_CC - AC_CONFIG_SRCDIR([prog.c]) - LT_INIT --sysroot=$with_sysroot -+sysroot=$with_libtool_sysroot - AC_SUBST([sysroot]) - AC_OUTPUT(Makefile) - _ATEOF diff --git a/meta/recipes-devtools/libtool/libtool/0006-libtool.m4-Handle-as-a-sysroot-correctly.patch b/meta/recipes-devtools/libtool/libtool/0006-libtool.m4-Handle-as-a-sysroot-correctly.patch index 435c52c7301..feb1048b554 100644 --- a/meta/recipes-devtools/libtool/libtool/0006-libtool.m4-Handle-as-a-sysroot-correctly.patch +++ b/meta/recipes-devtools/libtool/libtool/0006-libtool.m4-Handle-as-a-sysroot-correctly.patch @@ -17,7 +17,7 @@ Index: libtool-2.5.0/m4/libtool.m4 @@ -1253,18 +1253,18 @@ dnl lt_sysroot will always be passed unq dnl in case the user passed a directory name. lt_sysroot= - case $with_libtool_sysroot in #( + case $with_sysroot in #( - yes) + no) if test yes = "$GCC"; then @@ -29,10 +29,10 @@ Index: libtool-2.5.0/m4/libtool.m4 + yes|''|/) + ;; #( /*) - lt_sysroot=`echo "$with_libtool_sysroot" | $SED -e "$sed_quote_subst"` + lt_sysroot=`echo "$with_sysroot" | $SED -e "$sed_quote_subst"` ;; #( - no|'') - ;; #( *) - AC_MSG_RESULT([$with_libtool_sysroot]) + AC_MSG_RESULT([$with_sysroot]) AC_MSG_ERROR([The sysroot must be an absolute path.])
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> --- meta/classes-global/insane.bbclass | 2 +- meta/classes-recipe/autotools.bbclass | 9 +- .../libtool/libtool-2.5.0.inc | 1 - .../libtool/libtool-native_2.5.0.bb | 2 +- ...e-the-with-sysroot-option-to-avoid-c.patch | 171 ------------------ ...ool.m4-Handle-as-a-sysroot-correctly.patch | 6 +- 6 files changed, 6 insertions(+), 185 deletions(-) delete mode 100644 meta/recipes-devtools/libtool/libtool/0002-libtool.m4-Rename-the-with-sysroot-option-to-avoid-c.patch