Re: canonicalize test failures on Cygwin

[Ken Brown]

On 1/17/2021 11:04 AM, Bruno Haible wrote:
> Hi Paul,
>
> Some of the new tests that you added to test-canonicalize.c and
> test-canonicalize-lgpl.c on 2020-12-24 fail on Cygwin 2.9:
>
>    /* Check that a non-directory symlink with trailing slash yields NULL,
>       and likewise for other troublesome suffixes.  */
>    {
>      char const *const file_name[]
>        = {
>           BASE "/huk/",
>           BASE "/huk/.",
>           BASE "/huk/./",
>           BASE "/huk/./.",
>           BASE "/huk/x",
>           BASE "/huk/..",
>           BASE "/huk/../",
>           BASE "/huk/../.",
>           BASE "/huk/../x",
>           BASE "/huk/./..",
>           BASE "/huk/././../x",
>          };
>      for (int i = 0; i < sizeof file_name / sizeof *file_name; i++)
>        {
>          ...
>
> The tests in the 'for' loop fail for i = 5 ... 10. That is, something
> with the ".." handling is not working as expected.

I took a quick look, and it appears that this is a Cygwin bug in which 
realpath() fails with ENOENT instead of ENOTDIR.  (I've only checked this for 
the first failure, but I suspect it's the same for all of them.)  I'll submit a 
patch to Cygwin to fix this, probably tomorrow.

Ken