From: Peter Korsgaard <jacmet@uclibc.org>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] libtool: revert a commit which re-introduced an already fixed problem
Date: Mon, 17 Mar 2014 23:08:34 +0100 [thread overview]
Message-ID: <87txaw4gwt.fsf@dell.be.48ers.dk> (raw)
In-Reply-To: <1395078022-37381-1-git-send-email-Vincent.Riera@imgtec.com> (Vicente Olivert Riera's message of "Mon, 17 Mar 2014 17:40:22 +0000")
>>>>> "Vicente" == Vicente Olivert Riera <Vincent.Riera@imgtec.com> writes:
> The commit ed73d1d2e3703290e74bb076bc6dd0417aa3ba21 modified the changes
> made by the commit 4268d3967e2d691c151d6b5629e4051deb077b9a. The problem
> that was fixed by the former commit is present again due to those
> modifications. This patch reverts those modifications to have that
> problem fixed again.
> Fixes:
> http://autobuild.buildroot.net/results/b86/b86a83c6549004f226e7255242e54ef4e50c5ec3/
So this basically just reverts Arnouts fix for old automake versions? Do
we understand why it doesn't work? Can we come up with a solution that
both fixes the n64 issue and old automake?
> Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
> ---
> ...patch => libtool-0001-mips64-n64-linking.patch} | 0
> package/libtool/libtool.mk | 13 -------------
> 2 files changed, 0 insertions(+), 13 deletions(-)
> rename package/libtool/{libtool-mips64-n64-linking.post-install-patch => libtool-0001-mips64-n64-linking.patch} (100%)
> diff --git a/package/libtool/libtool-mips64-n64-linking.post-install-patch b/package/libtool/libtool-0001-mips64-n64-linking.patch
> similarity index 100%
> rename from package/libtool/libtool-mips64-n64-linking.post-install-patch
> rename to package/libtool/libtool-0001-mips64-n64-linking.patch
> diff --git a/package/libtool/libtool.mk b/package/libtool/libtool.mk
> index 603f1f1..2f6ea7c 100644
> --- a/package/libtool/libtool.mk
> +++ b/package/libtool/libtool.mk
> @@ -13,19 +13,6 @@ LIBTOOL_LICENSE_FILES = COPYING
> HOST_LIBTOOL_LIBTOOL_PATCH = NO
> -# libtool-mips64-n64-linking.post-install-patch is an upstream patch that
> -# fixes MIPS64 n64 link failures. However, because the patch touches an m4
> -# file, applying it triggers a run of autoconf, automake, etc. This sometimes
> -# leads to build failures due to incompatible system autotools. We cannot
> -# simply set HOST_LIBTOOL_AUTORECONF = YES because that would create a
> -# circular dependency on host-libtool. Therefore, just apply the patch
> -# directly on the installed file.
> -define HOST_LIBTOOL_FIXUP_LIBTOOL_M4
> - patch $(HOST_DIR)/usr/share/aclocal/libtool.m4 < \
> - package/libtool/libtool-mips64-n64-linking.post-install-patch
> -endef
> -HOST_LIBTOOL_POST_INSTALL_HOOKS += HOST_LIBTOOL_FIXUP_LIBTOOL_M4
> -
> $(eval $(autotools-package))
> $(eval $(host-autotools-package))
> --
> 1.7.1
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
Bye, Peter Korsgaard
next prev parent reply other threads:[~2014-03-17 22:08 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-17 17:40 [Buildroot] [PATCH] libtool: revert a commit which re-introduced an already fixed problem Vicente Olivert Riera
2014-03-17 22:08 ` Peter Korsgaard [this message]
2014-03-18 7:46 ` Arnout Vandecappelle
2014-03-18 10:44 ` Vicente Olivert Riera
2014-03-18 10:55 ` Vicente Olivert Riera
2014-03-19 7:54 ` Arnout Vandecappelle
2014-03-19 9:40 ` Vicente Olivert Riera
2014-03-19 13:50 ` Arnout Vandecappelle
2014-03-19 14:41 ` Vicente Olivert Riera
2014-03-19 15:02 ` Arnout Vandecappelle
2014-03-19 15:07 ` Vicente Olivert Riera
2014-03-19 16:23 ` Arnout Vandecappelle
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87txaw4gwt.fsf@dell.be.48ers.dk \
--to=jacmet@uclibc.org \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox