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

Allow version range for windows dependency #112

Closed
wants to merge 2 commits into from

Conversation

Kijewski
Copy link
Collaborator

@Kijewski Kijewski commented Aug 15, 2023

iana-time-zone works with any windows version since 0.30.0. We don't have to care which version the user actually uses. We only need to make sure that the selected version is not too old. Let cargo decide everything else.

The newest version of windows, 0.51.0, has an msrv of 1.56. If a user of iana-time-zone is not able or willing to update their rust installation, and want to continue using rustc 1.48, they can select an older version of the windows crate, and still receive any updates iana-time-zone might have in the future.

Resolves #111.

`iana-time-zone` works with any `windows` version since `0.30.0`. We
don't have to care which version the user actually uses. We only need
to make sure that the selected version is not too old. Let `cargo`
decide everything else.

The newest version of `windows`, `0.51.0`, has an msrv of `1.56`. If a
user of `iana-time-zone` is not able or willing to update their rust
installation, and want to continue using rustc `1.48`, they can select
an older version of the `windows` crate, and still receive any updates
`iana-time-zone` might have in the future.
@Kijewski Kijewski added dependencies Pull requests that update a dependency file Tier-1 Rust Tier-1 platform labels Aug 15, 2023
@MarijnS95
Copy link
Contributor

FYI if the MSRV range wasn't this strict this crate could migrate to windows-core and a "minimal" set of bindings emitted with windows-bindgen, more details here: #97 (comment)

Maybe something for the future 👍

@astraw
Copy link
Member

astraw commented Oct 17, 2023

This is redundant now with other recent merges. Closing, but please re-open (and update) if this is in error.

@astraw astraw closed this Oct 17, 2023
@Kijewski Kijewski deleted the pr-windows-051 branch October 17, 2023 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file Tier-1 Rust Tier-1 platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants