On Mon, Feb 19, 2007 at 03:31:51AM -0000, Ron Savage wrote:
> Hi Tim
> 
> > I'd greatly appreciate some feedback from Windows users. (That's all
> > that's
> > holding up announcing a release candidate to dbi-users for wider testing.)

> I'm using Strawberry Perl on this PC, which seems to have convincingly
> confused the installation process, since it activates calls to getpwuid.

It's t/85gofer that's calling getpwuid. I figured (incorrectly) it would
do something vaguely useful on windows. Fixed in RC6.

> Perl.c: In function `boot_DBD__Perl':
> Perl.c:1182: warning: unknown conversion type character `_' in format
> Perl.c:1182: warning: too many arguments for format

That looks like a problem with Strawberry Perl since the warning is
inside the XS_VERSION_BOOTCHECK macro which perl defines.

> DBI.xs:1729: warning: int format, IV arg (arg 3)
> DBI.xs:2340: warning: int format, long unsigned int arg (arg 3)
> DBI.xs:2518: warning: int format, long unsigned int arg (arg 3)

Fixed. Thanks.

> Failed Test       Stat Wstat Total Fail  Failed  List of Failed
> -------------------------------------------------------------------------------
> t/85gofer.t        255 65280    ??   ??       %  ??
> t/zvg_85gofer.t    255 65280    ??   ??       %  ??
> t/zvp_10examp.t      2   512   206    2   0.97%  5-6
> t/zvp_85gofer.t    255 65280    ??   ??       %  ??
> t/zvxgp_10examp.t    2   512   206    2   0.97%  5-6
> t/zvxgp_85gofer.t  255 65280    ??   ??       %  ??

The 85gofer.t failures were due to getpwuid - fixed in RC6.
Of course there are likely to be other problems on windows,
deeper in the gofer transport code. (If it even works at all :)

The 10examp.t failures are probably Strawberry Perl specific:

> #   Failed test 'Remove trace file: dbitrace.log'
> #   in ./t/10examp.t at line 49.
> #          got: '0'
> #     expected: '1'
> #   Failed test 'Trace file actually gone'
> #   in ./t/10examp.t at line 50.

the code is essentially:

        DBI->trace(3, $trace_file);             # enable trace before first 
driver load
        ...
        $dbh->dump_handle("dump_handle test, write to log file", 2);
        ...
        DBI->trace(0, undef);   # turn off and restore to STDERR
        SKIP: {
                skip "cygwin has buffer flushing bug", 1 if ($^O =~ /cygwin/i);
                ok( -s $trace_file, "trace file size = " . -s $trace_file);
        }
 ==>    is( unlink( $trace_file ), 1, "Remove trace file: $trace_file" );
        ok( !-e $trace_file, "Trace file actually gone" );

The unlink fails for some reason. Could you try RC6 for me?
(I'll release it in a few minutes...)

Thanks Ron!

Tim.

Reply via email to