Skip to content

Commit 940d9f7

Browse files
committed
Fix wheel import problem in macosx
Kudos to @davidrudlstorfer for identifying the fix in #516.
1 parent 9e9ab5e commit 940d9f7

2 files changed

Lines changed: 8 additions & 1 deletion

File tree

api/CMakeLists.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -102,6 +102,8 @@ if(RYML_BUILD_API_PYTHON)
102102
RUNTIME_OUTPUT_DIRECTORY "${r}")
103103
if(WIN32)
104104
target_compile_definitions(${t} PUBLIC __WIN32__)
105+
elseif(APPLE)
106+
set_property(TARGET ${t} APPEND PROPERTY LINK_FLAGS "-flat_namespace")
105107
endif()
106108
c4_get_transitive_property(ryml COMPILE_DEFINITIONS ryml_defs)
107109
target_compile_definitions(${t} PUBLIC ${RYML_SWIG_ARCH_DEFINES} ${ryml_defs})

changelog/current.md

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,4 +30,9 @@
3030
```
3131
- [PR#501](https://github.com/biojppm/rapidyaml/pull/501): fix missing tag in `- !!seq []`.
3232
- [PR#508](https://github.com/biojppm/rapidyaml/pull/508): fix build with cmake 4.
33-
- [PR#517](https://github.com/biojppm/rapidyaml/pull/517): fix python wheels for windows.
33+
- [PR#517](https://github.com/biojppm/rapidyaml/pull/517) (fixes [#516](https://github.com/biojppm/rapidyaml/issues/516)): fix python wheels for windows and macosx.
34+
35+
36+
### Thanks
37+
38+
- @davidrudlstorfer

0 commit comments

Comments
 (0)