Re: Cygwin smoke

Op een zonnige winterdag (Tuesday 25 February 2003 13:31), schreef alian:

[about problems with 'lib/warnings.t' on CygWin]
> >>PERLIO=perlio ./perl -Ilib lib/warnings.t
> >>...
> >>ok 412
> >>ok 413
> >>...
> >>EXPECTED:
> >>Insecure dependency in chdir while running with -T switch at - line 5.
> >>GOT:
> >>Insecure dependency in chdir while running with -T switch at - line 5.
> >>not ok 414
....
> > I've bugged Schwern with such long time ago, but he didn't see where. So
> > if you can supply the patch, I guess he would be willing to apply :)
>
> Here a solution for Test-Smoke-1.16_20, who can be found here:
>
> --- mktest.pl.orig      Tue Feb 25 13:08:12 2003
> +++ mktest.pl   Tue Feb 25 13:08:59 2003
> @@ -334,6 +334,7 @@
>               if ( $perlio ne 'locale' ) {
>                   $ENV{PERLIO} = $perlio;
>                   is_win32 and $ENV{PERLIO} .= " :crlf";
> +                $^O eq 'cygwin' and $ENV{PERLIO} .= " :crlf";

That doesn't look right to me. We are talking about one (1) test. 
I don't have (enough of) cygwin to check if 

	$ PERLIO='perlio :crlf' make test

doesn't give more problems (as it should), than it takes away...

From running the above (well, t/harness) on Linux it feels there is an issue 
with PerlIO and Taint stuff:

Failed Test           Stat Wstat Total Fail  Failed  List of Failed
-------------------------------------------------------------------------------
../lib/File/Compare.t               12    1   8.33%  12
io/tell.t                           27    4  14.81%  10 25-27
op/magic.t                          50    1   2.00%  24
op/taint.t                         206    2   0.97%  1 3
run/switcht.t                       11    2  18.18%  2 4


>                   $ENV{LC_ALL} = 'C' if $force_c_locale;
>               } else {
>                   $ENV{PERL_UNICODE} = 1;
>
> >>I presume that Cygwin build use a default value of perlio :crlf as make
> >
> > No.

Indeed it shouldn't.

Good luck,

Abe
-- 
I think that's the best choice: cygwin seems to be so weird regarding
this feature that we might as well be testing rand()...
                                   -- Jarkko Hietaniemi on p5p @ 2002-03-26