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

Improve windows-bindgen for std usage #2610

Merged
merged 2 commits into from Aug 16, 2023
Merged

Improve windows-bindgen for std usage #2610

merged 2 commits into from Aug 16, 2023

Conversation

kennykerr
Copy link
Collaborator

As discussed in #2609, this update makes the following changes:

  • Strips out comments from response files.
  • Removes the special case for SOCKET

Copy link
Collaborator

@ChrisDenton ChrisDenton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works for me! I've updated my std branch to move more of the config options into the windows_sys.lst file and that seems to be working.

@kennykerr
Copy link
Collaborator Author

Sweet, thanks for checking!

@kennykerr kennykerr merged commit 8f47253 into master Aug 16, 2023
47 checks passed
@kennykerr kennykerr deleted the riddle-std branch August 16, 2023 14:27
@kennykerr
Copy link
Collaborator Author

@ChrisDenton I'm preparing a release.

@ChrisDenton
Copy link
Collaborator

Great! My PR should be ready to go once that's done (give or take some minor tweaks).

@klensy
Copy link

klensy commented Aug 16, 2023

Wait a second, maybe fix for microsoft/win32metadata#1680 can be here too?

@kennykerr
Copy link
Collaborator Author

I'd need another release of the Win32 metadata. I don't think there is one yet with that change. That would also mean a breaking change to the API surface so unless that's really urgent I'd prefer to hold off on that. Thoughts?

@klensy
Copy link

klensy commented Aug 16, 2023

In that case, this can wait until next release.

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.

None yet

3 participants