This repository has been archived by the owner. It is now read-only.
Permalink
Browse files

autojump 21.0.3

Update files to reflect new source locations.

Existing users upgrading must note the new autojump.sh location
and modify ~/.bash_profile or ~/.zshrc appropriately as mentioned
in the caveat message.

Fixes #15797.
Closes #15793.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
  • Loading branch information...
1 parent 1ca93be commit 5da9c4d5eb38e6abb516ba648898fb374ec360cd @wting wting committed with adamv Nov 1, 2012
Showing with 14 additions and 14 deletions.
  1. +14 −14 Library/Formula/autojump.rb
@@ -1,28 +1,28 @@
require 'formula'
class Autojump < Formula
- homepage 'https://github.com/joelthelion/autojump/wiki'
- url 'https://github.com/downloads/joelthelion/autojump/autojump_v20.tar.gz'
- sha1 '7de157feb90dc22d5959914fa531844c68d5ba7b'
+ homepage 'https://github.com/joelthelion/autojump#name'
+ url 'https://github.com/downloads/joelthelion/autojump/autojump_v21.0.3.tar.gz'
+ sha1 'b81308d6ff50680cb74795195e93427557a938ad'
head 'https://github.com/joelthelion/autojump.git'
def install
- inreplace 'autojump.sh', '/etc/profile.d/', "#{prefix}/etc/"
+ inreplace 'bin/autojump.sh', ' /etc/profile.d/', " #{prefix}/etc/"
- bin.install 'autojump'
- man1.install 'autojump.1'
- (prefix+'etc').install 'autojump.sh' => 'autojump'
- (prefix+'etc').install 'autojump.bash', 'autojump.zsh'
- (share+'zsh/site-functions').install '_j'
+ bin.install 'bin/autojump'
+ man1.install 'docs/autojump.1'
+ (prefix+'etc').install 'bin/autojump.sh', 'bin/autojump.bash', 'bin/autojump.zsh'
+ (share+'zsh/site-functions').install 'bin/_j'
end
def caveats; <<-EOS.undent
- Add the following lines to your ~/.bash_profile or ~/.zshrc file (and
- remember to source the file to update your current session):
- if [ -f `brew --prefix`/etc/autojump ]; then
- . `brew --prefix`/etc/autojump
- fi
+
+ Add the following line to your ~/.bash_profile or ~/.zshrc file (and remember
+ to source the file to update your current session):
+
+ [[ -f `brew --prefix`/etc/autojump.sh ]] && . `brew --prefix`/etc/autojump.sh
+
EOS
end
end

3 comments on commit 5da9c4d

Member

mxcl replied Nov 2, 2012

Why did we move this? I have people angril at me from dotfiles commits.

Owner

MikeMcQuaid replied Nov 2, 2012

I think we need something stronger than a caveat for "requires user interaction" messages. I'd go as far as saying a prompt to make sure they've read it (I never read caveats). At the very least a symlink from the old location would be nice.

Member

mxcl replied Nov 2, 2012

The problem here is:

  1. we removed autojump for bad reasons
  2. we put it back
  3. we moved the script to autojump from autojump.bash
  4. we moved the script from autojump to autojump.sh

People think we're moronic and I agree.

Please sign in to comment.