Geoffrey Young wrote:
sub config_defines {
+ my $self = shift;
+
my @defines = ();
for my $item (qw(useithreads)) {
@@ -88,7 +90,9 @@
push @defines, "-DPERL_\U$item";
}
- push @defines, map { "-D$_" } split " ", shift->{config}->{vars}->{defines};
+ if (my $defines = $self->{config}->{vars}->{defines}) {
+ push @defines, map { "-D$_" } split " ", $defines;
+ }
"@defines";
}


were you seeing specific undef warnings?  I ran it with and without the
-defines option and didn't run across anything.  was there a specific case
where {vars}->{defines} wasn't initalized properly in the constructor (like
t/SMOKE or something)?

Yup, t/SMOKE was the one. It needs more polishing, but for now this fix was neded to get it running.


__________________________________________________________________
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

Reply via email to