Teach fmt-patch about --keep-subject

[git.git] / builtin-log.c

diff --git a/builtin-log.c b/builtin-log.c
index 1649f49..0027998 100644 (file)
--- a/builtin-log.c
+++ b/builtin-log.c
@@ -76,15 +76,22 @@ static int istitlechar(char c)
 }
 
 static FILE *realstdout = NULL;
+static char *output_directory = NULL;
 
-static void reopen_stdout(struct commit *commit, int nr)
+static void reopen_stdout(struct commit *commit, int nr, int keep_subject)
 {
        char filename[1024];
        char *sol;
-       int len;
+       int len = 0;
 
+       if (output_directory) {
+               strncpy(filename, output_directory, 1010);
+               len = strlen(filename);
+               if (filename[len - 1] != '/')
+                       filename[len++] = '/';
+       }
 
-       sprintf(filename, "%04d", nr);
+       sprintf(filename + len, "%04d", nr);
        len = strlen(filename);
 
        sol = strstr(commit->buffer, "\n\n");
@@ -93,7 +100,7 @@ static void reopen_stdout(struct commit *commit, int nr)
 
                sol += 2;
                /* strip [PATCH] or [PATCH blabla] */
-               if (!strncmp(sol, "[PATCH", 6)) {
+               if (!keep_subject && !strncmp(sol, "[PATCH", 6)) {
                        char *eos = strchr(sol + 6, ']');
                        if (eos) {
                                while (isspace(*eos))
@@ -128,8 +135,10 @@ int cmd_format_patch(int argc, const char **argv, char **envp)
        struct commit *commit;
        struct commit **list = NULL;
        struct rev_info rev;
-       int nr = 0, total;
+       int nr = 0, total, i, j;
        int use_stdout = 0;
+       int numbered = 0;
+       int keep_subject = 0;
 
        init_revisions(&rev);
        rev.commit_format = CMIT_FMT_EMAIL;
@@ -140,16 +149,41 @@ int cmd_format_patch(int argc, const char **argv, char **envp)
        rev.combine_merges = 0;
        rev.ignore_merges = 1;
        rev.diffopt.output_format = DIFF_FORMAT_PATCH;
-       argc = setup_revisions(argc, argv, &rev, "HEAD");
 
-       while (argc > 1) {
-               if (!strcmp(argv[1], "--stdout"))
+       /*
+        * Parse the arguments before setup_revisions(), or something
+        * like "git fmt-patch -o a123 HEAD^.." may fail; a123 is
+        * possibly a valid SHA1.
+        */
+       for (i = 1, j = 1; i < argc; i++) {
+               if (!strcmp(argv[i], "--stdout"))
                        use_stdout = 1;
-               else
-                       die ("unrecognized argument: %s", argv[1]);
-               argc--;
-               argv++;
+               else if (!strcmp(argv[i], "-n") ||
+                               !strcmp(argv[i], "--numbered"))
+                       numbered = 1;
+               else if (!strcmp(argv[i], "-k") ||
+                               !strcmp(argv[i], "--keep-subject")) {
+                       keep_subject = 1;
+                       rev.total = -1;
+               } else if (!strcmp(argv[i], "-o")) {
+                       if (argc < 3)
+                               die ("Which directory?");
+                       if (mkdir(argv[i + 1], 0777) < 0 && errno != EEXIST)
+                               die("Could not create directory %s",
+                                               argv[i + 1]);
+                       output_directory = strdup(argv[i + 1]);
+                       i++;
+               } else
+                       argv[j++] = argv[i];
        }
+       argc = j;
+
+       if (numbered && keep_subject < 0)
+               die ("-n and -k are mutually exclusive.");
+
+       argc = setup_revisions(argc, argv, &rev, "HEAD");
+       if (argc > 1)
+               die ("unrecognized argument: %s", argv[1]);
 
        if (!use_stdout)
                realstdout = fdopen(dup(1), "w");
@@ -164,11 +198,14 @@ int cmd_format_patch(int argc, const char **argv, char **envp)
                list[nr - 1] = commit;
        }
        total = nr;
+       if (numbered)
+               rev.total = total;
        while (0 <= --nr) {
                int shown;
                commit = list[nr];
+               rev.nr = total - nr;
                if (!use_stdout)
-                       reopen_stdout(commit, total - nr);
+                       reopen_stdout(commit, rev.nr, keep_subject);
                shown = log_tree_commit(&rev, commit);
                free(commit->buffer);
                commit->buffer = NULL;
@@ -177,6 +214,8 @@ int cmd_format_patch(int argc, const char **argv, char **envp)
                if (!use_stdout)
                        fclose(stdout);
        }
+       if (output_directory)
+               free(output_directory);
        free(list);
        return 0;
 }