threads / bug / 19200

[BUG] git config: cygwin git version 1.6.1.2 does not deletes empty sections

Subject: [BUG] git config: cygwin git version 1.6.1.2 does not deletes empty sections

## tl;dr

4 messages between May 6, 2009 and May 6, 2009.

replies: 3people: 3as markdown or json

Constantine Plotnikov· May 6, 2009, 15:50 UTC · lore

I have encountered a bug related to git config in cygwin git version 1.6.1.2

The git config --unset might leave an empty section when the last property is deleted. But consequent "git config key value" does not uses this empty section. So the following script leaves a sequence of empty sections "[a]". Looks like a disk space leak to me.

git config a.b a git config --unset a.b git config a.b a git config --unset a.b git config a.b a git config --unset a.b git config a.b a git config --unset a.b

The resulting .git/config file:

[core]
	repositoryformatversion = 0
	filemode = true
	bare = false
	logallrefupdates = true
	ignorecase = true
[a]
[a]
[a]
[a]

Was this bug fixed in later versions?

Constantine

Constantine Plotnikov· May 6, 2009, 16:04 UTC · re: Constantine Plotnikov · lore

Re: [BUG] git config: cygwin git version 1.6.1.2 does not deletes empty sections

The bug is also confirmed on MSYS git version 1.6.2.2.1669.g7eaf8.

Constantine

On Wed, May 6, 2009 at 7:50 PM, Constantine Plotnikov <constantine.plotnikov@gmail.com> wrote:

Show 33 quoted lines

> I have encountered a bug related to git config in cygwin git version 1.6.1.2
>
> The git config --unset might leave an empty section when the last
> property is deleted. But consequent "git config key value" does not
> uses this empty section. So the following script leaves a sequence of
> empty sections "[a]". Looks like a disk space leak to me.
>
> git config a.b a
> git config --unset a.b
> git config a.b a
> git config --unset a.b
> git config a.b a
> git config --unset a.b
> git config a.b a
> git config --unset a.b
>
> The resulting .git/config file:
>
> [core]
>        repositoryformatversion = 0
>        filemode = true
>        bare = false
>        logallrefupdates = true
>        ignorecase = true
> [a]
> [a]
> [a]
> [a]
>
> Was this bug fixed in later versions?
>
> Constantine
>

Erik Faye-Lund· May 6, 2009, 18:08 UTC · re: Constantine Plotnikov · lore

Re: [BUG] git config: cygwin git version 1.6.1.2 does not deletes empty sections

On Wed, May 6, 2009 at 6:04 PM, Constantine Plotnikov <constantine.plotnikov@gmail.com> wrote:

> The bug is also confirmed on MSYS git version 1.6.2.2.1669.g7eaf8.

The same issue is present in git 1.6.2.1 on Linux. Not that I'm too sure it's really a bug, though ;)

-- 
Erik "kusma" Faye-Lund
kusmabite@gmail.com
(+47) 986 59 656

Michał Kiedrowicz· May 6, 2009, 20:43 UTC · re: Erik Faye-Lund · lore

Re: [BUG] git config: cygwin git version 1.6.1.2 does not deletes empty sections

Erik Faye-Lund <kusmabite@googlemail.com> wrote:

Show 7 quoted lines

> On Wed, May 6, 2009 at 6:04 PM, Constantine Plotnikov
> <constantine.plotnikov@gmail.com> wrote:
> > The bug is also confirmed on MSYS git version 1.6.2.2.1669.g7eaf8.
> 
> The same issue is present in git 1.6.2.1 on Linux. Not that I'm too
> sure it's really a bug, though ;)
>

Happens also in master (v1.6.3-rc4-29-g8146f19). IMO this is because git_parse_file() reports only *variables*, not *sections* and store_aux() doesn't find empty section.

-- Michał Kiedrowicz