create_default_files(): don't set u+x bit on $GIT_DIR/config
commit1f32ecffd864ea5b339883f94463b0b69f4aeb35
authorMichael Haggerty <mhagger@alum.mit.edu>
Tue, 18 Nov 2014 13:50:24 +0000 (18 14:50 +0100)
committerJunio C Hamano <gitster@pobox.com>
Tue, 18 Nov 2014 18:10:54 +0000 (18 10:10 -0800)
tree3902a1037f51e73fa26b00b45f8c7b14c8e9edf1
parent76f8611a5fb7e81c1bada0fb190d573a66fc03f6
create_default_files(): don't set u+x bit on $GIT_DIR/config

Since time immemorial, the test of whether to set "core.filemode"
has been done by trying to toggle the u+x bit on $GIT_DIR/config,
which we know always exists, and then testing whether the change
"took".  I find it somewhat odd to use the config file for this
test, but whatever.

The test code didn't set the u+x bit back to its original state
itself, instead relying on the subsequent call to git_config_set()
to re-write the config file with correct permissions.

But ever since

    daa22c6f8d config: preserve config file permissions on edits (2014-05-06)

git_config_set() copies the permissions from the old config file to
the new one.  This is a good change in and of itself, but it
invalidates the create_default_files()'s assumption, causing "git
init" to leave the executable bit set on $GIT_DIR/config.

Reset the permissions on $GIT_DIR/config when we are done with the
test in create_default_files().

Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/init-db.c
t/t0001-init.sh