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

Change Makefile LUAV to "lua5.2"? #19

Open
Grissess opened this issue Aug 29, 2014 · 4 comments
Open

Change Makefile LUAV to "lua5.2"? #19

Grissess opened this issue Aug 29, 2014 · 4 comments

Comments

@Grissess
Copy link
Contributor

It's been a little annoying to have to specify that string manually on each make; is there a platform where leaving it as its default "lua" works? If so, would that platform also support specifying "lua5.2" (and have those libraries available) instead?

@sjrct
Copy link
Contributor

sjrct commented Aug 29, 2014

It does work that way on archlinux, and in addition specifying "lua5.2" is not supported. The purpose of including it was because it is called different things on different architectures. I honestly don't think it makes to much of a difference whether it is "lua5.2" or just "lua", it just changes who has to modify the arguments. Perhaps ideally we should detect which version is present?

@Grissess
Copy link
Contributor Author

That'd be a good idea, though that leaves to question how that should be done :P

Perhaps check if pkg-config fails with one, then try the other? (Maybe, additionally, some other cases like "lua5.1", "lua50", etc.?)

@BahaVv
Copy link

BahaVv commented Sep 8, 2014

Status update on this: this may be fixed by pulling LUA entirely from the project. TBA.

@Grissess
Copy link
Contributor Author

Updates on this effort?

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

3 participants