Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fix ZSH error when using symlink from _SUB_ROOT/sub instead of bin/sub #24

Open
wants to merge 1 commit into from

2 participants

@devn

I confirmed that this works with both bash and zsh.

[refs #23]

@qrush
Owner

This doesn't work for me when I'm trying it in our 37 sub, which I have in ~/.37.

I'm getting:

/Users/qrush/.37/bin/37: line 13: cd: .//Users/qrush/.37/bin: No such file or directory
37: no such command `init'

Re this comment on #23:

"Nevermind. My pull request breaks sub-init. Could you please explain why abs_dirname() is the way that it is? Is there a reason some combination of dirname and basename is insufficient?"

I'm not too sure. Maybe @sstephenson knows?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Nov 6, 2012
  1. @devn
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  libexec/sub
View
2  libexec/sub
@@ -19,7 +19,7 @@ abs_dirname() {
cd "$cwd"
}
-libexec_path="$(abs_dirname "$0")"
+libexec_path="$(abs_dirname ./"$0")"
export _SUB_ROOT="$(abs_dirname "$libexec_path")"
export PATH="${libexec_path}:$PATH"
Something went wrong with that request. Please try again.