[3.0.x] Lua: Make scripts compatible with Lua 5.4
Makes the scripts compatible with Lua 5.4, while at it, backport some minor changes to stay closer to master and fix minor issues, making future backports easier too.
Fixes #25036 (closed) for VLC 3.0
Merge request reports
Activity
assigned to @ePirat
changed milestone to %3.0.x maintenance
added Component::LUA scripts label
added MRStatus::Reviewable label
- Resolved by Marvin Scholz
What's really useful in 3.0 is removing all
module()
declarations from modules. We can also clean up allrequire
calls and other things in the lua scripts, for various reasons including consistency; either way we should probably backport fb808e34 too.Are we going to update lua contribs on 3.0?
added MRStatus::Acceptable label and removed MRStatus::Reviewable label
added MRStatus::Accepted label and removed MRStatus::Acceptable label
MR Acceptance result
This MergeRequest has been Accepted! Congratulations.MR acceptance checks details:
-
MR should be considered mergeable by Gitlab -
Last pipeline should be successful -
MergeRequest should have at least one external review and/or vote -
All threads should be resolved, and score >= 0 -
MergeRequest should have no activity (threads/votes) for (24h/24h)
-
added 14 commits
-
f6426b13...01dd8180 - 5 commits from branch
videolan:3.0.x
-
0e909a13 - share: lua: remove
Id
magic comments - e5ba5793 - lua: common: do not use deprecated module function
- 3ebac797 - lua: sandbox: do not use deprecated module function
- 251ef7ee - lua: simplexml: do not use deprecated module function
- b922d86f - lua: httprequests: do not use deprecated module function
- 182e7f9e - lua: host: do not use deprecated module function
- 3b25a57d - lua: http: custom: remove deprecated module function
- 99c124c5 - lua: intf: luac: remove unused requires
- cc4c37eb - lua: intf: luac: fix usage instructions
Toggle commit list-
f6426b13...01dd8180 - 5 commits from branch
enabled an automatic merge when the pipeline for cc4c37eb succeeds
changed milestone to %3.0.18