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 windows install instructions #114

Closed
wants to merge 1 commit into from
Closed

Add windows install instructions #114

wants to merge 1 commit into from

Conversation

ehayes2000
Copy link

Specify requirement for use on windows.

Specify windows requirement for use on windows
@ehayes2000 ehayes2000 marked this pull request as ready for review May 14, 2024 04:08
@LudovicRousseau
Copy link
Owner

You do not have to install OpenSC on Windows (or any other OS) to use PyKCS11.
OpenSC is not the only PKCS#11 library.

Or I missed something?

@ehayes2000
Copy link
Author

You are right there are other PKCS#11 implementations.

On windows PKCS#11 is not installed by default and PyKCS11 cannot be imported until a PKCS#11 implementation is added to the path. After spending a considerable amount of time yesterday resolving this import issue I amended the README with my fix. There may be a more accurate description of this fix, but regardless it should be made clear that PKCS#11 must be installed with this library.

@LudovicRousseau
Copy link
Owner

What do you mean by "PyKCS11 cannot be imported"?

import PyKCS11

should always work.

@ehayes2000
Copy link
Author

No. import PyKCS11 did not work until I installed OpenSC I got an error stating that some module could not be found. I'm using a different computer to write this so I'll try to reproduce the issue for you tomorrow and provide more details.

It looks like I should have made this an issue instead of a PR, my mistake.

@ehayes2000 ehayes2000 closed this by deleting the head repository Jun 18, 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 this pull request may close these issues.

2 participants