Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DX-415] Catch up to upstream for bundled lua 5.1 #3

Merged
merged 54 commits into from
Feb 29, 2024

Conversation

zapzap
Copy link

@zapzap zapzap commented Feb 29, 2024

Lua 5.1 is now bundled in lupa.

Le0Developer and others added 30 commits April 3, 2023 14:47
The 5.1.x branch was deleted from the github repo and the 5.1.5 tag is missing, so this adds the sources as distributed by the official Lua download server (where the archives use a different directory layout).

Code downloaded from https://www.lua.org/ftp

Also, disable the `test_decrease_memory` in Lua 5.1, which lacks the required infrastructure for limiting the memory usage.
…when no Lua is found.

Original patch by Richard Connon.

Closes #237
setuptools doesn't have a `build_requires` keyword. By using the
`setup_requires` keyword, `pip install --use-pep517` will be able to
install Cython before building the package.
scoder and others added 24 commits October 4, 2023 09:19
… attribute in order to control how the import is done.
A new context manager `allow_lua_module_loading()` is added.

Enabling module loading by default doesn't work when there are multiple
Lua specific Lupa modules because the symbols collide with each other
when loaded with `RTLD_GLOBAL`.

Therefore, we disable the dlopenflags by default and allow users to
conditionally enable them as part of the import.
…to allow finer control of allowed input and debug output. (#252)

These match the same arguments of the Lua `load()` function.

Closes #248
We do not currently translate them transitively, but we might do that in the future.
@rsiemens rsiemens merged commit d8b1bd0 into zapier:master Feb 29, 2024
87 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants