diff options
author | Conrad Irwin <conrad.irwin@gmail.com> | 2011-05-07 08:00:00 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2011-05-10 02:02:33 +0200 |
commit | e41fcfe955cb8080bf8f0e16352d3131d1f01ac8 (patch) | |
tree | ef2dc9166a47aab6a31cac04fada687cfbcc4053 | |
parent | t7501.8: feed a meaningful command (diff) | |
download | git-e41fcfe955cb8080bf8f0e16352d3131d1f01ac8.tar.xz git-e41fcfe955cb8080bf8f0e16352d3131d1f01ac8.zip |
Allow git commit --interactive with paths
Make git commit --interactive feel more like git add --interactive by
allowing the user to restrict the list of files they have to deal with.
A test in t7501 used to ensure that this is not allowed; no need for that
anymore.
Signed-off-by: Conrad Irwin <conrad.irwin@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r-- | builtin/commit.c | 2 | ||||
-rwxr-xr-x | t/t7501-commit.sh | 5 |
2 files changed, 3 insertions, 4 deletions
diff --git a/builtin/commit.c b/builtin/commit.c index 636aea6d62..7707af8844 100644 --- a/builtin/commit.c +++ b/builtin/commit.c @@ -1084,8 +1084,6 @@ static int parse_and_validate_options(int argc, const char *argv[], if (all && argc > 0) die(_("Paths with -a does not make sense.")); - else if (interactive && argc > 0) - die(_("Paths with --interactive does not make sense.")); if (null_termination && status_format == STATUS_FORMAT_LONG) status_format = STATUS_FORMAT_PORCELAIN; diff --git a/t/t7501-commit.sh b/t/t7501-commit.sh index 3d2b14d303..c2fd116d63 100755 --- a/t/t7501-commit.sh +++ b/t/t7501-commit.sh @@ -41,11 +41,12 @@ test_expect_success \ "echo King of the bongo >file && test_must_fail git commit -m foo -a file" -test_expect_success PERL 'cannot use paths with --interactive' ' +test_expect_success PERL 'can use paths with --interactive' ' echo bong-o-bong >file && # 2: update, 1:st path, that is all, 7: quit ( echo 2; echo 1; echo; echo 7 ) | - test_must_fail git commit -m foo --interactive file + git commit -m foo --interactive file && + git reset --hard HEAD^ ' test_expect_success \ |