From: Junio C Hamano <gitster@pobox.com>
Date: Tue, 03 May 2016 19:28:12 GMT
Subject: Re: [PATCH] t7300: mark test with SANITY
Message-ID: <xmqqmvo7lz5f.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <20160503190417.GD30530@sigill.intra.peff.net>

Jeff King <peff@peff.net> writes:

> On Tue, May 03, 2016 at 11:54:32AM -0700, Stefan Beller wrote:
>
>> As the test runs `chmod 0` on a file, we don't want to run that test
>> as root.
>
> This somehow misses the root (no pun intended) of the issue, to me.
> Perhaps:
>
>   We `chmod 0` a file and test a case where git is unable to read it.
>   If the test is run as root, the permissions are ignored, and our
>   simulated read failure does not happen.
>
> The patch itself looks obviously correct. :)
>
> -Peff

By the way, it is easy to make a mistake like this, not to notice it
during a review, and to leave it unnoticed for a long time,
especially because I do not think anybody active in the development
community runs tests as 'root'.

Perhaps in a future update, Travis should learn a step to catch
breakages like like this.  Or perhaps it is not worth the effort?