Glassvision/IfcOpenShell
1
0
Fork 0
mirror of https://github.com/IfcOpenShell/IfcOpenShell.git synced 2026-09-21 23:11:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

cmake: give the wasm IfcParse its Boost headers back

Browse Source 
ae2f27c3a replaced the global include_directories(${Boost_INCLUDE_DIRS})
with a Boost::headers target appended to Boost_LIBRARIES. The wasm branch of
IfcParse's link line never included Boost_LIBRARIES, so it lost the include
path and the Pyodide build has failed since with

  src/ifcparse/exception.h:25:10: fatal error: 'boost/lexical_cast.hpp' file not found

Nobody noticed because no Pyodide build had run since 2026-08-28.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This commit is contained in:
Dion Moult
2026-09-17 15:53:04 +10:00
parent 998060b686
commit e389fb6145
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ifcparse/CMakeLists.txt
+2 -1
View File
@@ -59,7 +59,8 @@ if (NOT CREATE_BUNDLE)
set_target_properties(IfcParse PROPERTIES VERSION "${PROJECT_VERSION}" SOVERSION "${PROJECT_VERSION_MAJOR}.${PROJECT_VERSION_MINOR}")
endif()
if(WASM_BUILD)
target_link_libraries(IfcParse plugin ${BCRYPT_LIBRARIES})
# No Boost link libraries for wasm, but the headers are still needed.
target_link_libraries(IfcParse plugin Boost::headers ${BCRYPT_LIBRARIES})
else()
target_link_libraries(IfcParse plugin ${Boost_LIBRARIES} ${BCRYPT_LIBRARIES})
endif()