waf: fix autobuilders failure
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/
Signed-off-by Maxime Hadjinlian <[email protected]>
--- package/waf/waf.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
package/waf/waf.mk +1 -1
@@ -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
Thomas Petazzoni
AppliedPATCH 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/Signed-off-by Maxime Hadjinlian <[email protected]>--- 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