Autojump plugin #345

Merged
merged 2 commits into from May 26, 2011

5 participants

@jacobat

Hiya,

I've added a little plugin for autojump installed from homebrew. I hope you'll consider pulling it.

@robbyrussell

Could you add a little description to our Plugins page fot autojump? Thanks in advance..

@robbyrussell robbyrussell reopened this May 26, 2011
@robbyrussell robbyrussell merged commit ab2a7b5 into robbyrussell:master May 26, 2011
@sorin-ionescu sorin-ionescu commented on the diff May 30, 2011
plugins/autojump/autojump.plugin.zsh
@@ -0,0 +1,3 @@
+if [ -f `brew --prefix`/etc/autojump ]; then
@sorin-ionescu
sorin-ionescu added a line comment May 30, 2011

Not everyone uses OS X, and not everyone on OS X uses brew. Please find where autojump is installed, and source the appropriate /etc/autojump file instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@d-fens
# debian flavour
if [ -f /usr/share/autojump/autojump.zsh ]; then
  . /usr/share/autojump/autojump.zsh
fi

# mac os x flavour - not tested
if (`command -v brew >/dev/null`); then
  if [ -f `brew --prefix`/etc/autojump ]; then
    . `brew --prefix`/etc/autojump
  fi
fi

We don't all use brew, and we don't all have brew installed, we don't all have mac os x either, which causes errors when enabling.

@jacobat

Not sure if I'm supposed to take action on this? Robby?

@sorin-ionescu

But, of coarse, you are. Reopen the issue, update the pull request. :p

@a-b

Tab autocompletion doesn't works for me :(

@kalos kalos referenced this pull request Jan 3, 2012
Merged

autojump plugin enhanced #801

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment