652037ebdfbe581e4c7dfcd0862ac2d1577c86d9

Author
ditzy <thesourceofx@gmail.com>
Committer
GitHub <noreply@github.com>
Date

Message

fix(jump)!: only resolve mark symlink (#11074)

BREAKING CHANGE: Previously, `jump` would resolve the symlink to the mark directory and resolve again if the mark directory was also a symlink. It only resolves once after this commit.

Diff

 1diff --git a/plugins/jump/jump.plugin.zsh b/plugins/jump/jump.plugin.zsh
 2index 829c9d9cbf2908f6b19e33802370da2a03f92985..c2b21e92e7f0f651d56f0c00544e75a71d2c76af 100644
 3--- a/plugins/jump/jump.plugin.zsh
 4+++ b/plugins/jump/jump.plugin.zsh
 5@@ -8,8 +8,10 @@
 6 #
 7 export MARKPATH=$HOME/.marks
 8 
 9+
10 jump() {
11-	builtin cd -P "$MARKPATH/$1" 2>/dev/null || {echo "No such mark: $1"; return 1}
12+	local markpath="$(readlink $MARKPATH/$1)" || {echo "No such mark: $1"; return 1}
13+	builtin cd "$markpath" 2>/dev/null || {echo "Destination does not exist for mark [$1]: $markpath"; return 2}
14 }
15 
16 mark() {