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

{{ common.servo_home }} seems to be wrong on Mac #772

Merged
merged 1 commit into from Jan 8, 2018
Merged

{{ common.servo_home }} seems to be wrong on Mac #772

merged 1 commit into from Jan 8, 2018

Conversation

@SimonSapin
Copy link
Member

SimonSapin commented Jan 8, 2018

servo/servo#19395 (comment)


This change is Reviewable

@SimonSapin
Copy link
Member Author

SimonSapin commented Jan 8, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Jan 8, 2018

📌 Commit 2441cf6 has been approved by SimonSapin

@highfive highfive assigned SimonSapin and unassigned aneeshusa Jan 8, 2018
@bors-servo
Copy link
Contributor

bors-servo commented Jan 8, 2018

Testing commit 2441cf6 with merge e15836f...

bors-servo added a commit that referenced this pull request Jan 8, 2018
{{ common.servo_home }} seems to be wrong on Mac

servo/servo#19395 (comment)

<!-- 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/772)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jan 8, 2018

☀️ Test successful - status-travis
Approved by: SimonSapin
Pushing e15836f to master...

@bors-servo bors-servo merged commit 2441cf6 into master Jan 8, 2018
1 of 3 checks passed
1 of 3 checks passed
continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
continuous-integration/travis-ci/push The Travis CI build could not complete due to an error
Details
homu Test successful
Details
@SimonSapin SimonSapin deleted the mac-path branch Jan 8, 2018
@jdm jdm removed the S-needs-deploy label Jan 10, 2018
@aneeshusa
Copy link
Member

aneeshusa commented Jan 28, 2018

@SimonSapin this is because the common.servo_home variable will always reflect the value for the current OS; because the Buildbot master runs on Linux, it will always return the Linux value (Salt doesn't know that you're trying to write a build config for macOS).

#327 tracks exposing all the different possible values so they can be referred to explicitly in the Buildbot configurations.

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

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