On Mon, Jun 18, 2012 at 10:24:16AM -0300, Tomas Guisasola Gorham wrote: > I noted that incompatibility but no one has mentioned any > problem until now. I thought that, in fact, the correct way to > use LuaSQL was not to assume the global luasql, but I think the > example does not follow that principle. Anyway, I think the > actual version being compatible with both Lua 5.1 and 5.2 pays > off the price of the mentioned incompatibility. But do you > think this incompatibility deserves a special conditional > compilation that could just assign to the global variable to > guarantee backwards compatibility? I think this is easy to do!
No, I think it does not deserve an #ifdef. For the Debian side, this bugreport documents the problem and provides the fix. I will also add a note inside the package NEWS file time permitting. I think you should just fix the examples, and maybe add a note in the changelog/readme, so that if one hits the problem and looks at the project homepage he finds the fix as well. Cheers -- Enrico Tassi -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
