This is the mail archive of the
cygwin-developers
mailing list for the Cygwin project.
Re: trailing spaces in 1.7.0
On May 12 12:58, Eric Blake wrote:
> I still don't know why coreutils compiled under 1.5.x fails when run under
> 1.7.0, but that's probably irrelevant, as I am now focusing on building
> coreutils-1.11 under 1.7.0. I just verified that the 1.7.0 image, using
> openat() and friends, has no problem with "rm -Rf 'tdir '".
>
> However, there definitely is a bug:
>
> $ mkdir 'tdir '
>
> creates 'tdir\uf020', but.
>
> $ mkdir 'tdir /'
>
> creates 'tdir ', and all subsequent attempts to stat() or open() "tdir "
> fail with ENOENT. It took 'rmdir tdir~1', and relying on the DOS 8.3 short
> name, to get rid of it. So, the code that munges trailing spaces and dots
> needs to strip trailing slashes before doing the munging.
I can easily fix that, it's a two-liner. However, that raises the
question if it's the right thing to do. Is it really necessary to
convert trailing dots and spaces? Is it really a problem if Win32
applications can't access these files? I'm a bit unsure if that's
not too indulging...
Corinna
--
Corinna Vinschen Please, send mails regarding Cygwin to
Cygwin Project Co-Leader cygwin AT cygwin DOT com
Red Hat