git-reset: refuse to do hard reset in a bare repository
authorJeff King <peff@peff.net>
Mon, 31 Dec 2007 07:13:52 +0000 (02:13 -0500)
committerJunio C Hamano <gitster@pobox.com>
Wed, 2 Jan 2008 10:28:54 +0000 (02:28 -0800)
It makes no sense since there is no working tree. A soft
reset should be fine, though.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin-reset.c
t/t7103-reset-bare.sh [new file with mode: 0755]

index 713c2d5..10dba60 100644 (file)
@@ -244,6 +244,9 @@ int cmd_reset(int argc, const char **argv, const char *prefix)
        if (reset_type == NONE)
                reset_type = MIXED; /* by default */
 
        if (reset_type == NONE)
                reset_type = MIXED; /* by default */
 
+       if (reset_type == HARD && is_bare_repository())
+               die("hard reset makes no sense in a bare repository");
+
        /* Soft reset does not touch the index file nor the working tree
         * at all, but requires them in a good order.  Other resets reset
         * the index file to the tree object we are switching to. */
        /* Soft reset does not touch the index file nor the working tree
         * at all, but requires them in a good order.  Other resets reset
         * the index file to the tree object we are switching to. */
diff --git a/t/t7103-reset-bare.sh b/t/t7103-reset-bare.sh
new file mode 100755 (executable)
index 0000000..b25a77f
--- /dev/null
@@ -0,0 +1,28 @@
+#!/bin/sh
+
+test_description='git-reset in a bare repository'
+. ./test-lib.sh
+
+test_expect_success 'setup non-bare' '
+       echo one >file &&
+       git add file &&
+       git commit -m one &&
+       echo two >file &&
+       git commit -a -m two
+'
+
+test_expect_success 'setup bare' '
+       git clone --bare . bare.git &&
+       cd bare.git
+'
+
+test_expect_success 'hard reset is not allowed' '
+       ! git reset --hard HEAD^
+'
+
+test_expect_success 'soft reset is allowed' '
+       git reset --soft HEAD^ &&
+       test "`git show --pretty=format:%s | head -n 1`" = "one"
+'
+
+test_done