On Thursday 02 February 2006 23:14, Philippe M. Chiasson wrote:
> > I suggest the patch below to solve the problem.
>
> Should be OPT_ALL instead of (~0), but it's a sane patch IMO.

I almost agree, but it should read

  OPT_ALL|OPT_INCNOEXEC|OPT_SYM_OWNER|OPT_MULTI

> Really, what's needed is something to expose these new override options to
> mod_perl and allow them too as arguments to add_config().

Well, I thought about that. For compatibility with 2.0 I think the best is to 
use just the patch. But generally I would rather like something like this:

  my $parms=Apache2::CmdParms->new;
  $parms->path('/some/path');
  $parms->override(OR_ALL);
  if( $parms->can('override_opts') ) {
    $parms->override_opts(OPT_ALL);
  }
  $r->add_config( [EMAIL PROTECTED], $parms );

add_config can check whether $_[1]->isa('Apache2::CmdParms') and do the right 
thing.

How about that?

Torsten

Attachment: pgpBAJdk5FTaB.pgp
Description: PGP signature

Reply via email to