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

First cut of Linux install intstuctions #16

Merged
merged 1 commit into from
May 13, 2018

Conversation

chastell
Copy link
Contributor

This includes the first cut of Linux installation instructions (#6).

I have no clue how to explain to Twee that the [[ in linux/path.tw2 is a Bash command and not a Twee link. 😢

Cheers from ROSS conf! 🎉

Copy link
Member

@kytrinyx kytrinyx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks excellent!

I have a couple of tiny nitpicks. Also, would you mind rebasing this onto master? I merged a PR that gets rid of the compiled/generated files so that we get fewer conflicts on merge.

::Linux Introduction
# Installing Exercism on Linux

To install Exorcism on Linux you need to get the `exorcism`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/Exorcism/Exercism/ :)

also s/exorcism/exercism/

(twice on the same line, I think you win!)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ooops. 😊

* If you’re using the command line (use the
right archive file name for your architecture):
```
tar -xf exercism-linux-64bit.tgz
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think tar takes a - before the options -- did you run this and copy/paste it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It’s… complicated, as tar supports old, short and long options. :)

All of those will work (and do the same thing):

tar xf exercism-linux-64bit.tgz
tar -xf exercism-linux-64bit.tgz
tar -x -f exercism-linux-64bit.tgz
tar --extract --file exercism-linux-64bit.tgz

I’m not a huge fan of the old (tar xf) options style as it’s not symmetric with any other tool, but I totally don’t mind it either, and any of the above would work for me, so just state your preference, please. :)

(I prefer the long style in scripts, as it makes them more readable, but in case of tutorials some people dutifully type the commands rather than copy-pasting them, and the longer the options the higher the chance for a typo…)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the -xf works, then I prefer that. I had just never seen it before.

linux/directory.tw2
linux/introduction.tw2
linux/path.tw2
linux/tarball.tw2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for alphabetizing these!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Of course! I’m not a monster!

@kytrinyx kytrinyx merged commit e242abf into exercism:master May 13, 2018
@kytrinyx
Copy link
Member

Thank you!

@chastell chastell deleted the linux_install branch May 14, 2018 20:16
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

Successfully merging this pull request may close these issues.

2 participants