Skip to content

rviz_ogre_vendor: don't build Ogre's EXR codec - #116

Open
Tobias-Fischer wants to merge 1 commit into
mainfrom
ports/ogre-exr
Open

Tobias-Fischer wants to merge 1 commit into
mainfrom
ports/ogre-exr

Conversation

@Tobias-Fischer

@Tobias-Fischer Tobias-Fischer commented Oct 18, 2026 •

Copy link
Copy Markdown
Collaborator

Switches off Ogre's EXR codec in rviz_ogre_vendor (all distributions) with -DOGRE_BUILD_PLUGIN_EXRCODEC=OFF. No functional change: Ogre 1.12's FindOpenEXR only knows OpenEXR 2's library names, so it never finds conda-forge's OpenEXR 3 and the codec isn't built on the CI runners (none of the published humble, jazzy, lyrical or rolling packages has Codec_EXR). On a machine with a system OpenEXR 2 (e.g. Ubuntu's libopenexr-dev), it mixes conda's headers with /usr's libraries and the build fails. RViz doesn't use the codec, so turning it off makes the build independent of the host.

Built locally

  • jazzy, linux-64 (a machine with libopenexr-dev 2.3 installed, where it failed before): builds; the package has the same Ogre plugins as the published one.
  • humble, osx-arm64: builds; no Codec_EXR, the other codecs as before (Assimp, FreeImage, STBI), and libOgreOverlay still links @rpath/libfreetype.6.dylib (humble's FreeType fix from humble: rosdistro sync 2026-10-12 #62).
  • lyrical, rolling: the patches apply to their sync versions; not built locally.

Prepared and built on top of the current rosdistro syncs (#62 humble, #61 jazzy, #60 lyrical, #99 rolling) merged together, so it targets their package versions: merge after them. CI is held with [skip ci] until the syncs are in. The humble (#62) and rolling (#99) syncs also change those patches; rebase after they merge.

🤖 Generated with Claude Code

Ogre 1.12's FindOpenEXR only knows OpenEXR 2's library names, so it never
finds conda-forge's OpenEXR 3 and skips the codec on the CI runners (none of
the published packages has Codec_EXR), but on a machine with a system
OpenEXR 2 it mixes conda's headers with /usr's libraries and fails. RViz
doesn't use the codec: switch it off, so the build no longer depends on
the host.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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