Skip to content

Conversation

artcore-c
Copy link

Description

Ensured CMake uses -L/opt/local/lib for libminizip resolution.
Confirmed valid with MacPorts minizip, assimp install location and symlinks.
Avoids brute-force or patch-on-patch approaches that can break if upstream changes.

Fixes # (resolves build error ld: library not found for -lminizip on MacOS x86_64)
Fixed linker path propagation into external Ogre plugins (Codec_Assimp).
Added OGRE_CMAKE_ARGS to inject the linker path into the plugin build environment directly.

artcore-c added 3 commits June 9, 2025 23:40
rviz/rviz_ogre_vendor/CMakeLists.txt

Signed-off-by: artcore-c <[email protected]>
Signed-off-by: artcore-c <[email protected]>
Fixed linker path propagation into external Ogre plugins (Codec_Assimp) on MacOS x86_64.
Ensured CMake uses -L/opt/local/lib for libminizip resolution.
Confirmed MacPorts file tree and symlinks valid.


Signed-off-by: artcore-c <[email protected]>
@artcore-c artcore-c changed the title Fix/linker path mac os x86 64 Fix/linker path mac os x86_64 Jun 10, 2025
@artcore-c artcore-c changed the title Fix/linker path mac os x86_64 Fix/linker path MacOS x86_64 Jun 10, 2025
@artcore-c artcore-c changed the title Fix/linker path MacOS x86_64 Fix/Linker path MacOS x86_64 Jun 10, 2025
@artcore-c
Copy link
Author

artcore-c commented Jun 10, 2025

the Linux build failed on a part I did not change, rviz_default_plugins. so the original CMakeLists.txt will still fail on Linux.

there was only a stderr warning on rviz_ogre_vendor, which is pretty normal for this large group of submodules

22:13:26 Failed <<< rviz_default_plugins [14min 7s, exited with code 2]
22:13:26
22:13:26 Summary: 7 packages finished [38min 39s]
22:13:26 1 package failed: rviz_default_plugins
22:13:26 3 packages had stderr output: rviz_common rviz_default_plugins rviz_ogre_vendor
22:13:26 1 package not processed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant