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: Side-by-side configuration is incorrect reported as permission denied


On Tue, Aug 14, 2012 at 02:07:32AM +0200, Pawel Jasinski wrote:
>hi,
>
>How about this one. Bash has already some cygwin specific stuff.
>
>rejap <at> win7dev /usr/src/bash-4.1.10-4/src/bash-4.1
>$ diff -uN execute_cmd.c.orig execute_cmd.c
>--- execute_cmd.c.orig  2012-08-14 00:36:57.092722900 +0200
>+++ execute_cmd.c       2012-08-14 01:50:03.248969800 +0200
>@@ -4838,6 +4838,10 @@
>            }
> #endif
>          errno = i;
>+#if __CYGWIN__
>+      if (errno==EACCES && GetLastError() == ERROR_SXS_CANT_GEN_ACTCTX)
>+          errno = ELIBBAD;
>+#endif
>          file_error (command);
>        }
>       return ((i == ENOENT) ? EX_NOTFOUND : EX_NOEXEC);        /* XXX
>Posix.2 says that exit status is 126 */

Why would we do that instead of patching the DLL?  That doesn't make
any sense.

Are you somehow not getting the fact that the DLL was going to be
changed despite the fact that I said I was going to do it?  This has
been done.  You don't need to do anything here.

http://cygwin.com/ml/cygwin/2012-08/msg00253.html
http://cygwin.com/ml/cygwin/2012-08/msg00281.html
http://cygwin.com/ml/cygwin/2012-08/msg00302.html

Please drop this now.  It's clear that no one has any clever ideas about
how to make this work that doesn't involve patching every application
that uses errno.  So, this discussion should just die.

cgf

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


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