Show what would be done and the user must confirm before actually
cleaning.

    Would remove ...
    Would remove ...
    Would remove ...

    Remove [y/n]?

Press "y" to start cleaning, and press "n" if you want to abort.

Signed-off-by: Jiang Xin <worldhello....@gmail.com>
---
 Documentation/git-clean.txt | 10 ++++++--
 builtin/clean.c             | 61 +++++++++++++++++++++++++++++++++++++++++----
 2 files changed, 64 insertions(+), 7 deletions(-)

diff --git a/Documentation/git-clean.txt b/Documentation/git-clean.txt
index bdc3a..186e34 100644
--- a/Documentation/git-clean.txt
+++ b/Documentation/git-clean.txt
@@ -8,7 +8,7 @@ git-clean - Remove untracked files from the working tree
 SYNOPSIS
 --------
 [verse]
-'git clean' [-d] [-f] [-n] [-q] [-e <pattern>] [-x | -X] [--] <path>...
+'git clean' [-d] [-f] [-i] [-n] [-q] [-e <pattern>] [-x | -X] [--] <path>...
 
 DESCRIPTION
 -----------
@@ -34,7 +34,13 @@ OPTIONS
 -f::
 --force::
        If the Git configuration variable clean.requireForce is not set
-       to false, 'git clean' will refuse to run unless given -f or -n.
+       to false, 'git clean' will refuse to run unless given -f, -n or
+       -i.
+
+-i::
+--interactive::
+       Show what would be done and the user must confirm before actually
+       cleaning.
 
 -n::
 --dry-run::
diff --git a/builtin/clean.c b/builtin/clean.c
index ccd4..127463 100644
--- a/builtin/clean.c
+++ b/builtin/clean.c
@@ -15,10 +15,11 @@
 #include "quote.h"
 
 static int force = -1; /* unset */
+static int interactive;
 static struct string_list del_list = STRING_LIST_INIT_DUP;
 
 static const char *const builtin_clean_usage[] = {
-       N_("git clean [-d] [-f] [-n] [-q] [-e <pattern>] [-x | -X] [--] 
<paths>..."),
+       N_("git clean [-d] [-f] [-i] [-n] [-q] [-e <pattern>] [-x | -X] [--] 
<paths>..."),
        NULL
 };
 
@@ -185,6 +186,50 @@ static const char *path_relative(const char *in, const 
char *prefix)
        return buf;
 }
 
+static void interactive_main_loop(void)
+{
+       struct strbuf confirm = STRBUF_INIT;
+       struct strbuf buf = STRBUF_INIT;
+       struct string_list_item *item;
+       const char *qname;
+
+       while (del_list.nr) {
+               putchar('\n');
+               for_each_string_list_item(item, &del_list) {
+                       qname = quote_path_relative(item->string, -1, &buf, 
NULL);
+                       printf(_(msg_would_remove), qname);
+               }
+               putchar('\n');
+
+               printf(_("Remove [y/n]? "));
+               if (strbuf_getline(&confirm, stdin, '\n') != EOF) {
+                       strbuf_trim(&confirm);
+               } else {
+                       /* Ctrl-D is the same as "quit" */
+                       string_list_clear(&del_list, 0);
+                       putchar('\n');
+                       printf_ln("Bye.");
+                       break;
+               }
+
+               if (confirm.len) {
+                       if (!strncasecmp(confirm.buf, "yes", confirm.len)) {
+                               break;
+                       } else if (!strncasecmp(confirm.buf, "no", confirm.len) 
||
+                                  !strncasecmp(confirm.buf, "quit", 
confirm.len)) {
+                               string_list_clear(&del_list, 0);
+                               printf_ln("Bye.");
+                               break;
+                       } else {
+                               continue;
+                       }
+               }
+       }
+
+       strbuf_release(&buf);
+       strbuf_release(&confirm);
+}
+
 int cmd_clean(int argc, const char **argv, const char *prefix)
 {
        int i, res;
@@ -204,6 +249,7 @@ int cmd_clean(int argc, const char **argv, const char 
*prefix)
                OPT__QUIET(&quiet, N_("do not print names of files removed")),
                OPT__DRY_RUN(&dry_run, N_("dry run")),
                OPT__FORCE(&force, N_("force")),
+               OPT_BOOL('i', "interactive", &interactive, N_("interactive 
cleaning")),
                OPT_BOOLEAN('d', NULL, &remove_directories,
                                N_("remove whole directories")),
                { OPTION_CALLBACK, 'e', "exclude", &exclude_list, N_("pattern"),
@@ -230,12 +276,16 @@ int cmd_clean(int argc, const char **argv, const char 
*prefix)
        if (ignored && ignored_only)
                die(_("-x and -X cannot be used together"));
 
-       if (!dry_run && !force) {
+       if (interactive) {
+               if (!isatty(0) || !isatty(1))
+                       die(_("interactive clean can not run without a valid 
tty; "
+                                 "refusing to clean"));
+       } else if (!dry_run && !force) {
                if (config_set)
-                       die(_("clean.requireForce set to true and neither -n 
nor -f given; "
+                       die(_("clean.requireForce set to true and neither -i, 
-n nor -f given; "
                                  "refusing to clean"));
                else
-                       die(_("clean.requireForce defaults to true and neither 
-n nor -f given; "
+                       die(_("clean.requireForce defaults to true and neither 
-i, -n nor -f given; "
                                  "refusing to clean"));
        }
 
@@ -309,7 +359,8 @@ int cmd_clean(int argc, const char **argv, const char 
*prefix)
                }
        }
 
-       /* TODO: do interactive git-clean here, which will modify del_list */
+       if (interactive && del_list.nr > 0)
+               interactive_main_loop();
 
        for_each_string_list_item(item, &del_list) {
                struct stat st;
-- 
1.8.3.rc1.404.gb9fcf3e

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to