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 initial windows support #57
add initial windows support #57
Conversation
ec80f58
to
95157a5
Compare
Builders look happy now, sorry for the spam in this CL. The CI-Force is not strong in me 😃. |
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.
just a couple nits. thanks for the PR!
Adds support for windows via winscard. This CL leverages the winscard api via calling into the dll. For more information see https://docs.microsoft.com/en-us/windows/win32/api/winscard/. Fixes go-piv#56 Signed-off-by: Tobias Kohlbau <tobias@kohlbau.de>
95157a5
to
53a7bd7
Compare
I've addressed your comments. I hope it's fine that I've force pushed. Github is still not working nicely with this workflow, sorry. |
It looks like the windows build worked fine. Am I seeing something different? https://github.com/go-piv/piv-go/pull/57/checks?check_run_id=638496144 Will merge this including the workflow if you think it's running correctly |
Oh I get it my comment is confusing. I referred to the workflow of force pushing which deletes the association between the new code and the old comments. The GitHub actions workflow run is totally fine. |
Add windows builder and move to setup-go@v2 as it supports adding GOPATH/bin to PATH without quirks. Signed-off-by: Tobias Kohlbau <tobias@kohlbau.de>
53a7bd7
to
aeec760
Compare
I've added a build constraint for |
Adds support for windows via winscard. This CL leverages the
winscard api via calling into the dll. For more information
see https://docs.microsoft.com/en-us/windows/win32/api/winscard/.
Fixes #56