Skip to content

/ servo

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

Enable sccache on macOS. #22160

Closed
wants to merge 1 commit into from
Closed

Enable sccache on macOS. #22160

wants to merge 1 commit into from

Conversation

@jdm
Copy link
Member

jdm commented Nov 9, 2018 •
edited by SimonSapin

Perhaps this will improve the long build times on the macOS build machines.

This change is Reviewable
@jdm
Enable sccache on macOS.
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
7679d73
@highfive
Copy link

highfive commented Nov 9, 2018

Heads up! This PR modifies the following files:
@jdm
Copy link
Member Author

jdm commented Nov 9, 2018

@bors-servo try=mac
@bors-servo
Copy link
Contributor

bors-servo commented Nov 9, 2018

⌛ Trying commit 7679d73 with merge de99709...
bors-servo added a commit that referenced this pull request Nov 9, 2018
@bors-servo
Auto merge of #22160 - servo:jdm-patch-17, r=<try>
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
de99709 
Enable sccache on macOS.

Perhaps this will improve the long build times on the macOS build machines.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/22160)
<!-- Reviewable:end -->
@jdm
Copy link
Member Author

jdm commented Nov 9, 2018

@bors-servo try-
@jdm
Copy link
Member Author

jdm commented Nov 9, 2018 

Caused by:
  process didn't exit successfully: `sccache rustc - --crate-name ___ --print=file-names -Dwarnings -W unused-extern-crates --crate-type bin --crate-type rlib --crate-type dylib --crate-type cdylib --crate-type staticlib --crate-type proc-macro` (signal: 5, SIGTRAP: trace/breakpoint trap)
--- stderr
dyld: Library not loaded: /usr/local/opt/openssl@1.1/lib/libssl.1.1.dylib
  Referenced from: /usr/local/bin/sccache
  Reason: image not found

Looks like we either need to change something about the openssl installation on the build machines or just resign ourselves to building sccache from source on mac and deploying those binaries instead.
@jdm
Copy link
Member Author

jdm commented Nov 9, 2018

@bors-servo r-
@jdm jdm closed this Nov 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@asajeffrey asajeffrey

Labels
S-awaiting-review
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
@jdm @highfive @bors-servo @asajeffrey
You can’t perform that action at this time.