[Buildroot] [PATCH 2/3] lutok: new package

[Arnout Vandecappelle]

On 06/03/15 20:47, Sebastien Bourdelin wrote:
> Lutok is a lightweight C++ API library for Lua.
> 
> https://github.com/jmmv/lutok
> 
> Signed-off-by: Sebastien Bourdelin <sebastien.bourdelin at savoirfairelinux.com>
> ---
>  package/Config.in       |  1 +
>  package/lutok/Config.in | 11 +++++++++++
>  package/lutok/lutok.mk  | 19 +++++++++++++++++++
>  3 files changed, 31 insertions(+)
>  create mode 100644 package/lutok/Config.in
>  create mode 100644 package/lutok/lutok.mk
> 
> diff --git a/package/Config.in b/package/Config.in
> index 78ed223..628f36c 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -457,6 +457,7 @@ endif
>  	source "package/lua/Config.in"
>  	source "package/luainterpreter/Config.in"
>  	source "package/luajit/Config.in"
> +	source "package/lutok/Config.in"
>  if BR2_PACKAGE_HAS_LUAINTERPRETER && !BR2_STATIC_LIBS
>  # lua modules are dynamically loaded, so not available on static builds
>  menu "Lua libraries/modules"
> diff --git a/package/lutok/Config.in b/package/lutok/Config.in
> new file mode 100644
> index 0000000..b94c093
> --- /dev/null
> +++ b/package/lutok/Config.in
> @@ -0,0 +1,11 @@
> +config BR2_PACKAGE_LUTOK
> +	bool "lutok"
> +	depends on BR2_INSTALL_LIBSTDCPP
> +	depends on BR2_PACKAGE_HAS_LUAINTERPRETER

 Did you check if it works with luajit as well?

 Have you checked if it works with STATIC?

> +	help
> +	  Lutok is a lightweight C++ API library for Lua.
> +
> +	  https://github.com/jmmv/lutok
> +
> +comment "lutok needs a toolchain w/ C++"
> +	depends on !BR2_INSTALL_LIBSTDCPP

 This should also depend on BR2_PACKAGE_HAS_LUAINTERPRETER

> diff --git a/package/lutok/lutok.mk b/package/lutok/lutok.mk
> new file mode 100644
> index 0000000..25f4070
> --- /dev/null
> +++ b/package/lutok/lutok.mk
> @@ -0,0 +1,19 @@
> +################################################################################
> +#
> +# lutok
> +#
> +################################################################################
> +
> +LUTOK_VERSION = 740c34180384520086302f5e107ae0ae37a7e617

 This is the uploaded 0.4 release, so you can use

https://github.com/jmmv/lutok/releases/download/lutok-0.4/lutok-0.4.tar.gz

instead, avoiding AUTORECONF, and making it possible to add a hash file.

> +LUTOK_SITE = git://github.com/jmmv/lutok
> +LUTOK_AUTORECONF = YES
> +LUTOK_INSTALL_STAGING = YES
> +LUTOK_DEPENDENCIES = host-pkgconf lua

 It seems to have an optional dependency on atf as well, so you need something like

ifeq ($(BR2_PACKAGE_ATF),y)
LUTOK_DEPENDENCIES += atf
LUTOK_CONF_OPTS += --with-atf
else
LUTOK_CONF_OPTS += --without-atf
endif

 Also it tries to use doxygen if available, we don't care about documentation so add
LUTOK_CONF_OPTS = --without-doxygen

> +LUTOK_LICENSE = BSD

 BSD-3c

> +LUTOK_LICENSE_FILES = COPYING
> +
> +ifneq ($(BR2_STATIC_LIBS),y)
> +LUTOK_MAKE_OPTS = LIBS=-ldl

 Is this really needed? The configure script should detect that automatically...


 Regards,
 Arnout

> +endif
> +
> +$(eval $(autotools-package))
> 


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F