Skip to content

Commit

Permalink
fix: trying to add back pip compilation, ok with shared
Browse files Browse the repository at this point in the history
  • Loading branch information
asonolet committed Jan 24, 2025
1 parent 5d6b0f0 commit 7412353
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 2 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -4,3 +4,4 @@ install/
venv/
__pycache__/
ConanPresets.json
venv-*
2 changes: 1 addition & 1 deletion conanfile.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ class Recipe(ConanFile):
"is_32bit": [True, False],
}
default_options = {
"shared": True,
"shared": False,
"fPIC": True,
"parallel": False,
"is_32bit": False,
Expand Down
2 changes: 1 addition & 1 deletion medcoupling_core/source/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,7 @@ set_target_properties(
INSTALL_RPATH ${MEDCOUPLING_INSTALL_LIBS_DEPS_RPATH}
OUTPUT_NAME "medcoupling")

target_link_libraries(medcouplingcpp PRIVATE medcoupling_core_flags)
target_link_libraries(medcouplingcpp PRIVATE $<BUILD_INTERFACE:medcoupling_core_flags>)
target_compile_definitions(medcouplingcpp PRIVATE interpkernel_EXPORTS)

target_include_directories(medcouplingcpp
Expand Down
1 change: 1 addition & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -16,3 +16,4 @@ wheel.packages = []
[tool.scikit-build.cmake.define]
CMAKE_PROJECT_TOP_LEVEL_INCLUDES = "ext/cmake-conan/conan_provider.cmake"
CMAKE_INSTALL_RPATH_USE_LINK_PATH = "ON"
BUILD_SHARED_LIBS = "ON"

0 comments on commit 7412353

Please sign in to comment.