-
Notifications
You must be signed in to change notification settings - Fork 10.6k
WinSDK: add Shell32 #33849
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
WinSDK: add Shell32 #33849
Conversation
This header and .lib features useful APIs such as [SHFileOperation](https://docs.microsoft.com/en-us/windows/win32/api/shellapi/nf-shellapi-shfileoperationw), which seems to be the preferred way to manipulate file systems for certain things.
|
@swift-ci smoke test |
|
@swift-ci smoke test Windows Platform |
|
smoke test Linux Platform |
compnerd
left a comment
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.
Hmm, I'm wondering if we want to rename the old module or not. The original module is the lightweight Shell API, and this is the full one. Either way, as long as it builds, this seems reasonable, but I would like to test the build locally, since we're missing coverage on Foundation :(. I learnt the hard way with the last rebranch.
|
ah. The only test I ran locally is modifying the installed .modulemap and use one of the APIs in the local build. It appeared to have worked well. |
|
@swift-ci smoke test Linux Platform |
compnerd
left a comment
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.
Seems that I can build Foundation with this change just fine. Still not sure about how to best handle the lightweight vs full API surface for shell, but that can be improved as a follow up.
|
@swift-ci please smoke test Linux platform |
This header and .lib features useful APIs such as SHFileOperation, which seems to be the preferred way to manipulate file systems for certain things.