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

Patch ntapi for v1.16.x to provide windows binaries #31982

Merged
merged 6 commits into from
Jun 6, 2023

Conversation

ryoqun
Copy link
Member

@ryoqun ryoqun commented Jun 6, 2023

This is cumulative collection of prs to re-enable Windows binaries for v1.16 (see commits list for details)

@yihau i'm not that sure about the precise release process these days. after this pr lands onto the v1.16 branch, v1.16.1 release page will automatically got attached with windows binaries? cc: @willhickey

* 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 (solana-labs#31353)"

This reverts commit 2dcdfff.

* Use solana-labs fork

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

* Patch sbf/Cargo.toml for consistency
@ryoqun ryoqun marked this pull request as draft June 6, 2023 02:48
@ryoqun ryoqun changed the title Patch ntapi for v1.16.x to provide windows binaries [wip] Patch ntapi for v1.16.x to provide windows binaries Jun 6, 2023
@ryoqun ryoqun changed the title [wip] Patch ntapi for v1.16.x to provide windows binaries [tentative] Patch ntapi for v1.16.x to provide windows binaries Jun 6, 2023
@codecov
Copy link

codecov bot commented Jun 6, 2023

Codecov Report

Merging #31982 (17fc407) into v1.16 (333bb3f) will increase coverage by 0.0%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            v1.16   #31982   +/-   ##
=======================================
  Coverage    81.8%    81.9%           
=======================================
  Files         759      759           
  Lines      207357   207357           
=======================================
+ Hits       169781   169872   +91     
+ Misses      37576    37485   -91     

@ryoqun ryoqun marked this pull request as ready for review June 6, 2023 05:49
@ryoqun ryoqun changed the title [tentative] Patch ntapi for v1.16.x to provide windows binaries Patch ntapi for v1.16.x to provide windows binaries Jun 6, 2023
@ryoqun ryoqun requested a review from yihau June 6, 2023 05:52
- v[0-9]+.[0-9]+
- '!v1.16'
Copy link
Member Author

Choose a reason for hiding this comment

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

:byebye:

Copy link
Member

@yihau yihau left a comment

Choose a reason for hiding this comment

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

thank you <3
yes. when this one merge into v1.16. we will have Windows binaries in the next tag!

@ryoqun ryoqun merged commit ca2881a into solana-labs:v1.16 Jun 6, 2023
4 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.

None yet

2 participants