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.

Sign up for GitHub

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

Jump to bottom

Lookup library path before loading #51

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Lookup library path before loading #51

wants to merge 2 commits into from

Conversation

cdlm
Copy link

@cdlm cdlm commented Jun 28, 2021

Fixes #50

cdlm added 2 commits June 28, 2021 14:31
@cdlm
Resolve library path before loading
518bb02 
I'm guessing `ctypes.cdll.LoadLibrary()` only looks in well-known paths
like `/usr/local/lib`. Resolving the full library path first with
`ctypes.util.find_library()` should be more robust.
@cdlm
Rename local variables for consistency
f2f6dd2
@cdlm cdlm mentioned this pull request Jun 28, 2021
Open
@d3sertx
Copy link

d3sertx commented Jul 28, 2022

Can we merge this pull request?

@bashu
Copy link

bashu commented Dec 16, 2022 •
edited
Loading

Using this patch in macports version of pyhidapi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Library lookup doesn't work with custom path
3 participants
@cdlm @d3sertx @bashu