You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@perl.apache.org by Stas Bekman <st...@stason.org> on 2004/02/09 23:41:46 UTC

Re: cvs commit: modperl STATUS

gozer@apache.org wrote:
> gozer       2004/02/09 14:30:07
> 
>   Modified:    .        STATUS
>   Log:
>   Record the patch+thread about correctly detecting if a script is
>   executable when filesystem ACLs are being used.

but this patch is not good, since it modifies all the calls to use 
POSIX::access, instead of just enabling:

use filetest 'access';

Hal has never responded to my message asking to test whether the above fixes 
the problem.

>   Revision  Changes    Path
>   1.46      +8 -1      modperl/STATUS
>   
>   Index: STATUS
>   ===================================================================
>   RCS file: /home/cvs/modperl/STATUS,v
>   retrieving revision 1.45
>   retrieving revision 1.46
>   diff -u -r1.45 -r1.46
>   --- STATUS	31 Dec 2003 19:24:48 -0000	1.45
>   +++ STATUS	9 Feb 2004 22:30:07 -0000	1.46
>   @@ -64,6 +64,13 @@
>    
>    Needs Patch or Further Investigation
>    
>   +    * ACL support
>   +        Report: 
>   +            http://marc.theaimsgroup.com/?t=105042055700005&r=1&w=2
>   +        Status:
>   +            Patch available with the orignial report, but needs to make sure it will work
>   +            across all platforms, POSIX or not.
>   +
>        * src/modules/ApacheModulePerl has been deprecated and replaced by src/modules/win32.
>          Since it's not even distributed anymore, can that directory be deleted?
>    
>   
>   
>   


-- 
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org


Re: cvs commit: modperl STATUS

Posted by "Philippe M. Chiasson" <go...@cpan.org>.
On Mon, 2004-02-09 at 14:41 -0800, Stas Bekman wrote:
> gozer@apache.org wrote:
> > gozer       2004/02/09 14:30:07
> > 
> >   Modified:    .        STATUS
> >   Log:
> >   Record the patch+thread about correctly detecting if a script is
> >   executable when filesystem ACLs are being used.
> 
> but this patch is not good, since it modifies all the calls to use 
> POSIX::access, instead of just enabling:
> 
> use filetest 'access';

Agreed, just wanted to log the issue in the STATUS to remember the whole
thing.

> Hal has never responded to my message asking to test whether the above fixes 
> the problem.

Well, on the minimal test I was able to run on my newly ACL-enabled ext3
filesystem, it works as expeted, at least.

Of course, the final solution would involve "use filetest qw(access)".

> >   Revision  Changes    Path
> >   1.46      +8 -1      modperl/STATUS
> >   
> >   Index: STATUS
> >   ===================================================================
> >   RCS file: /home/cvs/modperl/STATUS,v
> >   retrieving revision 1.45
> >   retrieving revision 1.46
> >   diff -u -r1.45 -r1.46
> >   --- STATUS	31 Dec 2003 19:24:48 -0000	1.45
> >   +++ STATUS	9 Feb 2004 22:30:07 -0000	1.46
> >   @@ -64,6 +64,13 @@
> >    
> >    Needs Patch or Further Investigation
> >    
> >   +    * ACL support
> >   +        Report: 
> >   +            http://marc.theaimsgroup.com/?t=105042055700005&r=1&w=2
> >   +        Status:
> >   +            Patch available with the orignial report, but needs to make sure it will work
> >   +            across all platforms, POSIX or not.
> >   +
> >        * src/modules/ApacheModulePerl has been deprecated and replaced by src/modules/win32.
> >          Since it's not even distributed anymore, can that directory be deleted?
> >    
> >   
> >   
> >   
> 
> 
> -- 
> __________________________________________________________________
> Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
> http://stason.org/     mod_perl Guide ---> http://perl.apache.org
> mailto:stas@stason.org http://use.perl.org http://apacheweek.com
> http://modperlbook.org http://apache.org   http://ticketmaster.com
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
> For additional commands, e-mail: dev-help@perl.apache.org
-- 
--------------------------------------------------------------------------------
Philippe M. Chiasson /gozer\@(cpan|ectoplasm)\.org/ 88C3A5A5 (122FF51B/C634E37B)
http://gozer.ectoplasm.org/    F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3 A5A5
Q: It is impossible to make anything foolproof because fools are so ingenious.
perl -e'$$=\${gozer};{$_=unpack(P7,pack(L,$$));/^JAm_pH\n$/&&print||$$++&&redo}'