On Thu, 2003-06-05 at 09:42, Stas Bekman wrote:
> Philippe M. Chiasson wrote:
> > On Wed, 2003-06-04 at 13:38, Stas Bekman wrote:
> >
> >>blead-perl w/o ithreads fails modules/module.t with this error:
> >>
> >>[Wed Jun 4 15:34:41 2003] [error] Can't locate
> >>auto/Apache/Module/top_module.al in @INC (@INC contains:
> >>/home/stas/apache.org/modperl/blib/lib /home/stas/apache.org/modperl/blib/arch
> >>/home/stas/apache.org/modperl/t/docs
> >>/home/stas/perl/blead/lib/5.9.0/i686-linux /home/stas/perl/blead/lib/5.9.0
> >>/home/stas/perl/blead/lib/site_perl/5.9.0/i686-linux
> >>/home/stas/perl/blead/lib/site_perl/5.9.0 /home/stas/perl/blead/lib/site_perl
> >>/home/stas/apache.org/modperl/t/ /home/stas/apache.org/modperl/t/lib/perl) at
> >>/home/stas/apache.org/modperl/t/net/perl/module.pl line 20
> >
> >
> > Can't reproduce that one ;-(
> >
> > Your test are possibly grabbign an version of Apache::Module compiled
> > with a previous (incompatible MMN) or something
> >
> > $> perl -MApache::Module -e'print $INC{"Apache/Module.pm"}'
> >
> > I just dumped Apache-Module-0.11 in my mod_perl build directory and it
> > got picked up and all tests passed fine, with this simple (yet odd)
> > patch:
>
> oh, you mean you were running 'make test' and having all those skipped and you
> considered them as passed? that sounds like a bad idea.
Quite the contrary, actually ;-) I've placed most of those modules,
Apache-Sandwidch, Apache-Module, into my mp1 directory. That way, perl
Makefile.PL picks them up and the test suite picks those modules up, and
not whatever I might have installed system wide.
So, it allows me to DO test those when I make test.
> > diff -u -I$Id -r1.28 src.pm
> > --- lib/Apache/src.pm 9 Jul 2001 15:03:13 -0000 1.28
> > +++ lib/Apache/src.pm 4 Jun 2003 07:16:47 -0000
> > @@ -94,7 +94,7 @@
> > # return 1 if $Is_Win32; #just assume, till Apache::src works under
> > win32
> > my $instsrc;
> > {
> > - local @INC = grep { !/blib/ } @INC;
> > + #local @INC = grep { !/blib/ } @INC;
> > my $instdir;
> > for (@INC) {
> > last if -d ($instdir = "$_/auto/Apache/include");
> >
> >
> > Can you check for something like that, otherwise I can't reproduce that
> > problem with your setup.
>
> With your t/TEST patch below and reinstalled Apache::Module it seems to go away.
>
> >>also here we advise to use t/TEST but its shebang line could be wrong (if
> >>/usr/local/bin/perl wasn't used).
> >>http://perl.apache.org/docs/1.0/guide/install.html#Manual_Testing
> >>Would be nice to adjust t/TEST to use the right perl path
> >
> >
> > Like this?
>
> +1, we want it in!
Done!
>
> > Index: MANIFEST
> > ===================================================================
> > RCS file: /home/cvs/modperl/MANIFEST,v
> > retrieving revision 1.71
> > diff -u -I$Id -r1.71 MANIFEST
> > --- MANIFEST 25 May 2003 10:54:06 -0000 1.71
> > +++ MANIFEST 4 Jun 2003 07:21:25 -0000
> > @@ -98,7 +98,7 @@
> > src/opcodes.txt
> > t/report.PL
> > t/README
> > -t/TEST
> > +t/TEST.PL
> > t/TEST.win32
> > t/conf/httpd.conf.pl
> > t/conf/httpd.conf-dist
> > Index: Makefile.PL
> > ===================================================================
> > RCS file: /home/cvs/modperl/Makefile.PL,v
> > retrieving revision 1.211
> > diff -u -I$Id -r1.211 Makefile.PL
> > --- Makefile.PL 16 May 2003 13:26:38 -0000 1.211
> > +++ Makefile.PL 4 Jun 2003 07:21:26 -0000
> > @@ -148,6 +148,7 @@
> > src/modules/perl/mod_perl_version.h
> > t/net/perl/cgi.pl
> > t/report
> > + t/TEST
> > t/httpd
> > apaci/find_source
> > apaci/apxs_cflags
> > @@ -166,6 +167,7 @@
> >
> > gen_script("t/net/perl/cgi.pl");
> > gen_script("t/report");
> > +gen_script("t/TEST");
> > gen_script("apaci/find_source");
> > gen_script("apaci/apxs_cflags");
> > gen_script("apaci/perl_config", "$PWD/lib");
> > Index: t/.cvsignore
> > ===================================================================
> > RCS file: /home/cvs/modperl/t/.cvsignore,v
> > retrieving revision 1.3
> > diff -u -I$Id -r1.3 .cvsignore
> > --- t/.cvsignore 9 Jun 1998 16:53:09 -0000 1.3
> > +++ t/.cvsignore 4 Jun 2003 07:21:26 -0000
> > @@ -1,4 +1,5 @@
> > report
> > +TEST
> > logs
> > httpd.conf
> > httpd.conf.old
> >
> > And rename t/TEST to t/TEST.PL
> >
> > Works fine for me and is nicely autogenerated. +1 ?
>
> +1, absolutely!
Done!
> __________________________________________________________________
> Stas Bekman JAm_pH ------> Just Another mod_perl Hacker
> http://stason.org/ mod_perl Guide ---> http://perl.apache.org
> mailto:[EMAIL PROTECTED] http://use.perl.org http://apacheweek.com
> http://modperlbook.org http://apache.org http://ticketmaster.com
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
--
-- -----------------------------------------------------------------------------
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}'
signature.asc
Description: This is a digitally signed message part
