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

Python 3.8: add NVDA source directory when attempting to import liblouis DLL #10416

Closed
josephsl opened this issue Oct 24, 2019 · 3 comments · Fixed by #12020
Closed

Python 3.8: add NVDA source directory when attempting to import liblouis DLL #10416

josephsl opened this issue Oct 24, 2019 · 3 comments · Fixed by #12020
Labels
Python 3.8 Issues or ideas that can be resolved or derive solutions via Python 3.8.
Milestone

Comments

@josephsl
Copy link
Collaborator

Hi,

A variation for #10385

Is your feature request related to a problem? Please describe.

Python 3.8 adds os.add_dll_directory function that takes path string. This is done for security reasons, as Python's DLL path lookup routine has changed. This affects all sorts of DLL's, including NVDA Helper (see #10385) and for this one, liblouis.dll.

Describe the solution you'd like

Use os.add_dll_directory function right before importing liblouis.dll in lous helper module.

Describe alternatives you've considered

Leave it as is, although it breaks Python 3.8 compatibility.

Additional context

See #10384 for details.

@josephsl josephsl added the Python 3.8 Issues or ideas that can be resolved or derive solutions via Python 3.8. label Oct 24, 2019
@Brian1Gaff
Copy link

Brian1Gaff commented Oct 24, 2019 via email

@josephsl
Copy link
Collaborator Author

Hi,

Eventually we will move onto a future Python release which will include this function, so we might as well start noting some things here and there.

Thanks.

@Adriani90
Copy link
Collaborator

Closing as fixed in #12020.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Python 3.8 Issues or ideas that can be resolved or derive solutions via Python 3.8.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants