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

travis: Upgrade OSX builders #48799

Merged
merged 1 commit into from
Mar 11, 2018
Merged

Conversation

alexcrichton
Copy link
Member

This upgrades the OSX builders to the xcode9.3-moar image which has 3 cores as
opposed to the 2 that our builders currently have. Should help make those OSX
builds a bit speedier!

@alexcrichton
Copy link
Member Author

r? @kennytm

@Mark-Simulacrum
Copy link
Member

Hm, I'm somewhat hesitant because of the debuginfo breakage but I also think this is somewhat more important -- breakage looks relatively minor. @bors r+

@bors
Copy link
Contributor

bors commented Mar 7, 2018

📌 Commit a3ac9f1 has been approved by Mark-Simulacrum

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Mar 7, 2018
@alexcrichton
Copy link
Member Author

@bors: r-

Looks like reproducible builds on 64-bit OSX regresses

However 32-bit builds get 40 minutes faster, so that's nice!

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Mar 7, 2018
@alexcrichton
Copy link
Member Author

@bors: r=Mark-Simulacrum

@bors
Copy link
Contributor

bors commented Mar 7, 2018

📌 Commit 51321c7 has been approved by Mark-Simulacrum

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Mar 7, 2018
@@ -1,5 +1,5 @@
-include ../tools.mk

all:
$(RUSTC) foo.rs -g
$(RUSTC) foo.rs -g -O
Copy link
Member Author

Choose a reason for hiding this comment

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

I found this test took forever without optimizations. It still takes a long time with them, but it's at least a little better

@@ -10,31 +10,27 @@ all: \

smoke:
rm -rf $(TMPDIR) && mkdir $(TMPDIR)
$(RUSTC) linker.rs -O
Copy link
Member Author

Choose a reason for hiding this comment

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

I rewrote this test to be a little more resilient against system linkers. We're sort of always assuming the system linker is deterministic, and if it's not then there's not much we can do about that...

This upgrades the OSX builders to the `xcode9.3-moar` image which has 3 cores as
opposed to the 2 that our builders currently have. Should help make those OSX
builds a bit speedier!
@alexcrichton
Copy link
Member Author

@bors: r=Mark-Simulacrum

confirmed fixed now!

@bors
Copy link
Contributor

bors commented Mar 7, 2018

📌 Commit 55a2fdf has been approved by Mark-Simulacrum

@alexcrichton
Copy link
Member Author

@bors: rollup

Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Mar 9, 2018
…-Simulacrum

travis: Upgrade OSX builders

This upgrades the OSX builders to the `xcode9.3-moar` image which has 3 cores as
opposed to the 2 that our builders currently have. Should help make those OSX
builds a bit speedier!
Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Mar 10, 2018
…-Simulacrum

travis: Upgrade OSX builders

This upgrades the OSX builders to the `xcode9.3-moar` image which has 3 cores as
opposed to the 2 that our builders currently have. Should help make those OSX
builds a bit speedier!
kennytm added a commit to kennytm/rust that referenced this pull request Mar 10, 2018
…-Simulacrum

travis: Upgrade OSX builders

This upgrades the OSX builders to the `xcode9.3-moar` image which has 3 cores as
opposed to the 2 that our builders currently have. Should help make those OSX
builds a bit speedier!
@kennytm
Copy link
Member

kennytm commented Mar 11, 2018

@bors rollup- p=1

@bors
Copy link
Contributor

bors commented Mar 11, 2018

⌛ Testing commit 55a2fdf with merge e5acb0c...

bors added a commit that referenced this pull request Mar 11, 2018
travis: Upgrade OSX builders

This upgrades the OSX builders to the `xcode9.3-moar` image which has 3 cores as
opposed to the 2 that our builders currently have. Should help make those OSX
builds a bit speedier!
@bors
Copy link
Contributor

bors commented Mar 11, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: Mark-Simulacrum
Pushing e5acb0c to master...

@bors bors merged commit 55a2fdf into rust-lang:master Mar 11, 2018
@alexcrichton alexcrichton deleted the more-osx-cores branch March 19, 2018 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants