Skip to content

Commit

Permalink
update patch to also skip building cucim_tests
Browse files Browse the repository at this point in the history
  • Loading branch information
grlee77 committed Oct 27, 2023
1 parent 1944dc4 commit 196c15a
Show file tree
Hide file tree
Showing 5 changed files with 66 additions and 26 deletions.
4 changes: 2 additions & 2 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ set(CUCIM_PACKAGE_NAME cucim) # cucim
################################################################################
add_subdirectory(cpp)
add_subdirectory(gds)
add_subdirectory(benchmarks)
# add_subdirectory(benchmarks)
add_subdirectory(examples/cpp)

################################################################################
Expand All @@ -155,7 +155,7 @@ configure_file(${CMAKE_CURRENT_LIST_DIR}/examples/cpp/CMakeLists.txt.examples.re
set(INSTALL_TARGETS
${CUCIM_PACKAGE_NAME}
fmt-header-only
cucim_benchmarks
# cucim_benchmarks
)

install(TARGETS ${INSTALL_TARGETS}
Expand Down
2 changes: 1 addition & 1 deletion ci/build_wheel.sh
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ else
apt install yasm libopenslide-dev -y
dpkg -L libopenslide-dev

git apply ci/omit-benchmarks-build.patch
git apply ci/omit-benchmarks-and-tests-from-build.patch
fi

# First build the C++ lib using CMake via the run script
Expand Down
62 changes: 62 additions & 0 deletions ci/omit-benchmarks-and-tests-from-build.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 0ab2e17..2a42462 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -138,7 +138,7 @@ set(CUCIM_PACKAGE_NAME cucim) # cucim
################################################################################
add_subdirectory(cpp)
add_subdirectory(gds)
-add_subdirectory(benchmarks)
+# add_subdirectory(benchmarks)
add_subdirectory(examples/cpp)

################################################################################
@@ -155,7 +155,7 @@ configure_file(${CMAKE_CURRENT_LIST_DIR}/examples/cpp/CMakeLists.txt.examples.re
set(INSTALL_TARGETS
${CUCIM_PACKAGE_NAME}
fmt-header-only
- cucim_benchmarks
+ # cucim_benchmarks
)

install(TARGETS ${INSTALL_TARGETS}
diff --git a/ci/omit-benchmarks-build.patch b/ci/omit-benchmarks-build.patch
index 24b8371..e69de29 100644
--- a/ci/omit-benchmarks-build.patch
+++ b/ci/omit-benchmarks-build.patch
@@ -1,22 +0,0 @@
-diff --git a/CMakeLists.txt b/CMakeLists.txt
-index 0ab2e17..2a42462 100644
---- a/CMakeLists.txt
-+++ b/CMakeLists.txt
-@@ -138,7 +138,7 @@ set(CUCIM_PACKAGE_NAME cucim) # cucim
- ################################################################################
- add_subdirectory(cpp)
- add_subdirectory(gds)
--add_subdirectory(benchmarks)
-+# add_subdirectory(benchmarks)
- add_subdirectory(examples/cpp)
-
- ################################################################################
-@@ -155,7 +155,7 @@ configure_file(${CMAKE_CURRENT_LIST_DIR}/examples/cpp/CMakeLists.txt.examples.re
- set(INSTALL_TARGETS
- ${CUCIM_PACKAGE_NAME}
- fmt-header-only
-- cucim_benchmarks
-+ # cucim_benchmarks
- )
-
- install(TARGETS ${INSTALL_TARGETS}
diff --git a/cpp/CMakeLists.txt b/cpp/CMakeLists.txt
index 67409a6..d2230b3 100644
--- a/cpp/CMakeLists.txt
+++ b/cpp/CMakeLists.txt
@@ -214,7 +214,7 @@ add_library(${CUCIM_PACKAGE_NAME}::${CUCIM_PACKAGE_NAME}-header-only ALIAS ${CUC
################################################################################
# Add tests
################################################################################
-add_subdirectory(tests)
+# add_subdirectory(tests)

#################################################################################
## Add bindings
22 changes: 0 additions & 22 deletions ci/omit-benchmarks-build.patch

This file was deleted.

2 changes: 1 addition & 1 deletion cpp/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -214,7 +214,7 @@ add_library(${CUCIM_PACKAGE_NAME}::${CUCIM_PACKAGE_NAME}-header-only ALIAS ${CUC
################################################################################
# Add tests
################################################################################
add_subdirectory(tests)
# add_subdirectory(tests)

#################################################################################
## Add bindings
Expand Down

0 comments on commit 196c15a

Please sign in to comment.