Re: [rfc] new ModPerl::MM

Geoffrey Young wrote:
> Stas Bekman wrote:
>  > Geoffrey Young wrote:
>  >
>  >>   # The whitespace in front of @echo MUST be a single tab!
>  >>   return <<'EOF';
>  >> test::
>  >>     @echo This test suite requires Apache::Test
>  >>     @echo available from the mod_perl 2.0 sources
>  >>     @echo or the httpd-test distribution.
>  >> EOF
>  >
>  >
>  > I think you should also include the 'pure_all' target, so 'make test'
>  > doesn't preclude 'make':
>
> I don't understand what you mean here.  what does pure_all cover and how 
> would this test target interfere with it?  I suppose that there are lots 
> of targets, but I figured that Apache::Test just installs its own test 
> target, so doing the same would be sufficient.  but I dunno.

You should be able to do: perl Makefile.PL && make test
In your case, this won't work. pure_all tells the test target to run the 'all' 
target first (which is the default target when you say 'make', equivalent to 
'make all').

>  >
>  >         return <<'EOF';
>  > test : pure_all
>  >     @echo \*** This test suite requires Apache::Test available from the
>  >     @echo \*** mod_perl 2.0 sources or the httpd-test distribution.
>  > EOF
>  >
>  > btw, what's the difference between : and :: in the makefile target?
>
> there is some difference, but I really forget what it is.  what I do 
> remember is that when I made the rarget test: make complained about 
> seeing two targets (test: and test::), so I figured that MakeMaker 
> wasn't picking it up properly.

Why do you end up with two test targets in first place?


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