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 to 0.48.0 #93

Merged
merged 1 commit into from Jul 3, 2023
Merged

Conversation

KapJI
Copy link
Contributor

@KapJI KapJI commented Jun 29, 2023

Updated version of #89.

Release notes: https://github.com/microsoft/windows-rs/releases/tag/0.48.0

@KapJI
Copy link
Contributor Author

KapJI commented Jun 29, 2023

Looks like Appveyor has issues with TLS now and it's unable to download rust installer or crates.

I'd be glad if you help me with this or just check it locally to make sure all tests pass.

@KapJI
Copy link
Contributor Author

KapJI commented Jun 29, 2023

By the way, maybe you want to migrate to Github Actions?

It's usually more reliable and has Windows Server 2022 and 2019 images. I can submit a PR for that if you want to.

@steffengy
Copy link
Owner

@KapJI yeah fine with migrating there vs spending time to fix it until the next appveyor thing breaks the next thing.

@steffengy
Copy link
Owner

@KapJI should be done.

@KapJI
Copy link
Contributor Author

KapJI commented Jul 3, 2023

I rebased this PR but GH checks need approval. That's quite annoying and I usually change this setting to only require approval from new github users in my repos. This can be done in repo settings:
Screenshot 2023-07-03 at 12 13 41

Also looks like Appveyor needs to be disabled in Settings->Webhooks. Because now it fallbacks to default config.

@steffengy
Copy link
Owner

@KapJI Changed setting, appveyor is fully removed already but that preexisting last build check sticks around.

@steffengy steffengy merged commit b453113 into steffengy:master Jul 3, 2023
7 of 8 checks passed
@KapJI KapJI deleted the windows-sys-0.48 branch July 3, 2023 11:33
@steffengy
Copy link
Owner

Released as v0.1.22

@djc
Copy link

djc commented Jul 3, 2023

Thanks @KapJI and @steffengy for getting this done!

facebook-github-bot pushed a commit to facebookincubator/reindeer that referenced this pull request Jul 5, 2023
Summary:
It includes steffengy/schannel-rs#93 which removes dependency on old version of `windows-sys` (or `winapi` in this case).

Now we should less dependencies on older `windows-sys` which should make it easier to maintain.

Reviewed By: zertosh

Differential Revision: D47195354

fbshipit-source-id: 301132c32148ce08599636b4c9688ac12597459a
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