threads / bug / 45588

Bug: 'git config --local user.email=<alt.email>' fails silently?

Subject: Bug: 'git config --local user.email=<alt.email>' fails silently?

## tl;dr

3 messages between Apr 2, 2017 and Apr 2, 2017.

replies: 2people: 2as markdown or json

Knut Omang· Apr 2, 2017, 05:47 UTC · lore
Hi,
From the documentation I would have expected 
git config --local user.email=alt.email@alt.domain
to create a section 
[user]
	email=alt.email@alt.domain
in the local .git/config.
Instead it returns status 1 with no error message.
Is this intentional?

If I add the [user] section manually, it works as expected/desired: my commits in that tree uses the alternate mail address.

git --version git version 2.7.4

Thanks, Knut Omang

-- 
Knut Omang, Ph.D
Principal Software Engineer
Oracle Linux and Virtualization Engineering
ORACLE Norway | Olaf Helsets vei 6 | 0694 Oslo
Email: knut.omang@oracle.com | Phone:  +47 41 46 22 10
Jeff King· Apr 2, 2017, 07:38 UTC · re: Knut Omang · lore

Re: Bug: 'git config --local user.email=<alt.email>' fails silently?

On Sun, Apr 02, 2017 at 07:47:23AM +0200, Knut Omang wrote:
Show 10 quoted lines
> From the documentation I would have expected 
> 
> git config --local user.email=alt.email@alt.domain
> 
> to create a section 
> 
> [user]
> 	email=alt.email@alt.domain
> 
> in the local .git/config.

When it sees one argument, git-config treats that argument as a key to be retrieved. When given two, the second is a value to be set. E.g.:

  $ git config foo.bar
  $ git config foo.bar some-value
  $ git config foo.bar
  some-value

So your command was interpreted as a request to fetch the value, which doesn't exist.

> Instead it returns status 1 with no error message.

Hopefully that explains the response you saw; we do not emit an error message when a key isn't found, which makes it easy for scripts to do things like:

  value=$(git config foo.bar || echo default-value)
without being unnecessarily noisy.

Usually we'd catch an error like yours and complain, because the key is syntactically invalid ("=" is not generally allowed in key names):

  $ git config foo.bar=some-value
  error: invalid key: foo.bar=some-value

But your argument actually _is_ a syntactically valid key, because of the dots. In a three-level key like "one.two.three", the second level subsection is allowed to contain any character (including "=" and more dots). So your "user.email=alt.email@alt.domain" tries to look up the config represented by:

  [user "email=alt.email@alt"]
  domain
Which of course did not exist.
> Is this intentional?

Yes, everything is working as intended. The documentation in git-config(1) seems to be quite poor at describing the various operating modes, though.

-Peff
Knut Omang· Apr 2, 2017, 07:53 UTC · re: Jeff King · lore

Re: Bug: 'git config --local user.email=<alt.email>' fails silently?

On Sun, 2017-04-02 at 03:38 -0400, Jeff King wrote:
Show 56 quoted lines
> On Sun, Apr 02, 2017 at 07:47:23AM +0200, Knut Omang wrote:
> 
> > From the documentation I would have expected 
> > 
> > git config --local user.email=alt.email@alt.domain
> > 
> > to create a section 
> > 
> > [user]
> > 	email=alt.email@alt.domain
> > 
> > in the local .git/config.
> 
> When it sees one argument, git-config treats that argument as a key to
> be retrieved. When given two, the second is a value to be set. E.g.:
> 
>   $ git config foo.bar
>   $ git config foo.bar some-value
>   $ git config foo.bar
>   some-value
> 
> So your command was interpreted as a request to fetch the value, which
> doesn't exist.
> 
> > Instead it returns status 1 with no error message.
> 
> Hopefully that explains the response you saw; we do not emit an error
> message when a key isn't found, which makes it easy for scripts to do
> things like:
> 
>   value=$(git config foo.bar || echo default-value)
> 
> without being unnecessarily noisy.
> 
> Usually we'd catch an error like yours and complain, because the key is
> syntactically invalid ("=" is not generally allowed in key names):
> 
>   $ git config foo.bar=some-value
>   error: invalid key: foo.bar=some-value
> 
> But your argument actually _is_ a syntactically valid key, because of
> the dots. In a three-level key like "one.two.three", the second level
> subsection is allowed to contain any character (including "=" and more
> dots). So your "user.email=alt.email@alt.domain" tries to look up the
> config represented by:
> 
>   [user "email=alt.email@alt"]
>   domain
> 
> Which of course did not exist.
> 
> > Is this intentional?
> 
> Yes, everything is working as intended. The documentation in
> git-config(1) seems to be quite poor at describing the various operating
> modes, though.

Ah - I see!  Thanks for the quick answer and excellent explanation, and sorry for the confusion - I should know well that config takes the write argument after a blank.

I think I'll go and get myself another cup of coffee  before I ask more questions anywhere...

Regards, Knut

> 
> -Peff

← back to recent threads