commit: 370d960c1d6725cf89e35ea7e542405e532d6059 Author: Marek Szuba <marecki <AT> gentoo <DOT> org> AuthorDate: Sat Nov 28 22:14:40 2020 +0000 Commit: Marek Szuba <marecki <AT> gentoo <DOT> org> CommitDate: Sat Nov 28 22:28:24 2020 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=370d960c
dev-lua/mpack: migrate to lua.eclass Had to suppress tests for the luajit target (one of the tests checks for memory leaks and LuaJIT does leak memory) as well as prevent the module from being unnecessarily linked against liblua (upstream have already fixed this, unfortunately a bump to a recent version would not be trivial). Otherwise, simple enough. Closes: https://bugs.gentoo.org/752906 Signed-off-by: Marek Szuba <marecki <AT> gentoo.org> dev-lua/mpack/mpack-1.0.4-r100.ebuild | 86 +++++++++++++++++++++++++++++++++++ profiles/package.mask | 1 + 2 files changed, 87 insertions(+) diff --git a/dev-lua/mpack/mpack-1.0.4-r100.ebuild b/dev-lua/mpack/mpack-1.0.4-r100.ebuild new file mode 100644 index 00000000000..7aca1045a87 --- /dev/null +++ b/dev-lua/mpack/mpack-1.0.4-r100.ebuild @@ -0,0 +1,86 @@ +# Copyright 1999-2020 Gentoo Authors +# Distributed under the terms of the GNU General Public License v2 + +EAPI=7 + +LUA_COMPAT=( lua5-{1..3} luajit ) + +inherit lua toolchain-funcs + +DESCRIPTION="Lua bindings for libmpack" +HOMEPAGE="https://github.com/libmpack/libmpack/"; +SRC_URI="https://github.com/libmpack/libmpack/archive/${PV}.tar.gz -> ${P}.tar.gz" +S="${WORKDIR}/libmpack-${PV}/binding/lua" + +LICENSE="MIT" +SLOT="0" +KEYWORDS="~amd64 ~arm ~arm64 ~x86" +IUSE="test" +RESTRICT="!test? ( test )" + +RDEPEND="${LUA_DEPS}" +DEPEND="${RDEPEND}" +BDEPEND="virtual/pkgconfig + test? ( + ${RDEPEND} + dev-lua/busted[${LUA_USEDEP}] + )" + +lua_src_compile() { + # Make sure we have got no leftovers from other implementations. + # There is no 'clean' target in the makefile, though. + rm -f ${PN}.so + + # We set LUA_LIB to an empty string while building version 1.0.4 because + # compiled Lua modules must not link against liblua. This has already been + # fixed upstream. + emake \ + CC="$(tc-getCC)" \ + USE_SYSTEM_LUA=yes \ + LUA_INCLUDE="$(lua_get_CFLAGS)" \ + LUA_LIB="" + + # Tag the result with current implementation and move it out of the way. + mv ${PN}.so ${PN}-${ELUA}.so || die +} + +src_compile() { + lua_foreach_impl lua_src_compile +} + +lua_src_test() { + # "[ FAILED ] test.lua @ 279: mpack should not leak memory" + # It doesn't seem upstream actually support LuaJIT so were this up to me + # I would drop it from LUA_COMPAT, unfortunately there are packages in the + # tree which currently expect it to be supported. + if [[ ${ELUA} == "luajit" ]]; then + ewarn "Not running tests under ${ELUA} because they are known to fail" + return + fi + + # The test suite must be able to find the module under its original name. + rm -f ${PN}.so + ln -s ${PN}-${ELUA}.so ${PN}.so || die + busted --lua="${ELUA}" -o gtest test.lua || die +} + +src_test() { + lua_foreach_impl lua_src_test +} + +lua_src_install() { + # This time we move the correct library file back in order not to risk + # confusing make with symlinks. + mv ${PN}-${ELUA}.so ${PN}.so || die + + emake \ + DESTDIR="${ED}" \ + USE_SYSTEM_LUA=yes \ + LUA_CMOD_INSTALLDIR="$(lua_get_cmod_dir)" \ + install +} + +src_install() { + lua_foreach_impl lua_src_install + einstalldocs +} diff --git a/profiles/package.mask b/profiles/package.mask index 9946a628585..ef27778a2a2 100644 --- a/profiles/package.mask +++ b/profiles/package.mask @@ -527,6 +527,7 @@ dev-lua/luacrypto >=dev-lua/luv-1.32.0.0-r100 >=dev-lua/mediator_lua-1.1.2_p0-r100 >=dev-lua/messagepack-0.3.2-r100 +>=dev-lua/mpack-1.0.4-r100 >=dev-lua/penlight-1.9.2-r100 >=dev-lua/say-1.3_p1-r100 >=dev-lua/toluapp-1.0.93_p20190513-r100
