Re: [PATCH 4/4] Use git_mkstemp_mode and xmkstemp_mode in odb_mkstemp, not chmod later.
- From
Matthieu Moy <matthieu.moy@grenoble-inp.fr>
- Date
- Feb 22, 2010, 07:55 UTC
- Message-ID
- <vpqfx4tln64.fsf@bauges.imag.fr>
- In-Reply-To
- <7vy6inekxq.fsf@alter.siamese.dyndns.org>
Junio C Hamano <gitster@pobox.com> writes:
Show 17 quoted lines
> Matthieu Moy <Matthieu.Moy@imag.fr> writes:
>
>> diff --git a/t/t1304-default-acl.sh b/t/t1304-default-acl.sh
>> index 4ee44a1..b8e9c34 100755
>> --- a/t/t1304-default-acl.sh
>> +++ b/t/t1304-default-acl.sh
>> @@ -27,7 +27,7 @@ modebits () {
>> ls -l "$1" | sed -e 's|^\(..........\).*|\1|'
>> }
>>
>> -test_expect_failure 'git gc does not break ACLs with restrictive umask' '
>> +test_expect_success 'git gc does not break ACLs with restrictive umask' '
>> umask 077 &&
>> git gc &&
>> actual=$(modebits .git/objects/pack/*.pack) &&
>
> This does not seem to pass for me; I get 0444 instead of 0440 as you expect.That is strange, it does pass (or it's skipped if I don't have setfacl) for me.
What system are you using? Do you have ACLs enabled on the filesystem where you run the tests? What permission do you get for packs with Git-without-my-patch (I used to get 0400)?
Thanks,
-- Matthieu Moy http://www-verimag.imag.fr/~moy/