Skip to contentSkip to conversation

waf: fix autobuilders failure

PATCH New Maxime Hadjinlian 2 participants 2 messages Apr 2, 2018

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 waf: fix autobuilders failure

The package is host only and so the variable to use must be prefixed by 'HOST_'

Fixes:

	http://autobuild.buildroot.org/results/01d/01d8c21df4baa9ecc6a0e45744fe6a4d086a0652/
---
 package/waf/waf.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
package/waf/waf.mk +1 -1
index 639c26a3b0..cb738a38fd 100644
@@ -9,7 +9,7 @@ WAF_SOURCE = waf-$(WAF_VERSION)
 WAF_SITE = https://waf.io/
 
 define HOST_WAF_EXTRACT_CMDS
-	$(INSTALL) -D -m 0755 $(WAF_DL_DIR)/waf-$(WAF_VERSION) $(@D)/waf
+	$(INSTALL) -D -m 0755 $(HOST_WAF_DL_DIR)/waf-$(WAF_VERSION) $(@D)/waf
 endef
 
 define HOST_WAF_INSTALL_CMDS
Signature · 1 line
2.16.2
#1 lore raw

PATCH 1/1 waf: fix autobuilders failure

Hello,

Quoted 10 lines · Maxime Hadjinlian

The package is host only and so the variable to use must be prefixed by 'HOST_'

Fixes:

	http://autobuild.buildroot.org/results/01d/01d8c21df4baa9ecc6a0e45744fe6a4d086a0652/
---
 package/waf/waf.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Applied to master after adjusting the commit title and commit log. Thanks!

However, I really think packages shouldn't have to use HOST_<pkg>_DL_DIR, and therefore the infra should probably define <pkg>_DL_DIR.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com