Re: [PATCH] Do not assume glibc glob internals

[Paul Smith]

On Sun, 2017-09-24 at 11:29 -0700, Paul Eggert wrote:
> Paul Smith wrote:
> > +      size_t plen = strlen (path);
> > +
> > +      /* If path ends in /. make sure it is a directory.  This is because
> > +         'lstat' on Windows normalizes the argument foo/. => foo without
> > +         checking first that foo is a directory.  */
> > +      if (plen > 1 && path[plen - 1] == '.'
> > +          && (path[plen - 2] == '/' || path[plen - 2] == '\\')
> > +          && !_S_ISDIR (buf->st_mode))
> > +        return -1;
> 
> Isn't part of the problem that MS-Windows also mishandles names like
> "foo/" that end in "/" as opposed to ending in "/."? (I don't use MS-
> Windows so perhaps I should keep my mouth shut here, but still...

I can't answer your question, but the previous version of this function
didn't do anything special with a "foo/" path, so if it doesn't work
here it didn't work there either...?