Hi, On Wed, 17 Jan 2007, Josh Boyer wrote: > Could we add an option to git-format-patch to use ".patch" as the file > suffix instead of ".txt"? Something like the below? > > diff --git a/builtin-log.c b/builtin-log.c > index a59b4ac..4eb2d32 100644 > --- a/builtin-log.c > +++ b/builtin-log.c > @@ -217,6 +217,7 @@ static int git_format_config(const char *var, > const char *value) > > static FILE *realstdout = NULL; > static const char *output_directory = NULL; > +static int psuffix = 0; Why not static const char *file_extension = ".txt" Hmm? > } > - strcpy(filename + len, ".txt"); Here, you would write strcpy(filename + len, file_extension); > @@ -436,6 +441,8 @@ int cmd_format_patch(int argc, const char **argv, > const char *prefix) > die("Need a Message-Id for --in-reply-to"); > in_reply_to = argv[i]; > } > + else if (!strcmp(argv[i], "--psuffix")) > + psuffix = 1; and here: else if (!strncmp(argv[i], "--extension=", 12)) file_extension = argv[i] + 12; You'd call it with "--extension=.patch", and if you really want, you can make a config variable from it. Ciao, Dscho - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html