config: Add new option to open an editor.

The idea was originated by discussion about usability of manually
editing the config file in 'special needs' systems such as Windows. Now
the user can forget a bit about where the config files actually are.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/Documentation/git-config.txt b/Documentation/git-config.txt
index 19a8917b83f4ff111e9eec0767c3261a2c39d995..7d140073b13224d0763a6f62f58fc765bfbf0003 100644 (file)
--- a/Documentation/git-config.txt
+++ b/Documentation/git-config.txt
@@ -22,6 +22,7 @@ SYNOPSIS
 'git config' [<file-option>] [-z|--null] -l | --list
 'git config' [<file-option>] --get-color name [default]
 'git config' [<file-option>] --get-colorbool name [stdout-is-tty]
 'git config' [<file-option>] [-z|--null] -l | --list
 'git config' [<file-option>] --get-color name [default]
 'git config' [<file-option>] --get-colorbool name [stdout-is-tty]

+'git config' [<file-option>] -e | --edit

 
 DESCRIPTION
 -----------
 
 DESCRIPTION
 -----------


@@ -157,6 +158,11 @@ See also <<FILES>>.
        output.  The optional `default` parameter is used instead, if
        there is no color configured for `name`.
 
        output.  The optional `default` parameter is used instead, if
        there is no color configured for `name`.
 

+-e::
+--edit::
+       Opens an editor to modify the specified config file; either
+       '--system', '--global', or repository (default).
+

 [[FILES]]
 FILES
 -----
 [[FILES]]
 FILES
 -----

diff --git a/builtin-config.c b/builtin-config.c
index f71016204b540d0d935323c909a0ffccb1abdbe2..6937eaf379aa060ece1bbbfbb3a830c1a349bfe6 100644 (file)
--- a/builtin-config.c
+++ b/builtin-config.c
@@ -3,7 +3,7 @@
 #include "color.h"
 
 static const char git_config_set_usage[] =
 #include "color.h"
 
 static const char git_config_set_usage[] =

-"git config [ --global | --system | [ -f | --file ] config-file ] [ --bool | --int | --bool-or-int ] [ -z | --null ] [--get | --get-all | --get-regexp | --replace-all | --add | --unset | --unset-all] name [value [value_regex]] | --rename-section old_name new_name | --remove-section name | --list | --get-color var [default] | --get-colorbool name [stdout-is-tty]";
+"git config [ --global | --system | [ -f | --file ] config-file ] [ --bool | --int | --bool-or-int ] [ -z | --null ] [--get | --get-all | --get-regexp | --replace-all | --add | --unset | --unset-all] name [value [value_regex]] | --rename-section old_name new_name | --remove-section name | --list | --get-color var [default] | --get-colorbool name [stdout-is-tty] | --edit | -e ]";

 
 static char *key;
 static regex_t *key_regexp;
 
 static char *key;
 static regex_t *key_regexp;


@@ -362,6 +362,17 @@ int cmd_config(int argc, const char **argv, const char *prefix)
                        return get_color(argc-2, argv+2);
                } else if (!strcmp(argv[1], "--get-colorbool")) {
                        return get_colorbool(argc-2, argv+2);
                        return get_color(argc-2, argv+2);
                } else if (!strcmp(argv[1], "--get-colorbool")) {
                        return get_colorbool(argc-2, argv+2);

+               } else if (!strcmp(argv[1], "--edit") || !strcmp(argv[1], "-e")) {
+                       const char *config_filename;
+                       if (argc != 2)
+                               usage(git_config_set_usage);
+                       if (config_exclusive_filename)
+                               config_filename = config_exclusive_filename;
+                       else
+                               config_filename = git_path("config");
+                       git_config(git_default_config, NULL);
+                       launch_editor(config_filename, NULL, NULL);
+                       return 0;

                } else
                        break;
                argc--;
                } else
                        break;
                argc--;