-
Notifications
You must be signed in to change notification settings - Fork 71
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
Update dependencies and fix Windows compatibility on Python 3.12 #183
base: master
Are you sure you want to change the base?
Conversation
"Topic :: Software Development :: Libraries :: Python Modules", | ||
"Topic :: System :: Hardware :: Hardware Drivers", | ||
"Topic :: System :: Hardware :: Hardware Drivers :: Smartcard", | ||
"Topic :: System :: Hardware :: Hardware Drivers :: Token" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only minor comment: please leave the indentation intact. Hard to say what you changed here?
* /Distributed under the terms of the OASIS IPR Policy, | ||
* [http://www.oasis-open.org/policies-guidelines/ipr], AS-IS, WITHOUT ANY | ||
* IMPLIED OR EXPRESS WARRANTY; there is no warranty of MERCHANTABILITY, FITNESS FOR A | ||
* PARTICULAR PURPOSE or NONINFRINGEMENT of the rights of others. | ||
*/ | ||
|
||
/* Latest version of the specification: | ||
* http://docs.oasis-open.org/pkcs11/pkcs11-base/v2.40/pkcs11-base-v2.40.html | ||
/* | ||
* PKCS #11 Specification Version 3.1 | ||
* OASIS Standard | ||
* 23 July 2023 | ||
* Copyright (c) OASIS Open 2023. All Rights Reserved. | ||
* Source: https://docs.oasis-open.org/pkcs11/pkcs11-spec/v3.1/os/include/pkcs11-v3.1/ | ||
* Latest stage of narrative specification: https://docs.oasis-open.org/pkcs11/pkcs11-spec/v3.1/pkcs11-spec-v3.1.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Distribution terms are no longer noted here. I think it's still good to keep them here, right?
To be transparent about the maintenance situation: I don't have any access to HSMs. I'm not sure about @kislyuk? This means that any changes cannot currently be tested with real HSMs. |
@mathiasertl I can't promise anything ongoing, but I might be able to run the tests on Linux against AWS CloudHSMv2, if that would be of any use? Someone would have to note down exactly how to run the tests though! (I'd rather not try to work it out, get it wrong, and invalidate the results.) |
Any update on this pull request? I'd really love to see 0.8.0 being released and finally being able to use python 3.12 on my windows box. |
I guess this means it won't work with Python 3.13 also? |
I guess so. |
This PR has minor changes requested, if they are implemented I can merge this. Note however that I don't think I can upload releases to PyPI at the moment. @kislyuk do you have upload permissions by now? |
I do not. @danni can you please invite me to the python-pkcs11 package on PyPI? |
Yes I can do that. Same username? |
Yes, my PyPI username is identical to my GitHub one.
…On Mon, Dec 9, 2024 at 16:26 Danielle Madeley ***@***.***> wrote:
Yes I can do that. Same username?
—
Reply to this email directly, view it on GitHub
<#183 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGSOPNR2R54YTHQ6MG5J6L2EYYKNAVCNFSM6AAAAABPCOW5AOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRZHEYDOMZRGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
My username is mati: https://pypi.org/user/mati/ |
Any updates on when this might get merged in? |
This is follow-up of the now closed PR #174. This new PR tries to update the PKCS#11 specification to the latest published version and brings compatibility to Python 3.12 on Windows.
The specific changes of this PR are:
The package has been tested on new virtual environments using Python 3.12 on Windows 11 with the asepkcs.dll (Athena Smartcard System ASE Card).