thoughtbot / dotfiles

A set of vim, zsh, git, and tmux configuration files.
https://thoughtbot.com
Other
7.97k stars 1.87k forks source link

Add binstubs to PATH #109

Closed croaky closed 10 years ago

croaky commented 11 years ago

Necessary for use with rbenv. https://github.com/thoughtbot/laptop/commit/3aeb5b59cc5fe4e56a75781cad964cf3f0c6368d

Proposed solution:

export PATH=./bin/stubs:$PATH
joshuaclayton commented 11 years ago

Not sure if I'm just being paranoid but this could potentially open up some nasty issues. I've been using @tpope's suggestion outlined in a tweet from Feb 2012 (https://twitter.com/tpope/status/165631968996900865, corresponding code in my dotfiles) to add bin/stubs relative to a .git/safe directory. While this requires you to mkdir .git/safe in projects you work on, I prefer feeling better about my $PATH and spending the additional one or two seconds to authorize binstubs per project.

jferris commented 11 years ago

I think that kind of paranoia is completely valid in a server setup, but I've never felt the need to be so protective of my development machine. This is one of those cases where I'd prefer convenience over safety given the relatively low risk involved.

croaky commented 10 years ago

I believe this is is superseded by https://github.com/thoughtbot/dotfiles/pull/191