[source] include/package.mk: remove old configured stamps before attempting configuration

LEDE Commits lede-commits at lists.infradead.org
Thu Dec 28 03:29:26 PST 2017


neoraider pushed a commit to source.git, branch master:
https://git.lede-project.org/307b29032fc357161505239cad794a99d07b69f4

commit 307b29032fc357161505239cad794a99d07b69f4
Author: Matthias Schiffer <mschiffer at universe-factory.net>
AuthorDate: Sun Dec 10 15:30:30 2017 +0100

    include/package.mk: remove old configured stamps before attempting configuration
    
    Some packages, e.g. busybox, explicitly remove old .configured stamps
    before attempting configuration, rather than after the actual configuration
    step. This seems like a good idea, as there will be no stamp left if
    configuration fails. Change generic rules to work like this, so package-
    specific rules can be dropped.
    
    Signed-off-by: Matthias Schiffer <mschiffer at universe-factory.net>
---
 include/package.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/package.mk b/include/package.mk
index e804b02..78ea5d0 100644
--- a/include/package.mk
+++ b/include/package.mk
@@ -168,11 +168,11 @@ define Build/CoreTargets
 
   $(call Build/Exports,$(STAMP_CONFIGURED))
   $(STAMP_CONFIGURED): $(STAMP_PREPARED) $(STAMP_CONFIGURED_DEPENDS)
+	rm -f $(STAMP_CONFIGURED_WILDCARD)
 	$(CleanStaging)
 	$(foreach hook,$(Hooks/Configure/Pre),$(call $(hook))$(sep))
 	$(Build/Configure)
 	$(foreach hook,$(Hooks/Configure/Post),$(call $(hook))$(sep))
-	rm -f $(STAMP_CONFIGURED_WILDCARD)
 	touch $$@
 
   $(call Build/Exports,$(STAMP_BUILT))



More information about the lede-commits mailing list