Re: resolving Apache::Test vs. Apache::test collision

2003-05-20 Thread Stas Bekman
David Wheeler wrote: On Tuesday, May 13, 2003, at 09:11 PM, Stas Bekman wrote: so that bug should be fixed in CPANPLUS, in any case new Apache/test will have a $VERSION Yes. I sent them a patch. We'll see if they apply it. So we probably should check whether UNINST=1 is on, and if not (and if

Re: resolving Apache::Test vs. Apache::test collision

2003-05-20 Thread Stas Bekman
David Wheeler wrote: On Monday, May 19, 2003, at 03:43 PM, Stas Bekman wrote: That's the trick. Each of these files contains both Apache::test and Apache::Test (do you see that each has require() called twice?). So it doesn't matter which one gets overwritten. Give it a try. But you can't have

Re: resolving Apache::Test vs. Apache::test collision

2003-05-20 Thread Stas Bekman
David Wheeler wrote: On Monday, May 19, 2003, at 05:08 PM, Stas Bekman wrote: That's right. Let's try this next: I've attached a new patch, which moves the creation of lib/Apache/test.pm into a Makefile.PL. On case-insensitive systems it'll overwrite lib/Apache/Test.pm. That appears to work,

Re: resolving Apache::Test vs. Apache::test collision

2003-05-20 Thread Stas Bekman
David Wheeler wrote: On Monday, May 19, 2003, at 06:41 PM, Stas Bekman wrote: So we need to figure out how to enforce UNINST the old Apache/test.pm if any. For example we could adjust MY::install to unlink it, without messing with UNINST=1, though the latter will be the simplest. Sounds okay

Re: resolving Apache::Test vs. Apache::test collision

2003-05-20 Thread Stas Bekman
David Wheeler wrote: On Monday, May 19, 2003, at 07:18 PM, Stas Bekman wrote: my $is_case_insensitive = -e catfile qw(lib Apache test.pm); Ah, yes, of course. So there _is_ a simple way to test for it! if it works, then yes! It works: % perl -e 'print -e lib/Apache/test.pm ? insensitive\n :

Re: resolving Apache::Test vs. Apache::test collision

2003-05-20 Thread Stas Bekman
Moreover, since now we know how to test for case-insensitive fs, we can copy Apache/testold.pm to Apache/test.pm on case-sensitive platforms which singles OSX as the only potentially problematic platform and only for 3rd party modules, which will be encouraged to rename

Re: resolving Apache::Test vs. Apache::test collision

2003-05-20 Thread Stas Bekman
The only problem is with those 1.x 3rd party modules, but they will updated to 'require Apache::testold', which we can distribute in Apache-Test. Ah, but as I said, if they have Apache::test in their distros (as in fact does MasonX::ApacheHandler::WithCallbacks and, IIRC, HTML::Mason), it

Re: resolving Apache::Test vs. Apache::test collision

2003-05-20 Thread Stas Bekman
David Wheeler wrote: On Monday, May 19, 2003, at 07:57 PM, Stas Bekman wrote: As long as they hide it from MakeMaker so it won't attempt to install it, which will just cause an inconvenience to its users. Of course. Mine is in t/lib, as is Mason's, I believe. And here is a patch that will try

Re: cvs commit: httpd-test/perl-framework/Apache-Test Changes

2003-05-20 Thread Sander Temme
on 5/19/03 23:01, [EMAIL PROTECTED] at [EMAIL PROTECTED] wrote: This breaks on RH 8.0 and Solaris 7, both on perl 5.8.0, with the following protest: pxs /tmp/apache2/bin/apxs Useless use of a constant in void context at Apache-Test/lib/Apache/TestConfig.pm line 784. Compilation failed in require

Re: cvs commit: httpd-test/perl-framework/Apache-Test Changes

2003-05-20 Thread Sander Temme
on 5/20/03 10:53, Sander Temme at [EMAIL PROTECTED] wrote: This breaks on RH 8.0 and Solaris 7, both on perl 5.8.0, with the following The following patch gets me back in the saddle, not only on Linux/Solaris but also on Darwin 6.6: Index: Apache-Test/lib/Apache/TestConfig.pm