<div dir="ltr"><div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">Le ven. 16 nov. 2018 à 14:53, Thomas Petazzoni <<a href="mailto:thomas.petazzoni@bootlin.com">thomas.petazzoni@bootlin.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">The host luarocks tool is needed when *extracting* luarocks<br>
packages. Since regular dependencies are only available at the<br>
configure step, it caused the "luarocks" program to be available too<br>
late to be used to extract luarocks packages.<br>
<br>
To work around this, host-luarocks was manually added as a dependency<br>
of the extract step of luarocks packages. While this worked fine until<br>
now, it breaks badly with per-package folders, because host-luarocks<br>
is not recorded as an extract dependency of luarocks packages, and<br>
therefore is not copied to the per-package host folder, causing a<br>
failure.<br>
<br>
In the mean time, the <pkg>_EXTRACT_DEPENDENCIES mechanism was added,<br>
which aims at fixing exactly this problem, so let's use it and<br>
simplify <a href="http://pkg-luarocks.mk" rel="noreferrer" target="_blank">pkg-luarocks.mk</a>.<br>
<br>
Cc: Francois Perrad <<a href="mailto:francois.perrad@gadz.org" target="_blank">francois.perrad@gadz.org</a>><br></blockquote><div><br></div><div>Acked-by: Francois Perrad <<a href="mailto:francois.perrad@gadz.org">francois.perrad@gadz.org</a>><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Signed-off-by: Thomas Petazzoni <<a href="mailto:thomas.petazzoni@bootlin.com" target="_blank">thomas.petazzoni@bootlin.com</a>><br>
---<br>
 package/<a href="http://pkg-luarocks.mk" rel="noreferrer" target="_blank">pkg-luarocks.mk</a> | 7 ++-----<br>
 1 file changed, 2 insertions(+), 5 deletions(-)<br>
<br>
diff --git a/package/<a href="http://pkg-luarocks.mk" rel="noreferrer" target="_blank">pkg-luarocks.mk</a> b/package/<a href="http://pkg-luarocks.mk" rel="noreferrer" target="_blank">pkg-luarocks.mk</a><br>
index e2b48cfb4c..29c5e66cd1 100644<br>
--- a/package/<a href="http://pkg-luarocks.mk" rel="noreferrer" target="_blank">pkg-luarocks.mk</a><br>
+++ b/package/<a href="http://pkg-luarocks.mk" rel="noreferrer" target="_blank">pkg-luarocks.mk</a><br>
@@ -42,7 +42,8 @@ $(2)_SITE             ?= $$(call qstrip,$$(BR2_LUAROCKS_MIRROR))<br>
<br>
 # Since we do not support host-luarocks-package, we know this is<br>
 # a target package, and can just add the required dependencies<br>
-$(2)_DEPENDENCIES      += host-luarocks luainterpreter<br>
+$(2)_DEPENDENCIES      += luainterpreter<br>
+$(2)_EXTRACT_DEPENDENCIES += host-luarocks<br>
<br>
 #<br>
 # Extract step. Extract into a temporary dir and move the relevant part to the<br>
@@ -71,10 +72,6 @@ endif<br>
 # make targets<br>
 $(call inner-generic-package,$(1),$(2),$(3),$(4))<br>
<br>
-# $(2)_DEPENDENCIES are handled for configure step (too late)<br>
-# but host-luarocks is required to do the extract<br>
-$$($(2)_TARGET_EXTRACT): | host-luarocks<br>
-<br>
 endef<br>
<br>
 ################################################################################<br>
-- <br>
2.19.1<br>
<br>
</blockquote></div></div></div>