gazebosim / gz-rendering

C++ library designed to provide an abstraction for different rendering engines. It offers unified APIs for creating 3D graphics applications.
https://gazebosim.org
Apache License 2.0
56 stars 51 forks source link

Try fixing test failure with RTLD_NODELETE #1038

Closed scpeters closed 3 months ago

scpeters commented 3 months ago

🦟 Bug fix

Alternative to https://github.com/gazebosim/gz-rendering/pull/1033

Summary

From #1033:

The INTEGRATION_load_unload test crashes with ogre 1.x. The tests checks to see if the render engine can be loaded and unloaded in a thread (which is how gz sim runs the render engine). The crash happens on ubuntu 24.04 with the system debs, see https://github.com/gazebosim/gz-rendering/issues/1007 for more info.

Instead of the approach from #1033, this uses the suggestion from https://github.com/gazebosim/gz-rendering/pull/1033#issuecomment-2289815324

To test (also from #1033)

The INTEGRATION_load_unload test should now pass with ogre 1.x on Ubuntu 24.04 with the system ogre debs:

GZ_ENGINE_TO_TEST=ogre  ./build/gz-rendering9/bin/INTEGRATION_load_unload

gz-sim should no longer crash on exit:

gz sim -v 4 -r --iterations 5 -s sensors_demo.sdf --render-engine ogre

Checklist

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

scpeters commented 3 months ago

we will use #1033