Skip to content

Commit

Permalink
Remove libuuid dependency from C++ runtime
Browse files Browse the repository at this point in the history
libuuid and its headers are not referenced anywhere, so remove it.

Signed-off-by: Bryan Tan <[email protected]>
Signed-off-by: Eric Vergnaud <[email protected]>
  • Loading branch information
Technius authored and ericvergnaud committed Sep 9, 2022
1 parent 543780e commit 90eaf64
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 8 deletions.
4 changes: 0 additions & 4 deletions runtime/Cpp/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -50,10 +50,6 @@ if(CMAKE_VERSION VERSION_EQUAL "3.3.0" OR
CMAKE_POLICY(SET CMP0054 OLD)
endif()

if(CMAKE_SYSTEM_NAME MATCHES "Linux")
find_package(PkgConfig REQUIRED)
pkg_check_modules(UUID REQUIRED uuid)
endif()
if(APPLE)
find_library(COREFOUNDATION_LIBRARY CoreFoundation)
endif()
Expand Down
5 changes: 1 addition & 4 deletions runtime/Cpp/runtime/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -62,10 +62,7 @@ if (ANTLR_BUILD_CPP_TESTS)
gtest_discover_tests(antlr4_tests)
endif()

if(CMAKE_SYSTEM_NAME MATCHES "Linux")
target_link_libraries(antlr4_shared ${UUID_LIBRARIES})
target_link_libraries(antlr4_static ${UUID_LIBRARIES})
elseif(APPLE)
if(APPLE)
target_link_libraries(antlr4_shared ${COREFOUNDATION_LIBRARY})
target_link_libraries(antlr4_static ${COREFOUNDATION_LIBRARY})
endif()
Expand Down

0 comments on commit 90eaf64

Please sign in to comment.