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

PR #709 breaks python 3.6.1 compatibility #745

Open
noamkremen opened this issue Jan 8, 2022 · 6 comments
Open

PR #709 breaks python 3.6.1 compatibility #745

noamkremen opened this issue Jan 8, 2022 · 6 comments

Comments

@noamkremen
Copy link

noamkremen commented Jan 8, 2022

#709
src/nacl/bindings/crypto_generichash.py:14
from typing import NoReturn, TypeVar

NoReturn was introduced in python 3.6.2, on 3.6.1 fails with:
ImportError: cannot import name ‘NoReturn’

@reaperhulk
Copy link
Member

Interesting -- what are you running on that has Python 3.6.1 out of curiosity?

@reaperhulk
Copy link
Member

The two potential fixes are to drop NoReturn or drop support for < 3.6.2. I've been unable to find a Linux distribution under current support that ships 3.6.0/3.6.1, but I'm interested in your particular case.

@noamkremen
Copy link
Author

noamkremen commented Jan 8, 2022

An internal build environment, via pyenv.
typing_extensions makes NoReturn available in python 3.6.1, if you are willing to introduce it as a dependency.

@noamkremen
Copy link
Author

To give more context, my specific path to this error was via paramiko, which will pull in the latest (> 1.0.1) pynacl version by default during installation.
https://github.com/paramiko/paramiko/blob/7714caf79a09dc455a32c6071dd22ba37c399758/setup.py

@yuvalshirav
Copy link

yuvalshirav commented Jan 11, 2022

This is an issue with PyGithub (https://github.com/PyGithub/PyGithub) on 3.6.1.

@reaperhulk
Copy link
Member

@yuvalshirav are you seeing a lot of 3.6.0/3.6.1 users? I'm inclined to just drop NoReturn but am still considering dropping < 3.6.2 if it's sufficiently rare.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants