Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

ci: stop windows build on v1.16 branch #31893

Merged
merged 1 commit into from
May 31, 2023

Conversation

yihau
Copy link
Contributor

@yihau yihau commented May 31, 2023

Problem

due to the same reason with #31353. stop windows building on v1.16 for a while until we upgrade tokio crates successfully 😢

@yihau yihau force-pushed the ignore-windows-build-for-116 branch from 5daff5b to 48edd16 Compare May 31, 2023 15:34
@yihau yihau requested review from ryoqun and CriesofCarrots May 31, 2023 15:45
@yihau yihau merged commit 30f9e43 into solana-labs:v1.16 May 31, 2023
@yihau yihau deleted the ignore-windows-build-for-116 branch May 31, 2023 17:26
@ryoqun
Copy link
Contributor

ryoqun commented Jun 4, 2023

I've been pinging the maintainer for weeks. but there's no response... MSxDOS/ntapi#15 (comment)

@ryoqun
Copy link
Contributor

ryoqun commented Jun 5, 2023

hehe, i hacked up #31961

we upgrade tokio crates successfully

btw, how soon? i wonder my additional few hours to polish the pr should worth to restore windows build meanwhile.

@yihau
Copy link
Contributor Author

yihau commented Jun 5, 2023

AFAIK the last discussion is here https://discord.com/channels/428295358100013066/560503042458517505/1100447210937790526
(it's about 2 months ago. not sure do we have a deadline for it.)

@ryoqun ryoqun changed the title ci: stop windows build ci: stop windows build on v1.16 branch Jun 5, 2023
ryoqun added a commit to ryoqun/solana that referenced this pull request Jun 6, 2023
ryoqun added a commit to ryoqun/solana that referenced this pull request Jun 6, 2023
ryoqun added a commit that referenced this pull request Jun 6, 2023
* Shift crossbeam comment for upcoming 2nd patch... (#31963)

* Patch ntapi to restore windows build (#31961)

* Patch ntapi to restore windows build

* Update Cargo.lock...

* Add comment for justification of this patching

MSxDOS/ntapi#11
MSxDOS/ntapi#12

* Revert "ci: stop windows building on master temporarily (#31353)"

This reverts commit 2dcdfff.

* Use solana-labs fork

* Ugh..

* Patch ntapi more thoroughly (#31970)

* Patch spl-token-cli build as well...

* Patch sbf/Cargo.toml for consistency

* Remove --locked for cli-arg based patch... (#31971)

* Bump patched ntapi from v0.3.6 to v0.3.7 (#31981)

* Revert "ci: stop windows build (#31893)"

This reverts commit 30f9e43.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants