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

Handle missing dependencies of plugins during plugin_load_try #13

Merged
merged 1 commit into from Sep 5, 2014
Merged

Handle missing dependencies of plugins during plugin_load_try #13

merged 1 commit into from Sep 5, 2014

Conversation

ghost
Copy link

@ghost ghost commented Sep 5, 2014

Instead of letting m64py crash with an OSError when the loading of a plugin fails, it should better create an error message but continue working with the remaining plugins.

An error can for example look like this:

Frontend: ERROR: failed to load plugin /usr/lib/x86_64-linux-gnu/mupen64plus/mupen64plus-video-glide64mk2.so: libpng15.so.15: cannot open shared object file: No such file or directory

This is basically a workaround for the crash mentioned in issue #5. But the specific plugin would still be unusable for the user

Instead of letting m64py crash with an OSError when the loading of a plugin
fails, it should better create an error message but continue working with the
remaining plugins.

An error can for example look like this:

Frontend: ERROR: failed to load plugin /usr/lib/x86_64-linux-gnu/mupen64plus/mupen64plus-video-glide64mk2.so: libpng15.so.15: cannot open shared object file: No such file or directory
gen2brain added a commit that referenced this pull request Sep 5, 2014
Handle missing dependencies of plugins during plugin_load_try
@gen2brain gen2brain merged commit 5ba1e04 into mupen64plus:master Sep 5, 2014
@ghost ghost deleted the failure_plugin branch September 5, 2014 19:18
@ghost ghost mentioned this pull request Oct 8, 2014
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