iiag / iiag-legacy

Iiag is a game
Other
7 stars 7 forks source link

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

Open Grissess opened 10 years ago

Grissess commented 10 years ago

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 commented 10 years ago

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 commented 10 years ago

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 commented 10 years ago

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

Grissess commented 9 years ago

Updates on this effort?