Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documented possible exception being thrown in ClassLoader destructor. #234

Open
wants to merge 1 commit into
base: noetic-devel
Choose a base branch
from

Conversation

remod
Copy link

@remod remod commented Mar 1, 2022

The class member class_loader::MultiLibraryClassLoader lowlevel_class_loader_ can throw an exception when being destroyed (which actually happened to me). This can be hard to fix, especially if one defines the ClassLoader instance to be static. In this case the instance is destroyed when the executable exits where you can't catch exceptions.

Documenting this behavior should help others to be careful. Also it is consistent with the documentation of the other methods.

@remod
Copy link
Author

remod commented Mar 1, 2022

It looks like the pipeline failed due to connection issues. Will retry later.

10:00:09 E: Unable to locate package ros-noetic-catkin
10:00:09 E: Unable to locate package ros-noetic-class-loader
10:00:09 E: Unable to locate package ros-noetic-cmake-modules
10:00:09 E: Unable to locate package ros-noetic-rosconsole
10:00:09 E: Unable to locate package ros-noetic-roslib
10:00:09 'apt install' failed and likely requires 'apt update' to run again

@remod
Copy link
Author

remod commented Mar 7, 2022

@nuclearsandwich kind ping

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.

None yet

1 participant