arbor-sim / gui

Other
2 stars 6 forks source link

Fix CI #18

Closed brenthuisman closed 1 year ago

brenthuisman commented 1 year ago
brenthuisman commented 1 year ago

On BUILD_SHARED_LIBS before add_subdirectory(glfw): you mean instead of passing the flag to CMake?

The lib messing stuff up is glm, not glfw. But I can't find a add_subdirectory(glm) anywhere in the project, only target_include_directories(arbor-gui-deps SYSTEM INTERFACE 3rd-party/glm) (header only?). Would setting it before this line be what you want to try?

thorstenhater commented 1 year ago

Yes glm is header only, so it cannot set BUILD_SHARED_LIBS=ON, because we never executed their CMake. Thus, glm cannot be the issue.

Also I use glm since the beginning of this project. glbinding is new though, so I start poking there and they seem to be the culprit

3rd-party/glbinding/CMakeLists.txt
86:option(BUILD_SHARED_LIBS              "Build shared instead of static libraries."                 ON)
brenthuisman commented 1 year ago

OK, let's see if that works.

brenthuisman commented 1 year ago

So, jobs are passing, AppImage #worksforme, but the DiskIm does not work on @thorstenhater's M1.