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

Update sccache to 0.2.7 #911

Merged
merged 1 commit into from Nov 8, 2018
Merged

Update sccache to 0.2.7 #911

merged 1 commit into from Nov 8, 2018

Conversation

@SimonSapin
Copy link
Member

SimonSapin commented Nov 6, 2018

This hopefully works around rust-lang/cargo#6269


This change is Reviewable

@SimonSapin SimonSapin force-pushed the sccacheup branch from b6477f8 to 1b38c95 Nov 6, 2018
@SimonSapin
Copy link
Member Author

SimonSapin commented Nov 6, 2018

upload: ./0.2.7-sccache-x86_64-apple-darwin to s3://servo-deps/sccache/0.2.7-sccache-x86_64-apple-darwin
upload: ./0.2.7-sccache-x86_64-pc-windows-msvc to s3://servo-deps/sccache/0.2.7-sccache-x86_64-pc-windows-msvc
upload: ./0.2.7-sccache-x86_64-unknown-linux-musl to s3://servo-deps/sccache/0.2.7-sccache-x86_64-unknown-linux-musl
@jdm
Copy link
Member

jdm commented Nov 8, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Nov 8, 2018

📌 Commit 1b38c95 has been approved by jdm

bors-servo added a commit that referenced this pull request Nov 8, 2018
Update sccache to 0.2.7

This hopefully works around rust-lang/cargo#6269

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

bors-servo commented Nov 8, 2018

Testing commit 1b38c95 with merge 8ffeb4b...

@bors-servo
Copy link
Contributor

bors-servo commented Nov 8, 2018

💔 Test failed - status-travis

@jdm
Copy link
Member

jdm commented Nov 8, 2018

Looks like you need to make the new files public:

----------
          ID: sccache
    Function: file.managed
        Name: /usr/local/bin/sccache
      Result: False
     Comment: Failed to cache https://servo-deps.s3.amazonaws.com/sccache/0.2.7-sccache-x86_64-unknown-linux-musl: Error: HTTP 403: Forbidden reading https://servo-deps.s3.amazonaws.com/sccache/0.2.7-sccache-x86_64-unknown-linux-musl
     Started: 15:14:32.439584
    Duration: 179.893 ms
     Changes:   
@jdm
Copy link
Member

jdm commented Nov 8, 2018

@bors-servo retry
Did that through the s3 console.

@jdm
Copy link
Member

jdm commented Nov 8, 2018

@jdm
Copy link
Member

jdm commented Nov 8, 2018

@bors-servo retry

@jdm
Copy link
Member

jdm commented Nov 8, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Nov 8, 2018

📌 Commit 1b38c95 has been approved by jdm

@jdm
Copy link
Member

jdm commented Nov 8, 2018

@jdm
Copy link
Member

jdm commented Nov 8, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Nov 8, 2018

📌 Commit 1b38c95 has been approved by jdm

@jdm
Copy link
Member

jdm commented Nov 8, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Nov 8, 2018

Testing commit 1b38c95 with merge 20483ac...

bors-servo added a commit that referenced this pull request Nov 8, 2018
Update sccache to 0.2.7

This hopefully works around rust-lang/cargo#6269

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

bors-servo commented Nov 8, 2018

☀️ Test successful - status-travis
Approved by: jdm
Pushing 20483ac to master...

@bors-servo bors-servo merged commit 1b38c95 into master Nov 8, 2018
1 of 3 checks passed
1 of 3 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/travis-ci/push The Travis CI build failed
Details
homu Test successful
Details
@jdm jdm removed the S-needs-deploy label Nov 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.