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-strings crate #3125

Merged
merged 12 commits into from
Jun 25, 2024
Merged

Add windows-strings crate #3125

merged 12 commits into from
Jun 25, 2024

Conversation

kennykerr
Copy link
Collaborator

@kennykerr kennykerr commented Jun 25, 2024

This update pulls out the Windows string types from the windows-core crate and makes them available directly without a dependency on the windows or windows-core crates. This will help to make these string types more accessible to library crates such as windows-registry (#3121).

These types are still included in the windows-core and windows crates as a dependency so it should not affect existing code too much.

@kennykerr
Copy link
Collaborator Author

Looks like the readme tests were accidentally disabled - cf9b35b turns them back on.

@kennykerr kennykerr requested a review from riverar June 25, 2024 20:25
@kennykerr
Copy link
Collaborator Author

This whole non-Windows appeasement thing is getting old. It's making it too difficult to support Windows, which is the whole point of this project. 🤪 Anyway, the latest version retains basic Linux compatibility for windows-core only to support COM. For now, I needed to keep windows-strings at least building on non-Windows but I plan to limit non-Windows support to COM and nothing more.

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.

I think this is a good approach, not just for windows- crates but also for the windows ecosystem in general.

@kennykerr kennykerr merged commit 472b563 into master Jun 25, 2024
75 checks passed
@kennykerr kennykerr deleted the windows-strings2 branch June 25, 2024 21:54
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.

3 participants