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

Re provide homebrew upload token #533

Merged
merged 2 commits into from Oct 31, 2016

Conversation

@aneeshusa
Copy link
Member

aneeshusa commented Oct 31, 2016

This was added to the previous make_step implementation,
but apparently was not copied over to this implementation,
causing our Homebrew uploads to fail for the last 4 days.
Re-add this logic.

Note that because adding the token to the environment and telling
Buildbot not to log the environment are configured together,
the token was not leaked during this time.

r? @larsbergstrom
Fixes #532.


This change is Reviewable

aneeshusa added 2 commits Oct 31, 2016
This style avoid spurious line diff noise, since all indents are
multiples of 4.
This was added to the previous `make_step` implementation,
but apparently was not copied over to this implementation,
causing our Homebrew uploads to fail for the last 4 days.
Re-add this logic.

Note that because adding the token to the environment and telling
Buildbot not to log the environment are configured together,
the token was not leaked during this time.
@aneeshusa
Copy link
Member Author

aneeshusa commented Oct 31, 2016

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Oct 31, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Oct 31, 2016

📌 Commit 37f946f has been approved by larsbergstrom

@bors-servo
Copy link
Contributor

bors-servo commented Oct 31, 2016

Test exempted - status

@bors-servo bors-servo merged commit 37f946f into servo:master Oct 31, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test exempted
Details
bors-servo added a commit that referenced this pull request Oct 31, 2016
…rsbergstrom

Re provide homebrew upload token

 This was added to the previous `make_step` implementation,
but apparently was not copied over to this implementation,
causing our Homebrew uploads to fail for the last 4 days.
Re-add this logic.

Note that because adding the token to the environment and telling
Buildbot not to log the environment are configured together,
the token was not leaked during this time.

r? @larsbergstrom
Fixes #532.

<!-- 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/533)
<!-- Reviewable:end -->
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.