Merge remote-tracking branch 'archwyrm/clone-trailing-slash-fix'

[code/vcsh.git] / vcsh

diff --git a/vcsh b/vcsh
index 39240093afd50dd0a47b528abb2249fedf9e1d27..b585f5c53ee2e848cfa02477072f8180ca7c4dd5 100755 (executable)
--- a/vcsh
+++ b/vcsh
@@ -18,7 +18,8 @@
 basename() {
        # Implemented in shell to avoid spawning another process
        local file
-       file="${1##*/}"
+       file="${1%/}"
+       file="${file##*/}"
        [ -z "$2" ] || file="${file%$2}"
        echo "$file"
 }
@@ -288,7 +289,7 @@ rename() {
        # Now that the repository has been renamed, we need to fix up its configuration
        # Overwrite old name..
        GIT_DIR="$GIT_DIR_NEW"
-       $VCSH_REPO_NAME="$VCSH_REPO_NAME_NEW"
+       VCSH_REPO_NAME="$VCSH_REPO_NAME_NEW"
        # ..and clobber all old configuration
        upgrade
 }
@@ -419,6 +420,7 @@ if [ "$VCSH_COMMAND" = 'clone' ]; then
        [ -z "$2" ] && fatal "$VCSH_COMMAND: please specify a remote" 1
        GIT_REMOTE="$2"
        [ -n "$3" ] && VCSH_REPO_NAME="$3" || VCSH_REPO_NAME=$(basename "$GIT_REMOTE" .git)
+       [ -z "$VCSH_REPO_NAME" ] && fatal "$VCSH_COMMAND: could not determine repository name" 1
        export VCSH_REPO_NAME
        export GIT_DIR="$VCSH_REPO_D/$VCSH_REPO_NAME.git"
 elif [ "$VCSH_COMMAND" = 'version' ]; then