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

The readme should include steps to install the tool #11

Closed
dkaplan-pivotal opened this issue Dec 13, 2016 · 4 comments
Closed

The readme should include steps to install the tool #11

dkaplan-pivotal opened this issue Dec 13, 2016 · 4 comments

Comments

@dkaplan-pivotal
Copy link

Not sure how to install this correctly. Wish it were in apt-get. We've been doing this:

dst="${env.HOME}/bin/mo"

if [ ! -x "\$dst" ]; then
  curl -sSo "\$dst" https://raw.githubusercontent.com/tests-always-included/mo/master/mo
  chmod +x "\$dst"
fi
@fidian
Copy link
Contributor

fidian commented Dec 14, 2016

That works well. I suppose it depends on whether you want this to be installed locally (which may also involve modifying your PATH environment variable) or globally (where you need root privileges).

I'll add some words to the README because this is a very good point and I don't have that topic covered.

@fidian fidian closed this as completed in 17f0e4b Dec 14, 2016
@fidian fidian reopened this Dec 14, 2016
@fidian
Copy link
Contributor

fidian commented Dec 14, 2016

I should have not said "this closes #11" because I would like for you to double check that the wording is sufficient and the instructions are clear. What do you think of the commit I just added?

@dkaplan-pivotal
Copy link
Author

I don't understand that last section, but that's probably my ignorance of bash scripting. The other sections make sense to me. Thanks for the quick turn around time.

@fidian
Copy link
Contributor

fidian commented Dec 14, 2016

That last bit is for others who write Bash scripts and they want to exercise the full power of mo, complete with iterating across arrays and calling functions. Not a typical use case.

I'm glad the docs work for you. Closing the issue.

@fidian fidian closed this as completed Dec 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants