Re: [PATCH 2/2] test-lib.sh: Dynamic test for the prerequisite SANITY
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jan 28, 2015, 20:38 UTC
- Message-ID
- <xmqq7fw6hc83.fsf@gitster.dls.corp.google.com>
- In-Reply-To
- <54C9362E.2000705@web.de>
Torsten Bögershausen <tboegi@web.de> writes:
Show 7 quoted lines
>> OK, but MacOS X does not have SANITY problem; "is the / writable?" test >> was misdetecting and declaring a system with SANITY does not have one. >> >> Perhaps roll Cygwin and Mingw into a single Windows category? I dunno. > The whole discussion actually started with Mac OS X, > and the conclusion was that Mac OS X should have SANITY set, but hadn't, > because / is writable (if you install from scratch):
Exactly.
On MacOSX running as non-root-but-can-write-to-slash-user, we should say SANITY is satisfied, but "is the / writable?" check was misdetecting and declaring the system does not have SANITY.
So we are in agreement, no?