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

change load_library to return the paths of files that failed to load #646

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

airyclam
Copy link

This is related to issue #645 , where the user received error message

  File "/Users/<User>/opt/anaconda3/envs/<envname>/lib/python3.11/site-packages/wand/api.py", line 143, in load_library
    raise IOError('cannot find library; tried paths: ' + repr(tried_paths))

where tried_paths included a path with a library file. The root cause was that the file existed but could not be loaded as it was incompatible.

The new change includes in the error message that a file was found but was unable to be loaded correctly. This indicates to the user that they should check that file to make sure it's compatible with their system.

@Tsubashi
Copy link

Having run into this with users as well, this seems like a good idea.

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.

2 participants