-
Notifications
You must be signed in to change notification settings - Fork 13.8k
Make macOS dist build configuration match where reasonable #145883
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
Conversation
@bors2 try jobs=dist-aarch64-apple,dist-x86_64-apple |
Make macOS dist build configuration match where reasonable try-job: dist-aarch64-apple try-job: dist-x86_64-apple
This comment has been minimized.
This comment has been minimized.
💥 Test timed out after |
@bors2 try jobs=dist-aarch64-apple,dist-x86_64-apple |
Make macOS dist build configuration match where reasonable try-job: dist-aarch64-apple try-job: dist-x86_64-apple
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine, feel free to r=me if CI passes
🤷 @bors2 try jobs=dist-aarch64-apple,dist-x86_64-apple |
This comment has been minimized.
This comment has been minimized.
Make macOS dist build configuration match where reasonable try-job: dist-aarch64-apple try-job: dist-x86_64-apple
☔ The latest upstream changes (presumably #146513) made this pull request unmergeable. Please resolve the merge conflicts. |
0bcde87
to
5df0be3
Compare
This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
@bors2 try jobs=dist-aarch64-apple,dist-x86_64-apple |
This comment has been minimized.
This comment has been minimized.
Make macOS dist build configuration match where reasonable try-job: dist-aarch64-apple try-job: dist-x86_64-apple
Before: This code, after running a second time to warm the cache... 🥁 🥁 🥁 Seems reasonable to me! |
Rollup of 4 pull requests Successful merges: - #145883 (Make macOS dist build configuration match where reasonable) - #146457 (Skip cleanups on unsupported targets) - #147152 (builtin `Fn`-trait impls: instantiate binder before the return type `Sized` check) - #147153 ([rustdoc] Move doc cfg propagation pass before items stripping passes) r? `@ghost` `@rustbot` modify labels: rollup
Rollup merge of #145883 - shepmaster:unify-macos-ci, r=madsmtm Make macOS dist build configuration match where reasonable r? `@madsmtm`
r? @madsmtm