[Buildroot] [PATCH] New package: LuaExpat

[François Perrad]

2012/7/14 Danomi Manchego <danomimanchego123 at gmail.com>:
> François,
>
>> but Config.in contains an useless line :
>>    depends on BR2_PACKAGE_LUA_SHARED_LIBRARY
>
> To be honest, I was just following the example of most of the other
> lua-related packages.  Currently, the same depends-on is listed in
> luafilesystem, luasocket, cgilua, rings, xavante, and copas (with comment
> indicating that this is due to luasocket).  How is usefulness determined in
> these cases?
>

Well, Lua module packages need some clean up, see
http://patchwork.ozlabs.org/patch/162294/

The dependency of a Lua module (like LuaExpat) against Lua is
implicit, because in package/Config.in
    source "package/lua/Config.in"
    if BR2_PACKAGE_LUA
    menu "LUA libraries/modules"
    ...
    source "package/luaexpat/Config.in"
    ...
    endmenu
    endif
So, don't add explicit things in package/luaexpat/Config.in like :
    depends on BR2_PACKAGE_LUA_SHARED_LIBRARY
or
    depends on BR2_PACKAGE_LUA

By this way, your module could work with Lua or LuaJIT.
(BR2_PACKAGE_LUA_SHARED_LIBRARY is an internal of Lua package)

François

> Danomi -
>
>