Hi Roberto, On Fri, Feb 24, 2012 at 10:40:38PM -0500, Roberto C. Sánchez wrote: > On Sat, Feb 18, 2012 at 01:36:54PM -0500, Peter Colberg wrote: > > > > Could you please consider the Lua 5.2 patches [1] for inclusion? > > > > Lua 5.2: replace lua_equal and lua_lessthan with lua_compare > > Lua 5.2: replace occurrences of deprecated LUA_GLOBALSINDEX > > Lua 5.2: replace lua_strlen with lua_rawlen > > Lua 5.2: replace lua_[gs]etfenv with lua_[gs]etuservalue > > Lua 5.2: pass extra parameter NULL to lua_resume > > Lua 5.2: replace lua_open with luaL_newstate > > Lua 5.2: fix test_free_functions unit test > > Lua 5.2: use new macro lua_pushglobaltable > > > > [1] http://git.colberg.org/luabind.git > > > > I am going to contact upstream about incorporating this change since it > is probably too much to carry this as a Debian-only patch.
It would be great if you managed to contact the upstream maintainer. In fact I was thinking of having the Lua 5.2 patch series included upstream first, but so far there was no reaction from the maintainer on the Luabind list. The same goes for the cast graph patch in #660380. The patches do not change any functionality of Luabind, but merely add Lua 5.2 compatibility. So even if upstream does not react now, it would nevertheless be good to have them in Debian, since we have lua5.2 in Debian. There was recently a user discussion about the future of Luabind [1]. Unfortunately noone wants to step up to become the maintainer of Luabind. [1] http://article.gmane.org/gmane.comp.lang.lua.bind.user/3002 Maybe the GNU/Linux distributions that ship Luabind could work together on a maintainance-focused project? Apart from the issues brought up in this bug and #660380, Luabind is in good shape, and still the most powerful C++ Lua binding library out there, so it would be sad to see it bitrotting over time. > If you could rebase against upstream commit 87898f6012, that would > help for both this and the cast graph patch in #660380. Please let > me know. I based the patch series on the prior commit 3044a90 intentionally, since I was not sure about the stability of the latest commit 87898f6 adding initial C++11 support (commit message says “Work in progress”). But if you intend to base the Debian package on the latest commit, or, even better, if upstream includes the Lua 5.2 patch series in the repository, I will of course rebase the patches. Let's see what the upstream maintainer says. Regards, Peter -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
