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

cargo-bootstrap: Update to 1.59.0 #36267

Closed
wants to merge 1 commit into from

Conversation

subnut
Copy link
Contributor

@subnut subnut commented Mar 21, 2022

DRAFT because I think we probably also need to update the cargo package too?
IDK. Let me know if you think this is appropriate to merge as-is, and I wil un-draft it. 🙂

Testing the changes

  • I tested the changes in this PR: NO

@subnut subnut force-pushed the cargo-bootstrap branch 2 times, most recently from 73fc568 to 1004f4d Compare March 21, 2022 19:26
@subnut
Copy link
Contributor Author

subnut commented Mar 21, 2022

@q66 Please review.
(I made some pretty drastic changes, not gonna lie 😅)

@q66
Copy link
Contributor

q66 commented Mar 21, 2022

rust, rust-bootstrap, cargo, cargo-bootstrap are always updated together in a single PR

@q66 q66 closed this Mar 21, 2022
@q66
Copy link
Contributor

q66 commented Mar 21, 2022

(those changes are also incorrect, the ppc special case should not be removed)

@subnut
Copy link
Contributor Author

subnut commented Mar 21, 2022

@q66

Okay. So what should be the updated value of version ?
Or should I keep version as-it-is and bump revision instead?
Or something else?

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