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

add_dll_directory #8

Closed
m-kuhn opened this issue Feb 12, 2024 · 2 comments · Fixed by #3
Closed

add_dll_directory #8

m-kuhn opened this issue Feb 12, 2024 · 2 comments · Fixed by #3

Comments

@m-kuhn
Copy link
Contributor

m-kuhn commented Feb 12, 2024

To load deps, python needs add_dll_directory() calls. At the moment, python cannot import any of the libraries.
Patching individual ports doesn't seem to be the way to happiness. Should the python3 port be patched to add the main library directory on load? Should another utility port be added that adds this to the python path (sitecustomize or similar)?

@Neumann-A
Copy link
Member

At the moment, python cannot import any of the libraries.
Patching individual ports doesn't seem to be the way to happiness.

It also breaks embedding since the files become impossible to relocate.

Should the python3 port be patched to add the main library directory on load

I already have a patch (actual two) in my python overlay derived from conda.

@m-kuhn
Copy link
Contributor Author

m-kuhn commented Feb 13, 2024

Should the python3 port be patched to add the main library directory on load

I already have a patch (actual two) in my python overlay derived from conda.

Nice, are you planning to port these or would you like me to do so?

@m-kuhn m-kuhn mentioned this issue Mar 3, 2024
@m-kuhn m-kuhn closed this as completed in #3 Mar 5, 2024
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 a pull request may close this issue.

2 participants