From: Richard Hartmann Date: Mon, 9 Dec 2013 09:18:34 +0000 (+0100) Subject: Revert "Implement basename in shell" X-Git-Url: https://git.madduck.net/code/vcsh.git/commitdiff_plain/ba41fabe4e8c1ffb5b6ae064089595be64008344?ds=sidebyside;hp=-c Revert "Implement basename in shell" This reverts commit 794f0aac869dda034300211e9be45db123d84377. Conflicts: vcsh Turns out the shell implementation didn't account for trailing slashes. While, arguably, a proper interactive shell should eat them before execution anyway, this can also be called in script context. This means that vcsh clone foo/ made it fail and even after commit 7e33e38a869989d04d12f2ef160d514041eb01e3 vcsh clone foo// would have failed. Revert to safe and time-proven default: Standalone `basename`. For further information, see: https://github.com/RichiH/vcsh/pull/92 --- ba41fabe4e8c1ffb5b6ae064089595be64008344 diff --git a/vcsh b/vcsh index b585f5c..9f8e274 100755 --- a/vcsh +++ b/vcsh @@ -15,15 +15,6 @@ # This should always be the first line of code to facilitate debugging [ -n "$VCSH_DEBUG" ] && set -vx -basename() { - # Implemented in shell to avoid spawning another process - local file - file="${1%/}" - file="${file##*/}" - [ -z "$2" ] || file="${file%$2}" - echo "$file" -} - SELF=$(basename $0) VERSION='1.20130909.git-HEAD'