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

cdialect option does not support c23 #2410

Open
stu opened this issue Jan 11, 2025 · 2 comments · May be fixed by #2414
Open

cdialect option does not support c23 #2410

stu opened this issue Jan 11, 2025 · 2 comments · May be fixed by #2414

Comments

@stu
Copy link

stu commented Jan 11, 2025

What seems to be the problem?
setting

cdialect "c23"

fails with

Error: invalid value 'c23' for cdialect

What did you expect to happen?
C23 dialect.... to be supported.

What have you tried so far?
reverting back to C17

How can we reproduce this?
add cdialect "c23" to your premake5.lua

What version of Premake are you using?
commit baa0e37, from january 11 2025, built on linux with clang-20

@stu stu added the bug label Jan 11, 2025
@nickclark2016
Copy link
Member

C23 is not supported yet. Should be trivial to add. Will investigate adding it this weekend.

@stu
Copy link
Author

stu commented Jan 11, 2025

Thanks @nickclark2016, I looked at the commits for #2277 and realised it's not as simple as I thought for writing a patch myself.

@nickclark2016 nickclark2016 linked a pull request Jan 11, 2025 that will close this issue
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants