This is the mail archive of the cygwin mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: bug with touch t/


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

According to Corinna Vinschen on 3/5/2008 11:36 AM:
|> $ : > t/
|> t/: Is a directory.
|>
|
| Should be fixed in the new 1.5.25-11 test release.

Yes, in 1.5.25-11, it is _correctly_ failing with ENOENT.  But in CVS, it
looks like you changed it to return EISDIR instead?
http://sourceware.org/cgi-bin/cvsweb.cgi/src/winsup/cygwin/fhandler.cc.diff?cvsroot=src&r1=1.313&r2=1.314

POSIX requires ENOENT, not EISDIR (if neither 't' nor 't/' exist, and you
are performing file name resolution for open("t/",O_RDONLY|O_CREAT), then
the resolution should be performed as if it were
open("t/.",O_RDONLY|O_CREAT) and fail with ENOENT since t is not a
directory).  EISDIR is reserved for cases like open("t/",O_WRONLY) when
"t/" already exists.

- --
Don't work too hard, make some time for fun as well!

Eric Blake             ebb9@byu.net
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Cygwin)
Comment: Public key at home.comcast.net/~ericblake/eblake.gpg
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFHz+4384KuGfSFAYARAtlLAKCIVIZQca059CICB6DIMg+AIr5XEQCfUGvM
41aotxz7nFUBCcakRNiMvYI=
=Xqmd
-----END PGP SIGNATURE-----

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]