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

Please add support for lua 5.4 #17787

Open
2 tasks done
panlinux opened this issue Jan 7, 2025 · 0 comments
Open
2 tasks done

Please add support for lua 5.4 #17787

panlinux opened this issue Jan 7, 2025 · 0 comments
Assignees

Comments

@panlinux
Copy link
Contributor

panlinux commented Jan 7, 2025

Description

Current frr release, and git snapshots, do not support lua 5.4:

checking for a Lua interpreter with version >= 5.3, < 5.4... none
configure: error: Lua 5.3 is required to build with Lua support. No other version is supported.

In Ubuntu, this is preventing us from enabling lua support, because version 5.3 is in the universe component and only 5.4 is in main. Since frr is in main, we can't have it depend on a universe package.
I know this is an Ubuntu problem, but I suppose at some point frr will benefit from newer lua versions. I didn't find a bug report about this, just the old commit from 4y ago that made the check stricter.

Version

10.2.1

How to reproduce

Build frr with lua5.4 installed (and not 5.3)

Expected behavior

I would expect frr to be able to build with a newer version of lua.

Actual behavior

The build fails.

Additional context

No response

Checklist

  • I have searched the open issues for this bug.
  • I have not included sensitive information in this report.
@panlinux panlinux added the triage Needs further investigation label Jan 7, 2025
@ton31337 ton31337 self-assigned this Jan 7, 2025
@ton31337 ton31337 added enhancement lua and removed triage Needs further investigation labels Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants