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

Improved readability of ensure_bootstrapped #10620

Merged
merged 1 commit into from Apr 17, 2016

Conversation

@autrilla
Copy link
Contributor

autrilla commented Apr 14, 2016

This change is Reviewable

@highfive
Copy link

highfive commented Apr 14, 2016

Heads up! This PR modifies the following files:

@Ms2ger
Copy link
Contributor

Ms2ger commented Apr 15, 2016

Where did the system-rust support go?

@autrilla autrilla force-pushed the autrilla:mach-command-improvements branch from 6dfb386 to 8005808 Apr 15, 2016
@autrilla
Copy link
Contributor Author

autrilla commented Apr 15, 2016

@Ms2ger I must have removed that by accident. It's back now.

if not self.config["tools"]["system-cargo"] and \
not path.exists(path.join(
self.config["tools"]["cargo-root"], "cargo", "bin", "cargo" + BIN_SUFFIX)):
if not self.config["tools"]["system-cargo"] and not rustc_binary_exists:

This comment has been minimized.

Copy link
@wafflespeanut

wafflespeanut Apr 16, 2016

Member

Why aren't we checking for cargo binary here?

This comment has been minimized.

Copy link
@autrilla

autrilla Apr 16, 2016

Author Contributor

Again, another mistake, sorry!

@autrilla autrilla force-pushed the autrilla:mach-command-improvements branch from 8005808 to ec58332 Apr 16, 2016
@wafflespeanut
Copy link
Member

wafflespeanut commented Apr 17, 2016

Thanks! :)

@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented Apr 17, 2016

📌 Commit ec58332 has been approved by Wafflespeanut

@bors-servo
Copy link
Contributor

bors-servo commented Apr 17, 2016

Testing commit ec58332 with merge 474507a...

bors-servo added a commit that referenced this pull request Apr 17, 2016
…eanut

Improved readability of ensure_bootstrapped

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

bors-servo commented Apr 17, 2016

💔 Test failed - mac-rel-wpt

@KiChjang
Copy link
Member

KiChjang commented Apr 17, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Apr 17, 2016

Testing commit ec58332 with merge a87fa10...

bors-servo added a commit that referenced this pull request Apr 17, 2016
…eanut

Improved readability of ensure_bootstrapped

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

bors-servo commented Apr 17, 2016

@bors-servo bors-servo merged commit ec58332 into servo:master Apr 17, 2016
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
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

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