Re: renamed modules again

2005-02-15 Thread Stas Bekman
Joe Orton wrote: On Mon, Feb 14, 2005 at 03:42:45PM -0500, Stas Bekman wrote: If you look at the current code with your latest commit Joe, it tries to deal with the same issue twice. The unless condition already tries to tackle that, but won't do it if there is no c extension. unless

Re: renamed modules again

2005-02-15 Thread Joe Orton
On Mon, Feb 14, 2005 at 03:42:45PM -0500, Stas Bekman wrote: > If you look at the current code with your latest commit Joe, it tries to > deal with the same issue twice. The unless condition already tries to > tackle that, but won't do it if there is no c extension. > > unless ($mod

Re: renamed modules again

2005-02-14 Thread Geoffrey Young
>> Indeed! OK thanks, committed that. > > > Where did the commit message go? I didn't get any. I think we still have some email issues. joe mentioned earlier today that the permissions need to change on /x1/www/perl.apache.org/mail/test-dev/* and I suspect that applies to the test-dev dir

Re: renamed modules again

2005-02-14 Thread Stas Bekman
Joe Orton wrote: On Mon, Feb 14, 2005 at 08:53:44AM -0500, Geoffrey Young wrote: looks ok to me, though it means that in addition to have_module('mod_rewrite.c'); have_module('mod_rewrite'); working have_module('rewrite'); have_module('rewrite.c'); will also work. hopefully that is intuitive i

Re: renamed modules again

2005-02-14 Thread Joe Orton
On Mon, Feb 14, 2005 at 08:53:44AM -0500, Geoffrey Young wrote: > looks ok to me, though it means that in addition to > > have_module('mod_rewrite.c'); > have_module('mod_rewrite'); > > working > > have_module('rewrite'); > have_module('rewrite.c'); > > will also work. hopefully that i

Re: renamed modules again

2005-02-14 Thread Geoffrey Young
Joe Orton wrote: > [not sure if this made it through to the list first time; resent] > > Renamed modules are causing me to lose hair again; would it be OK to do > something like the below, so a call like have_module("proxy_http.c") > will succeed if either a module named either "proxy_http.c" or