-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Copy lua51.dll (luajit) to bindir #6148
Conversation
src/CMakeLists.txt
Outdated
@@ -292,6 +292,8 @@ if(WIN32) | |||
set(VORBIS_DLL "" CACHE FILEPATH "Path to libvorbis.dll for installation (optional)") | |||
set(VORBISFILE_DLL "" CACHE FILEPATH "Path to libvorbisfile.dll for installation (optional)") | |||
endif() | |||
|
|||
set(LUA_DLL "" CACHE FILEPATH "Path to lua51.dll for installation (optional)") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be inside if(USE_LUAJIT)
also shouldn't the dll be named libluajit.dll
or similar?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
Well, if you use a custom build script, the name may differ, but using the official build instructions, its called lua51.dll
If you are building with luajit, you had to manually copy the lua51.dll to the Minetest bin dir.
This PR lets CMake do it like the other dll's.