Skip to content

Commit

Permalink
cmake: address some small nits
Browse files Browse the repository at this point in the history
Don't emit a message when processing tests/CMakeLists.txt, and emit a
slightly cleaner error if Lua was not found.

Signed-off-by: Kyle Evans <[email protected]>
  • Loading branch information
kevans91 committed Jan 23, 2024
1 parent 90a598a commit 803dbdc
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
5 changes: 4 additions & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,10 @@ cmake_minimum_required(VERSION 3.18)
project(orch)

find_package("Lua")
# XXX Make sure it was found?

if(NOT LUA_FOUND)
message(FATAL_ERROR "No Lua installation found; please install Lua")
endif()

set(LUA_MODLIBDIR "/usr/local/lib/lua/${LUA_VERSION_MAJOR}.${LUA_VERSION_MINOR}"
CACHE PATH "Path to install lua dynamic library modules into")
Expand Down
1 change: 0 additions & 1 deletion tests/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,3 +1,2 @@
message("${CMAKE_BINARY_DIR}")
add_custom_target(check
COMMAND env ORCHBIN="${CMAKE_BINARY_DIR}/src/orch" ORCHLUA_PATH="${CMAKE_SOURCE_DIR}/lib" sh "${CMAKE_CURRENT_SOURCE_DIR}/basic_test.sh")

0 comments on commit 803dbdc

Please sign in to comment.