Re: [PATCH perl.c] Fixing PERL5OPT (was Re: Warnings, strict, and CPAN)
[email protected] (Jarkko Hietaniemi) Sun, 18 Feb 2001 11:13:27 -0600
| Newsgroups | perl.perl5.porters,perl.perl6.language.strict |
|---|---|
| Message-ID | <[email protected]> |
On Sun, Feb 18, 2001 at 01:11:35AM -0500, [email protected] wrote: > On Sun, Feb 18, 2001 at 04:45:46AM +0000, Simon Cozens wrote: > > On Sat, Feb 17, 2001 at 05:00:51PM -0800, Peter Scott wrote: > > > Simon Cozens submitted a patch which failed the test > > > > ...and MJD and Jarkko and I worked on it and we put together something > > which was OK. > > Both Simon's and Peter's patches... > http://www.xray.mpe.mpg.de/mailing-lists/perl5-porters/2000-11/msg01131.html > http://www.xray.mpe.mpg.de/mailing-lists/perl5-porters/2001-02/msg00913.html > > Fail mjd's (revised) test suite. > http://www.xray.mpe.mpg.de/mailing-lists/perl5-porters/2000-11/msg01256.html > > Fortunately, its the test suite that's wrong! :) Doncha love a happy > ending? > > Here's the code patch against bleedperl along with a revised version > of mjd's suite. The revision of the test isn't quite complete. What > should really be done to prevent future mis-failures is compare > PERL5OPT switches vs those on the command line. Something like: > > PERL5OPT='-Mstrict -w' ./perl -e 'print $::x' > ./perl -Mstrict -w -e 'print $::x' > > The proper way to write the test is to make sure both do the same thing. > But this is good enough for now. > > > --- t/run/runenv.t 2001/02/18 05:58:06 1.1 > +++ t/run/runenv.t 2001/02/18 06:09:10 Applied, thanks. -- $jhi++; # http://www.iki.fi/jhi/ # There is this special biologist word we use for 'stable'. # It is 'dead'. -- Jack Cohen