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

Bump windows-sys version to 0.59 #133

Merged
merged 7 commits into from
Oct 23, 2024
Merged

Conversation

zaddach
Copy link
Contributor

@zaddach zaddach commented Oct 22, 2024

This updates the windows-sys version to 0.59.


This change is Reviewable

Copy link
Member

@faern faern left a comment

Choose a reason for hiding this comment

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

Thanks for the upgrade! A few things to fix before merge.

src/sc_handle.rs Outdated Show resolved Hide resolved
Cargo.toml Outdated Show resolved Hide resolved
@faern
Copy link
Member

faern commented Oct 22, 2024

This apparently bumps the MSRV. So you need to update the CI to run on 1.60.0 instead of 1.58.0. See the comment in build-and-test.yml

@zaddach
Copy link
Contributor Author

zaddach commented Oct 23, 2024

I've updated the MSRV, and tested that both cargo test and cargo doc finish without errors.

@faern
Copy link
Member

faern commented Oct 23, 2024

You need to update what Rust version the CI runs on. It still runs on 1.58. Please see https://github.com/mullvad/windows-service-rs/blob/main/.github/workflows/build-and-test.yml#L24-L28

Also don't forget my comment about the documentation links invalidly pointing to Security::SC_HANDLE

@zaddach
Copy link
Contributor Author

zaddach commented Oct 23, 2024

Ahahah, I must still have been a bit tired in the morning. Forgot to actually add the files before committing. Here you go.

Copy link
Member

@faern faern left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks for the contribution!

@faern faern merged commit 5809788 into mullvad:main Oct 23, 2024
10 checks passed
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.

2 participants