From: cgf AT cygnus DOT com (Christopher Faylor)
Subject: Re: handling file attributes
23 Oct 1998 12:02:09 -0700
Message-ID: <19981023144544.A29546.cygnus.cygwin32.developers@cygnus.com>
References: <3627056A DOT 1BD34CC4 AT jaist DOT ac DOT jp>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
To: Kazuhiro Fujieda <fujieda AT jaist DOT ac DOT jp>, cygwin32-developers AT cygnus DOT com

On Fri, Oct 23, 1998 at 06:37:06PM +0900, Kazuhiro Fujieda wrote:
>>>> On Thu, 22 Oct 1998 08:23:33 -0400
>>>> Christopher Faylor <cgf AT cygnus DOT com> said:
>
>> If you'd like to rework
>> this so that it is only active when NTEA is not activated then I'll add
>> that change.
>
>Umm...
>What do you think about this ?
>
>--- fhandler.cc-	Wed Oct 21 14:29:47 1998
>+++ fhandler.cc	Fri Oct 23 17:39:16 1998
>@@ -1076,7 +1076,10 @@
>   if (!res)
>     goto out;
> 
>-  if (!real_path.exec_p && os_being_run != winNT)
>+  extern BOOL allow_ntea;
>+
>+  if (!real_path.exec_p &&
>+      (os_being_run != winNT || !allow_ntea))
>     {
>       DWORD done;
>       char magic[3];

I've applied this patch.

>By the way, the following code seems to be wastful.
>
>path.cc:1517-
>      /* Check the file's extended attributes, if it has any.  */
>      int unixattr;
>
>      if (get_file_attribute (path, &unixattr) > 0)
>	{
>	  if ((unixattr & STD_XBITS) != 0)
>	    *exec = 1;
>	  if (! S_ISLNK (unixattr))
>	    ;
>	}
>
>The exec flag set by this code is used only to skip checking a
>file suffix in fhandler.cc:1064-1066. get_file_attribute() is
>too heavy for this purpose becase it read NTEA. These code can
>be eliminted safly.

Unless I'm missing something the exec flag set there is used
in a number of places that use path_conv.

cgf