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

Commit

Permalink
zsh: remove caveats for system file modifications.
Browse files Browse the repository at this point in the history
  • Loading branch information
@MikeMcQuaid
MikeMcQuaid committed Jan 21, 2014
1 parent 4f990ea commit 0e8bc06
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions Library/Formula/zsh.rb
View file
Original file line number Diff line number Diff line change
Expand Up @@ -46,16 +46,6 @@ def test
end

def caveats; <<-EOS.undent
To use this build of Zsh as your login shell, add it to /etc/shells.
If you have administrator privileges, you must fix an Apple miss
configuration in Mac OS X 10.7 Lion by renaming /etc/zshenv to
/etc/zprofile, or Zsh will have the wrong PATH when executed
non-interactively by scripts.
Alternatively, install Zsh with /etc disabled:
brew install --disable-etcdir zsh
Add the following to your zshrc to access the online help:
unalias run-help
autoload run-help
Expand Down

4 comments on commit 0e8bc06

@mislav
Copy link
Contributor

@mislav mislav commented on 0e8bc06 Oct 4, 2015

Choose a reason for hiding this comment

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

@MikeMcQuaid Why was this removed? In pre-El Capitan OS X verisons, this caveat was still valid. Was there a conversation around this removal somewhere?

@MikeMcQuaid
Copy link
Member Author

Choose a reason for hiding this comment

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

@mislav We don't like to tell people to modify system files like this as it's too easy for users to make mistakes and accidentally hose their systems. If you know enough to make these changes then you don't need Homebrew to tell you what to do.

@mislav
Copy link
Contributor

@mislav mislav commented on 0e8bc06 Oct 13, 2015

Choose a reason for hiding this comment

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

That makes sense. It would be nice if rationale was included in the commit message. Thanks!

@MikeMcQuaid
Copy link
Member Author

Choose a reason for hiding this comment

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

@mislav I agree, it's not a good commit message.

Please sign in to comment.