Skip to content

Commit

Permalink
build: remove workaround for incorrectly packaged libluv
Browse files Browse the repository at this point in the history
This removes a workaround for incorrectly packaged libluv in
90e44ec as it should not be needed
anymore.
  • Loading branch information
dundargoc committed Mar 5, 2023
1 parent ca3bc56 commit da0c66b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 16 deletions.
11 changes: 1 addition & 10 deletions cmake/FindLibluv.cmake
@@ -1,14 +1,5 @@
find_path(LIBLUV_INCLUDE_DIR luv/luv.h)

# Explicitly look for luv.so. #10407
list(APPEND LIBLUV_NAMES luv_a luv libluv_a luv${CMAKE_SHARED_LIBRARY_SUFFIX})

find_library(LIBLUV_LIBRARY NAMES ${LIBLUV_NAMES})

set(LIBLUV_LIBRARIES ${LIBLUV_LIBRARY})
set(LIBLUV_INCLUDE_DIRS ${LIBLUV_INCLUDE_DIR})

find_library(LIBLUV_LIBRARY NAMES luv_a luv libluv_a luv.so)
find_package_handle_standard_args(Libluv DEFAULT_MSG
LIBLUV_LIBRARY LIBLUV_INCLUDE_DIR)

mark_as_advanced(LIBLUV_INCLUDE_DIR LIBLUV_LIBRARY)
8 changes: 2 additions & 6 deletions src/nvim/CMakeLists.txt
Expand Up @@ -14,12 +14,8 @@ else()
endif()

find_package(Libluv 1.43.0 REQUIRED)
target_include_directories(main_lib SYSTEM BEFORE INTERFACE ${LIBLUV_INCLUDE_DIRS})
# Use "luv" as imported library, to work around CMake using "-lluv" for
# "luv.so". #10407
add_library(luv UNKNOWN IMPORTED)
set_target_properties(luv PROPERTIES IMPORTED_LOCATION ${LIBLUV_LIBRARIES})
target_link_libraries(main_lib INTERFACE luv)
target_include_directories(main_lib SYSTEM BEFORE INTERFACE ${LIBLUV_INCLUDE_DIR})
target_link_libraries(main_lib INTERFACE ${LIBLUV_LIBRARY})

find_package(Iconv REQUIRED)
find_package(Libtermkey 0.22 REQUIRED)
Expand Down

0 comments on commit da0c66b

Please sign in to comment.