Re: Test failure: Test #3 in t1304-default-acl
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Sep 17, 2012, 16:37 UTC
- Message-ID
- <7v4nmwwqox.fsf@alter.siamese.dyndns.org>
- In-Reply-To
- <vpqvcfcps2z.fsf@grenoble-inp.fr>
Matthieu Moy <Matthieu.Moy@grenoble-inp.fr> writes:
Show 35 quoted lines
> Ramkumar Ramachandra <artagnon@gmail.com> writes:
>
>> Hi again,
>>
>> Matthieu Moy wrote:
>>> Does this user have the same UID as your usual user
>>> (id kseygold; id $LOGNAME)?
>>
>> Yes. What do you propose we do about the test?
>
> On a GNU system, something like this should do the trick:
> ...
> but it seems FreeBSD's getfacl doesn't have --numeric
> (http://www.unix.com/man-page/FreeBSD/1/getfacl/), so to be portable, we
> need some more surgery, like this:
>
> --- a/t/t1304-default-acl.sh
> +++ b/t/t1304-default-acl.sh
> @@ -33,7 +33,10 @@ check_perms_and_acl () {
> test -r "$1" &&
> getfacl "$1" > actual &&
> grep -q "user:root:rwx" actual &&
> - grep -q "user:${LOGNAME}:rwx" actual &&
> + grep "user:.*:rwx" actual | grep -v 'user:root:' | \
> + cut -d : -f 2 | xargs id -u > acl-uid &&
> + id -u > actual-uid &&
> + test_cmp acl-uid actual-uid &&
> egrep "mask::?r--" actual > /dev/null 2>&1 &&
> grep -q "group::---" actual || false
> }
>
> Can you check that this second patch fixes your problem?
>
> I'm a bit hesitant to submit it to git.git, as I find it a bit too
> hacky. What do other people think?I haven't been paying attention, but does that mean on that system, a total stranger kseygold can write, modify, and remove whatever Ram owns? I am hoping that is not the case.