This fixes two issues with the patch_libtool fixup when used by a host
build.

1) The '@' in $(patch_libtool) is a shell syntax error if patch_libtool
is not the first fixup, so I think should be removed regardless of (2)

2) The conditionals for Hooks/HostConfigure/Pre were causing
patch_libtool to be applied twice for host builds. I don't see why the
logic for patch_libtool should be different than the other fixups. Note
that HOST_FIXUP is assigned ?= $(PKG_FIXUP) earlier in the makefile.

Signed-off-by: Andy Leiserson <a...@leiserson.org>
---
Index: include/autotools.mk
===================================================================
--- include/autotools.mk        (revision 36687)
+++ include/autotools.mk        (working copy)
@@ -51,7 +51,7 @@
# 1: build dir
 define patch_libtool
-       @(cd $(1); \
+       (cd $(1); \
                for lt in $$$$($$(STAGING_DIR_HOST)/bin/find . -name 
ltmain.sh); do \
                        lt_version="$$$$($$(STAGING_DIR_HOST)/bin/sed -ne 
's,^[[:space:]]*VERSION="\?\([0-9]\.[0-9]\+\).*,\1,p' $$$$lt)"; \
                        case "$$$$lt_version" in \
@@ -139,14 +139,10 @@
     $(HOST_BUILD_DIR)))
 endef
-ifneq ($(filter patch-libtool,$(PKG_FIXUP)),)
+ifneq ($(filter patch-libtool,$(HOST_FIXUP)),)
   Hooks/HostConfigure/Pre += patch_libtool_host
 endif
-ifneq ($(filter patch-libtool,$(HOST_FIXUP)),)
-  Hooks/HostConfigure/Pre += $(strip $(call patch_libtool,$(HOST_BUILD_DIR)))
-endif
-
 ifneq ($(filter libtool,$(HOST_FIXUP)),)
  ifeq ($(filter no-autoreconf,$(HOST_FIXUP)),)
   Hooks/HostConfigure/Pre += autoreconf_host
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to