-
-
Notifications
You must be signed in to change notification settings - Fork 393
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 NCryptFreeObject #319
Add NCryptFreeObject #319
Conversation
Don't forget to add ncrypt to the dev dependencies of winapi so CI can actually test building it. |
Fixed! |
We have to pull some ncrypt stuff in temporarily until retep998/winapi-rs#319 lands.
Ping |
This PR will need to be updated to the winapi 0.3 new world order. #316 |
Updated! The mingw dlls have not been added anywhere since the appropriate crates haven't been created yet. |
Could you rebase once a nightly comes out with rust-lang/rust#37361 so this PR can be tested properly? |
Sure, will do. |
ca30d4a
to
f4b86ec
Compare
Updated, but looks like the GNU build is failing due to the missing library shims. |
@@ -14,3 +14,4 @@ | |||
#[cfg(feature = "um-wincon")] pub mod wincon; | |||
#[cfg(feature = "um-wingdi")] pub mod wingdi; | |||
#[cfg(feature = "um-winnt")] pub mod winnt; | |||
#[cfg(feature = "um-ncrypt")] pub mod ncrypt; |
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.
Alphabetical order
@@ -69,7 +73,9 @@ headers = ["headers-shared", "headers-um", "headers-vc"] | |||
"vc-vcruntime" = [] | |||
libraries = [ | |||
"kernel32", | |||
"ole32" | |||
"ole32", | |||
"ncrypt" |
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.
Alphabetical order
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.
I think you might have forgot about this when updating.
@@ -56,6 +59,7 @@ headers = ["headers-shared", "headers-um", "headers-vc"] | |||
"um-combaseapi" = ["shared-basetsd", "shared-minwindef", "shared-wtypesbase", "um-objidlbase", "ole32"] | |||
"um-consoleapi" = ["shared-minwindef", "um-wincon", "um-winnt", "kernel32"] | |||
"um-minwinbase" = ["shared-ntstatus"] | |||
"um-ncrypt" = ["shared-basetsd", "shared-sspi"] |
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.
This should also depend on "ncrypt"
because it has functions from that library.
Updated! |
I added crates in the |
Updated! |
No description provided.