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

Fix steps reading on Windows #514

Merged
merged 1 commit into from Oct 19, 2016

Conversation

@aneeshusa
Copy link
Member

aneeshusa commented Oct 19, 2016

cat is not available, so use type which does the same thing.

r? @larsbergstrom
I spent about ~10 seconds searching for this, but it looks like it'll work...


This change is Reviewable

`cat` is not available, so use `type` which does the same thing.
@larsbergstrom
Copy link
Contributor

larsbergstrom commented Oct 19, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Oct 19, 2016

📌 Commit 7be34e6 has been approved by larsbergstrom

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Oct 19, 2016

Cool, I'll roll it out and push to the servers and retry something after this. No queue to speak of right now, so not creating a huge backlog yet :-)

@bors-servo
Copy link
Contributor

bors-servo commented Oct 19, 2016

Testing commit 7be34e6 with merge be66cf9...

bors-servo added a commit that referenced this pull request Oct 19, 2016
…rgstrom

Fix steps reading on Windows

`cat` is not available, so use `type` which does the same thing.

r? @larsbergstrom
I spent about ~10 seconds searching for this, but it looks like it'll work...

<!-- 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/514)
<!-- Reviewable:end -->
@larsbergstrom
Copy link
Contributor

larsbergstrom commented Oct 19, 2016

This passed Travis. Merging so I can roll it out.

@larsbergstrom larsbergstrom merged commit 14b6697 into servo:master Oct 19, 2016
1 of 2 checks passed
1 of 2 checks passed
homu Testing commit 7be34e6 with merge be66cf9...
Details
continuous-integration/travis-ci/pr The Travis CI build passed
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

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