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 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.