Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] moarvm: remove host-luajit dependency
@ 2016-03-23 17:56 Francois Perrad
  2016-03-23 20:34 ` Thomas Petazzoni
  0 siblings, 1 reply; 2+ messages in thread
From: Francois Perrad @ 2016-03-23 17:56 UTC (permalink / raw)
  To: buildroot

now, use the minilua which is embedded in the package.
that fixes the build issue, see http://autobuild.buildroot.net/results/4dc/4dcbcde0046e265317e170a977e699e5c3a06b87/

Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
 package/moarvm/moarvm.mk | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/package/moarvm/moarvm.mk b/package/moarvm/moarvm.mk
index 7456902..b89060b 100644
--- a/package/moarvm/moarvm.mk
+++ b/package/moarvm/moarvm.mk
@@ -10,7 +10,7 @@ MOARVM_SOURCE = MoarVM-$(MOARVM_VERSION).tar.gz
 MOARVM_LICENSE = Artistic-2.0
 MOARVM_LICENSE_FILES = Artistic2.txt
 MOARVM_INSTALL_STAGING = YES
-MOARVM_DEPENDENCIES = host-luajit libuv libatomic_ops
+MOARVM_DEPENDENCIES = libuv libatomic_ops
 
 MOARVM_CONF_OPTS = \
 	--build=$(GNU_HOST_NAME) \
@@ -19,7 +19,6 @@ MOARVM_CONF_OPTS = \
 	--cc="$(TARGET_CC)" \
 	--ld="$(TARGET_CC)" \
 	--prefix="/usr" \
-	--lua=$(HOST_DIR)/usr/bin/luajit \
 	--has-libuv \
 	--has-libatomic
 
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [Buildroot] [PATCH] moarvm: remove host-luajit dependency
  2016-03-23 17:56 [Buildroot] [PATCH] moarvm: remove host-luajit dependency Francois Perrad
@ 2016-03-23 20:34 ` Thomas Petazzoni
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Petazzoni @ 2016-03-23 20:34 UTC (permalink / raw)
  To: buildroot

Hello,

On Wed, 23 Mar 2016 18:56:44 +0100, Francois Perrad wrote:
> now, use the minilua which is embedded in the package.
> that fixes the build issue, see http://autobuild.buildroot.net/results/4dc/4dcbcde0046e265317e170a977e699e5c3a06b87/
> 
> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
> ---
>  package/moarvm/moarvm.mk | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

If moarvm is capable of using an external Lua interpreter rather than
its own embedded copy, then it should use the external Lua interpreter.

The fact that there is a build issue means that it must be solved, not
worked around.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-03-23 20:34 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-03-23 17:56 [Buildroot] [PATCH] moarvm: remove host-luajit dependency Francois Perrad
2016-03-23 20:34 ` Thomas Petazzoni

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox