wting / autojump

A cd command that learns - easily navigate directories from the command line
Other
16.25k stars 705 forks source link

autojump-22.2.2 on Arch Linux broken for zsh #340

Open graysky2 opened 9 years ago

graysky2 commented 9 years ago

The recent version bump breaks autojump for zsh users. Upon updating, a new shell can be opened but invoking autojump ends in the following error; the only way to recover is to downgrade to

% j
j will not execute and completion _j exists.<shell is hung here until you hit Enter>

zsh: command not found: j

Steps to reproduce:

  1. Update and open a new shell (using zsh as your default).
  2. Hit j and hit enter.

Here is how we are packaging autojump, is it correct?

pkgdir=local_for_our_package_manager
./install.py --destdir "${pkgdir}" --prefix 'usr/' --zshshare 'usr/share/zsh/site-functions'
graysky2 commented 9 years ago

I see the following two related issues: https://github.com/joelthelion/autojump/pull/339 https://github.com/joelthelion/autojump/pull/338

I tried building with the patch but ended up with the same problem as I reported above: https://github.com/youtux/autojump/commit/c29a369289e7d388af9762b7b8973aa53df821f8.patch

ryneeverett commented 9 years ago

Are you sure this issue is limited to zsh? I get "command not found" with bash too.