Skip to content

Commit

Permalink
cleanups
Browse files Browse the repository at this point in the history
  • Loading branch information
20DM committed Sep 12, 2024
1 parent 968bcda commit d059a9d
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 20 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/documentation.yml
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ jobs:
export CMAKE_PREFIX_PATH=${{github.workspace}}/local:$CMAKE_PREFIX_PATH
mkdir -p ${{github.workspace}}/build
cd ${{github.workspace}}/build
cmake .. -DCMAKE_INSTALL_PREFIX=${PWD} -Ddompi=OFF -Dopenmp=OFF -Ddocs=ON
cmake .. -DCMAKE_INSTALL_PREFIX=${PWD}/locaL -Ddompi=OFF -Dopenmp=OFF -Ddocs=ON
- name: Build
run: |
Expand Down
19 changes: 0 additions & 19 deletions cpp/purify/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -75,19 +75,6 @@ target_include_directories(libpurify PUBLIC
$<BUILD_INTERFACE:${PROJECT_BINARY_DIR}/include>
$<INSTALL_INTERFACE:include/>)

if (cppflow)
target_link_libraries(libpurify "${TENSORFLOW_LIB}")
# Make cppflow include directory public. Install interface can't point to source directory,
# so it needs to be separately defined, explained in
# https://stackoverflow.com/questions/25676277/cmake-target-include-directories-prints-an-error-when-i-try-to-add-the-source
# Add /usr/local/include for default location of TensorFlow headers
target_include_directories(libpurify PUBLIC
$<BUILD_INTERFACE:${cppflow_INCLUDE_DIR}>
$<BUILD_INTERFACE:/usr/local/include>
$<INSTALL_INTERFACE:cppflow/include/>
)
endif()

add_include_dir(
${EIGEN3_INCLUDE_DIR}
${Boost_INCLUDE_DIR}
Expand Down Expand Up @@ -124,12 +111,6 @@ if(PURIFY_CASACORE_LOOKUP)
add_dependencies(libpurify Lookup-CasaCore)
endif()

# Add spdlog as direct dependency
if(spdlog_FOUND)
target_link_libraries(libpurify spdlog::spdlog)
target_include_directories(libpurify SYSTEM PUBLIC ${spdlog_INCLUDE_DIR})
endif()

install(FILES ${HEADERS} DESTINATION include/purify)
install(TARGETS libpurify
EXPORT PurifyTargets
Expand Down

0 comments on commit d059a9d

Please sign in to comment.