[Buildroot] Question about libtool patching

[Sven Neumann <s.neumann@raumfeld.com>]

On Sun, 2009-10-04 at 20:41 +0200, Lionel Landwerlin wrote:
> [PATCH] Prevent libtool patching before autoreconf when autoreconf is required
> 
> Signed-off-by: Lionel Landwerlin <lionel.landwerlin@openwide.fr>
> ---
>  package/Makefile.autotools.in |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/package/Makefile.autotools.in b/package/Makefile.autotools.in
> index c4c4b1c..d36c884 100644
> --- a/package/Makefile.autotools.in
> +++ b/package/Makefile.autotools.in
> @@ -188,7 +188,8 @@ $(BUILD_DIR)/%/.stamp_extracted:
>  $(BUILD_DIR)/%/.stamp_libtool_patch:
>  	$(call MESSAGE,"Patching libtool")
>  # if the package uses libtool, patch it for cross-compiling in buildroot
> -	$(Q)if test "$($(PKG)_LIBTOOL_PATCH)" = "YES"; then \
> +	$(Q)if test "$($(PKG)_LIBTOOL_PATCH)" = "YES" -a \
> +	"$($(PKG)_AUTORECONF)" != "YES"; then \
>  	for i in `find $(@D) -name ltmain.sh`; do \
>  		toolchain/patch-kernel.sh $${i%/*} package buildroot-libtool.patch; \
>  		done \

Totally makes sense to me and we also ran into problems because of this.
I have applied this in our tree and it seems to work fine. Actually, it
finally fixed a long-standing problem :)

Tested-By: Sven Neumann <s.neumann@raumfeld.com>


Sven