threads / patch / 39261

patchconfig --global --edit: generate correct .gitconfig template

Subject: [PATCH] config --global --edit: generate correct .gitconfig template

## tl;dr

5 messages between May 6, 2015 and May 7, 2015. Diffs are folded; open one to read it.

replies: 4people: 3as markdown or json

Pete Harlan· May 6, 2015, 17:41 UTC · lore

When the user asks to edit an empty .gitconfig file, we learned to generate a default version in 9830534e (config --global --edit: create a template file if needed, 2014-07-25), but the generated config mistakenly set core.user and core.email instead of user.name and user.email. Fix this.

Signed-off-by: Pete Harlan <pchpublic88@gmail.com>
---

I ran up against this yesterday and it is easy enough to fix, but I don't have time right now to write tests for it.

 builtin/config.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
Show changes to builtin/config.c +2 −2
diff --git a/builtin/config.c b/builtin/config.c
index d32c532..bfd3016 100644
--- a/builtin/config.c
+++ b/builtin/config.c
@@ -455,9 +455,9 @@ static char *default_user_config(void)
 	struct strbuf buf = STRBUF_INIT;
 	strbuf_addf(&buf,
 		    _("# This is Git's per-user configuration file.\n"
-		      "[core]\n"
+		      "[user]\n"
 		      "# Please adapt and uncomment the following lines:\n"
-		      "#	user = %s\n"
+		      "#	name = %s\n"
 		      "#	email = %s\n"),
 		    ident_default_name(),
 		    ident_default_email());
-- 
2.4.0.rc3
Junio C Hamano· May 6, 2015, 17:56 UTC · re: Pete Harlan · lore

Re: [PATCH] config --global --edit: generate correct .gitconfig template

Pete Harlan <pchpublic88@gmail.com> writes:
Show 8 quoted lines
> When the user asks to edit an empty .gitconfig file, we learned to
> generate a default version in 9830534e (config --global --edit: create
> a template file if needed, 2014-07-25), but the generated config
> mistakenly set core.user and core.email instead of user.name and
> user.email.  Fix this.
>
> Signed-off-by: Pete Harlan <pchpublic88@gmail.com>
> ---

Thanks, I think this was already done in 7e110524 (config: fix settings in default_user_config template, 2015-04-17).

Show 23 quoted lines
>
> I ran up against this yesterday and it is easy enough to fix, but I
> don't have time right now to write tests for it.
>
>  builtin/config.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/builtin/config.c b/builtin/config.c
> index d32c532..bfd3016 100644
> --- a/builtin/config.c
> +++ b/builtin/config.c
> @@ -455,9 +455,9 @@ static char *default_user_config(void)
>  	struct strbuf buf = STRBUF_INIT;
>  	strbuf_addf(&buf,
>  		    _("# This is Git's per-user configuration file.\n"
> -		      "[core]\n"
> +		      "[user]\n"
>  		      "# Please adapt and uncomment the following lines:\n"
> -		      "#	user = %s\n"
> +		      "#	name = %s\n"
>  		      "#	email = %s\n"),
>  		    ident_default_name(),
>  		    ident_default_email());
Pete Harlan· May 6, 2015, 18:10 UTC · re: Junio C Hamano · lore

Re: [PATCH] config --global --edit: generate correct .gitconfig template

On Wed, May 6, 2015 at 10:56 AM, Junio C Hamano <gitster@pobox.com> wrote:
> Thanks, I think this was already done in 7e110524 (config: fix
> settings in default_user_config template, 2015-04-17).
Whoops......indeed, I'll check more than just master next time, thanks.
--Pete
Junio C Hamano· May 7, 2015, 18:41 UTC · re: Pete Harlan · lore

Re: [PATCH] config --global --edit: generate correct .gitconfig template

Pete Harlan <pchpublic88@gmail.com> writes:
Show 5 quoted lines
> On Wed, May 6, 2015 at 10:56 AM, Junio C Hamano <gitster@pobox.com> wrote:
>> Thanks, I think this was already done in 7e110524 (config: fix
>> settings in default_user_config template, 2015-04-17).
>
> Whoops......indeed, I'll check more than just master next time, thanks.

Building 'next' and using it for your everyday Git need is a good thing to do. You'll get well-eyeballed fixes earlier that way.

Matthieu Moy· May 7, 2015, 08:45 UTC · re: Pete Harlan · lore

Re: [PATCH] config --global --edit: generate correct .gitconfig template

Pete Harlan <pchpublic88@gmail.com> writes:
Show 5 quoted lines
> When the user asks to edit an empty .gitconfig file, we learned to
> generate a default version in 9830534e (config --global --edit: create
> a template file if needed, 2014-07-25), but the generated config
> mistakenly set core.user and core.email instead of user.name and
> user.email.

Oops, indeed. I don't know how I could came up with such mistake and not notice.

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/

← back to recent threads