Re: [PATCH v1 1/1] test: Correct detection of UTF8_NFD_TO_NFC for APFS
On Mon, Apr 30, 2018 at 8:41 AM, Torsten Bögershausen wrote: > On 30.04.18 17:33, Elijah Newren wrote: > >> On Sun, Apr 29, 2018 at 11:35 PM, wrote: >>> From: Torsten Bögershausen >>> >>> @@ -1106,12 +1106,7 @@ test_lazy_prereq UTF8_NFD_TO_NFC ' >> >> I'm not sure what "NFD" and "NFC" stand for, but I suspect the test >> prerequisite name may be specific to how HFS handled things. If so, >> should it be renamed from UTF8_NFD_TO_NFC to something else, such as >> UTF8_NORMALIZATION? > > NFD and NFC both come from the unicode standard, and are just taken > "as is" into the Git world: > https://en.wikipedia.org/wiki/Unicode_equivalence > > If you are otherwise happy with the patch, would it be possible > to run it on your system ? > (I don't have a High Sierra box, but I am confident that the test work > for you). Sure, feel free to add a Tested-by: Elijah Newren if you like. (Works with both 'test -r', as in your patch, and 'test -f', as suggested by Junio.)
Re: [PATCH v1 1/1] test: Correct detection of UTF8_NFD_TO_NFC for APFS
On 30.04.18 17:33, Elijah Newren wrote: > Hi, > > On Sun, Apr 29, 2018 at 11:35 PM, wrote: >> From: Torsten Bögershausen >> >> 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 >> Signed-off-by: Torsten Bögershausen >> --- >> t/test-lib.sh | 7 +-- >> 1 file changed, 1 insertion(+), 6 deletions(-) >> >> diff --git a/t/test-lib.sh b/t/test-lib.sh >> index ea2bbaaa7a..e206250d1b 100644 >> --- a/t/test-lib.sh >> +++ b/t/test-lib.sh >> @@ -1106,12 +1106,7 @@ test_lazy_prereq UTF8_NFD_TO_NFC ' > > I'm not sure what "NFD" and "NFC" stand for, but I suspect the test > prerequisite name may be specific to how HFS handled things. If so, > should it be renamed from UTF8_NFD_TO_NFC to something else, such as > UTF8_NORMALIZATION? NFD and NFC both come from the unicode standard, and are just taken "as is" into the Git world: https://en.wikipedia.org/wiki/Unicode_equivalence If you are otherwise happy with the patch, would it be possible to run it on your system ? (I don't have a High Sierra box, but I am confident that the test work for you). The other comments may be addressed later, may be. In any case, they should go into a different commit.
Re: [PATCH v1 1/1] test: Correct detection of UTF8_NFD_TO_NFC for APFS
Hi, On Sun, Apr 29, 2018 at 11:35 PM, wrote: > From: Torsten Bögershausen > > 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 > Signed-off-by: Torsten Bögershausen > --- > t/test-lib.sh | 7 +-- > 1 file changed, 1 insertion(+), 6 deletions(-) > > diff --git a/t/test-lib.sh b/t/test-lib.sh > index ea2bbaaa7a..e206250d1b 100644 > --- a/t/test-lib.sh > +++ b/t/test-lib.sh > @@ -1106,12 +1106,7 @@ test_lazy_prereq UTF8_NFD_TO_NFC ' I'm not sure what "NFD" and "NFC" stand for, but I suspect the test prerequisite name may be specific to how HFS handled things. If so, should it be renamed from UTF8_NFD_TO_NFC to something else, such as UTF8_NORMALIZATION? > auml=$(printf "\303\244") > aumlcdiar=$(printf "\141\314\210") > >"$auml" && > - case "$(echo *)" in > - "$aumlcdiar") > - true ;; > - *) > - false ;; > - esac > + test -r "$aumlcdiar" > ' > > test_lazy_prereq AUTOIDENT ' > -- > 2.16.0.rc0.8.g5497051b43
Re: [PATCH v1 1/1] test: Correct detection of UTF8_NFD_TO_NFC for APFS
On 30.04.18 09:56, Junio C Hamano wrote: > tbo...@web.de writes: > >> From: Torsten Bögershausen >> >> 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 >> Signed-off-by: Torsten Bögershausen >> --- >> 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 ?
Re: [PATCH v1 1/1] test: Correct detection of UTF8_NFD_TO_NFC for APFS
tbo...@web.de writes: > From: Torsten Bögershausen > > 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 > Signed-off-by: Torsten Bögershausen > --- > 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. > diff --git a/t/test-lib.sh b/t/test-lib.sh > index ea2bbaaa7a..e206250d1b 100644 > --- a/t/test-lib.sh > +++ b/t/test-lib.sh > @@ -1106,12 +1106,7 @@ test_lazy_prereq UTF8_NFD_TO_NFC ' > auml=$(printf "\303\244") > aumlcdiar=$(printf "\141\314\210") > >"$auml" && > - case "$(echo *)" in > - "$aumlcdiar") > - true ;; > - *) > - false ;; > - esac > + test -r "$aumlcdiar" > ' > > test_lazy_prereq AUTOIDENT '
[PATCH v1 1/1] test: Correct detection of UTF8_NFD_TO_NFC for APFS
From: Torsten Bögershausen 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 Signed-off-by: Torsten Bögershausen --- t/test-lib.sh | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/t/test-lib.sh b/t/test-lib.sh index ea2bbaaa7a..e206250d1b 100644 --- a/t/test-lib.sh +++ b/t/test-lib.sh @@ -1106,12 +1106,7 @@ test_lazy_prereq UTF8_NFD_TO_NFC ' auml=$(printf "\303\244") aumlcdiar=$(printf "\141\314\210") >"$auml" && - case "$(echo *)" in - "$aumlcdiar") - true ;; - *) - false ;; - esac + test -r "$aumlcdiar" ' test_lazy_prereq AUTOIDENT ' -- 2.16.0.rc0.8.g5497051b43