All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/turbolua: Fix optional openssl dependency
@ 2016-05-13 10:00 Marcin Niestroj
  2016-05-13 20:19 ` Thomas Petazzoni
  0 siblings, 1 reply; 2+ messages in thread
From: Marcin Niestroj @ 2016-05-13 10:00 UTC (permalink / raw)
  To: buildroot

turbolua fails to build when we have openssl selected, because it didn't
mention openssl in the dependency list.

Fix build failure by specifying openssl in dependency list when it is
selected.

Signed-off-by: Marcin Niestroj <m.niestroj@grinn-global.com>
---
 package/turbolua/turbolua.mk | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/package/turbolua/turbolua.mk b/package/turbolua/turbolua.mk
index cf66e70..fe6db31 100644
--- a/package/turbolua/turbolua.mk
+++ b/package/turbolua/turbolua.mk
@@ -12,9 +12,15 @@ TURBOLUA_LICENSE_FILES = LICENSE
 
 TURBOLUA_MAKE_OPTS = \
 	$(TARGET_CONFIGURE_OPTS) \
-	SSL=$(if $(BR2_PACKAGE_OPENSSL),openssl,none) \
 	LUAJIT_VERSION="$(LUAJIT_VERSION)"
 
+ifeq ($(BR2_PACKAGE_OPENSSL),y)
+TURBOLUA_MAKE_OPTS += SSL=openssl
+TURBOLUA_DEPENDENCIES += openssl
+else
+TURBOLUA_MAKE_OPTS += SSL=none
+endif
+
 define TURBOLUA_BUILD_CMDS
 	$(MAKE) $(TURBOLUA_MAKE_OPTS) -C $(@D) all
 endef
-- 
2.8.2

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

* [Buildroot] [PATCH] package/turbolua: Fix optional openssl dependency
  2016-05-13 10:00 [Buildroot] [PATCH] package/turbolua: Fix optional openssl dependency Marcin Niestroj
@ 2016-05-13 20:19 ` Thomas Petazzoni
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Petazzoni @ 2016-05-13 20:19 UTC (permalink / raw)
  To: buildroot

Hello,

On Fri, 13 May 2016 12:00:24 +0200, Marcin Niestroj wrote:
> turbolua fails to build when we have openssl selected, because it didn't
> mention openssl in the dependency list.
> 
> Fix build failure by specifying openssl in dependency list when it is
> selected.
> 
> Signed-off-by: Marcin Niestroj <m.niestroj@grinn-global.com>
> ---
>  package/turbolua/turbolua.mk | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)

Applied to master, thanks.

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-05-13 20:19 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-13 10:00 [Buildroot] [PATCH] package/turbolua: Fix optional openssl dependency Marcin Niestroj
2016-05-13 20:19 ` Thomas Petazzoni

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.