Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

Commit

Permalink
Don't provide destructive instructions
Browse files Browse the repository at this point in the history
If /usr/local doesn't exist you've just sudo chowned $PWD
  • Loading branch information
mxcl committed Mar 14, 2010
1 parent 3c77c3f commit 3684fae
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,16 +10,15 @@ Run <http://gist.github.com/323731> and afterwards, [install Xcode][xcode].

Umm… I thought I could install it anywhere?
-------------------------------------------
Indeed, you can. Please read our [complete installation instructions][install].
Indeed, you can. Refer to our [complete installation instructions][install].


Dude! Just give me a three-liner!
--------------------------------------
Dude! Just give me a two-liner!
-------------------------------
Alright then… as long as you know what you are doing:

cd /usr/local
sudo chown -R $USER .
curl -Lsf http://github.com/mxcl/homebrew/tarball/master | tar xz --strip 1
sudo chown -R $USER /usr/local
curl -Lsf http://github.com/mxcl/homebrew/tarball/master | tar xvz -C/usr/local --strip 1


More Documentation
Expand Down

3 comments on commit 3684fae

@roja
Copy link
Contributor

@roja roja commented on 3684fae Jun 14, 2010

Choose a reason for hiding this comment

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

I think the one-liner should well be changed to:

ruby -e "$(curl http://gist.github.com/raw/323731/install_homebrew.rb)"

@godfat
Copy link
Contributor

@godfat godfat commented on 3684fae Jun 14, 2010

Choose a reason for hiding this comment

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

If that's the case, I propose to use below instead because $() didn't work on my shell (fish shell)
curl http://gist.github.com/raw/323731/install_homebrew.rb | ruby -e 'eval($stdin.read)'

@mkhl
Copy link
Contributor

@mkhl mkhl commented on 3684fae Jun 14, 2010

Choose a reason for hiding this comment

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

…which should be identical to:
curl http://gist.github.com/raw/323731/install_homebrew.rb | ruby

(Also, if you changed your shell to fish, you should be aware that is spells $() as ().)

Please sign in to comment.