From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christian Couder Subject: [PATCH v2 4/5] reset: disallow "reset --keep" outside a work tree Date: Tue, 19 Jan 2010 05:26:00 +0100 Message-ID: <20100119042602.4510.2378.chriscool@tuxfamily.org> References: <20100119042404.4510.48855.chriscool@tuxfamily.org> Cc: git@vger.kernel.org, Linus Torvalds , Johannes Schindelin , Stephan Beyer , Daniel Barkalow , Jakub Narebski , Paolo Bonzini , Johannes Sixt , Stephen Boyd , Andreas Schwab , Daniel Convissor To: Junio C Hamano X-From: git-owner@vger.kernel.org Tue Jan 19 05:24:57 2010 Return-path: Envelope-to: gcvg-git-2@lo.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by lo.gmane.org with esmtp (Exim 4.50) id 1NX5eC-00074w-F3 for gcvg-git-2@lo.gmane.org; Tue, 19 Jan 2010 05:24:56 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755043Ab0ASEYe (ORCPT ); Mon, 18 Jan 2010 23:24:34 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754984Ab0ASEYb (ORCPT ); Mon, 18 Jan 2010 23:24:31 -0500 Received: from smtp3-g21.free.fr ([212.27.42.3]:53664 "EHLO smtp3-g21.free.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754784Ab0ASEYU (ORCPT ); Mon, 18 Jan 2010 23:24:20 -0500 Received: from smtp3-g21.free.fr (localhost [127.0.0.1]) by smtp3-g21.free.fr (Postfix) with ESMTP id C365F818062; Tue, 19 Jan 2010 05:24:00 +0100 (CET) Received: from bureau.boubyland (gre92-7-82-243-130-161.fbx.proxad.net [82.243.130.161]) by smtp3-g21.free.fr (Postfix) with ESMTP id 65FE681806A; Tue, 19 Jan 2010 05:23:57 +0100 (CET) X-git-sha1: 4cdac672e7feb8fd4c11e0268736f616e1041380 X-Mailer: git-mail-commits v0.5.2 In-Reply-To: <20100119042404.4510.48855.chriscool@tuxfamily.org> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: It is safer and consistent with "--merge" and "--hard" resets to disallow "git reset --keep" outside a work tree. So let's just do that and add some tests while at it. Signed-off-by: Christian Couder --- builtin-reset.c | 2 +- t/t7103-reset-bare.sh | 25 +++++++++++++++++-------- 2 files changed, 18 insertions(+), 9 deletions(-) diff --git a/builtin-reset.c b/builtin-reset.c index da61f20..52584af 100644 --- a/builtin-reset.c +++ b/builtin-reset.c @@ -319,7 +319,7 @@ int cmd_reset(int argc, const char **argv, const char *prefix) if (reset_type == NONE) reset_type = MIXED; /* by default */ - if (reset_type == HARD || reset_type == MERGE) + if (reset_type != SOFT && reset_type != MIXED) setup_work_tree(); if (reset_type == MIXED && is_bare_repository()) diff --git a/t/t7103-reset-bare.sh b/t/t7103-reset-bare.sh index afb55b3..1eef93c 100755 --- a/t/t7103-reset-bare.sh +++ b/t/t7103-reset-bare.sh @@ -11,21 +11,26 @@ test_expect_success 'setup non-bare' ' git commit -a -m two ' -test_expect_success 'hard reset requires a worktree' ' +test_expect_success '"hard" reset requires a worktree' ' (cd .git && test_must_fail git reset --hard) ' -test_expect_success 'merge reset requires a worktree' ' +test_expect_success '"merge" reset requires a worktree' ' (cd .git && test_must_fail git reset --merge) ' -test_expect_success 'mixed reset is ok' ' +test_expect_success '"keep" reset requires a worktree' ' + (cd .git && + test_must_fail git reset --keep) +' + +test_expect_success '"mixed" reset is ok' ' (cd .git && git reset) ' -test_expect_success 'soft reset is ok' ' +test_expect_success '"soft" reset is ok' ' (cd .git && git reset --soft) ' @@ -40,19 +45,23 @@ test_expect_success 'setup bare' ' cd bare.git ' -test_expect_success 'hard reset is not allowed in bare' ' +test_expect_success '"hard" reset is not allowed in bare' ' test_must_fail git reset --hard HEAD^ ' -test_expect_success 'merge reset is not allowed in bare' ' +test_expect_success '"merge" reset is not allowed in bare' ' test_must_fail git reset --merge HEAD^ ' -test_expect_success 'mixed reset is not allowed in bare' ' +test_expect_success '"keep" reset is not allowed in bare' ' + test_must_fail git reset --keep HEAD^ +' + +test_expect_success '"mixed" reset is not allowed in bare' ' test_must_fail git reset --mixed HEAD^ ' -test_expect_success 'soft reset is allowed in bare' ' +test_expect_success '"soft" reset is allowed in bare' ' git reset --soft HEAD^ && test "`git show --pretty=format:%s | head -n 1`" = "one" ' -- 1.6.6.271.gc8799