Re: [PATCH v1 1/1] test: Correct detection of UTF8_NFD_TO_NFC for APFS
- From
Torsten Bögershausen <tboegi@web.de>
- Date
- Apr 30, 2018, 08:55 UTC
- Message-ID
- <255708be-3cf2-0285-b5e7-4fbc9e0e8ddd@web.de>
- In-Reply-To
- <xmqqvac9xh4d.fsf@gitster-ct.c.googlers.com>
On 30.04.18 09:56, Junio C Hamano wrote:
Show 29 quoted lines
> tboegi@web.de writes: > >> From: Torsten Bögershausen <tboegi@web.de> >> >> On HFS (which is the default Mac filesystem prior to High Sierra), >> unicode names are "decomposed" before recording. >> On APFS, which appears to be the new default filesystem in Mac OS High >> Sierra, filenames are recorded as specified by the user. >> >> APFS continues to allow the user to access it via any name >> that normalizes to the same thing. >> >> This difference causes t0050-filesystem.sh to fail two tests. >> >> Improve the test for a NFD/NFC in test-lib.sh: >> Test if the same file can be reached in pre- and decomposed unicode. >> >> Reported-By: Elijah Newren <newren@gmail.com> >> Signed-off-by: Torsten Bögershausen <tboegi@web.de> >> --- >> t/test-lib.sh | 7 +------ >> 1 file changed, 1 insertion(+), 6 deletions(-) > > Thanks. > > Wouldn't it logically make more sense to check for the target being > an existing file with "-f"? It is not an essential part of the test > for the target to be "readable", but "can be stat(2)ed with the > other UTF-8 representation" is.
That make sense. Would you like to amend the patch ?