Modifies `build/geanylua.m4` to use LuaJIT when available. Otherwise, falls back on Lua 5.1.
Resolves #1228. Possibly #1133. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany-plugins/pull/1233
-- Commit Summary --
* GeanyLua: Use LuaJIT when available
-- File Changes --
M build/geanylua.m4 (16)
-- Patch Links --
https://github.com/geany/geany-plugins/pull/1233.patch https://github.com/geany/geany-plugins/pull/1233.diff
@xiota pushed 1 commit.
96d39df306b9d03c027cbddcc0d258312265382c GeanyLua: Give priority to specified Lua package
IMO the default should stay lua5.1 for one release to allow packagers to go through the process of changing dependencies. For example I think Debian requires an approval process to change them. Also "somebody" needs to integrate it into the Windows bundle.
We can document that it will change in the release after and as soon as the next release happens we can push the change of default. Of course distros that want to change immediately can just set the CL option to luajit.
@xiota pushed 1 commit.
5df759949d148cd80b90ccf7b79ace5a7312536f GeanyLua: Build with Lua 5.1 by default, fallback to LuaJIT
... the default should stay lua5.1...
Done.
We can document that it will change in the release after...
Is that something I should change/add to this PR? The `NEWS` file?
Is that something I should change/add to this PR? The NEWS file?
Yeah, its not commonly done, but putting a note in the 1.39 section of the `NEWS` so its remembered is a good idea.
@xiota pushed 1 commit.
c809d427536fde732937fdb8bbe3967fa5dd8ba9 GeanyLua: Add note about build default to NEWS file:
Closed #1233.
github-comments@lists.geany.org