Bug #72770 for DynaLoader-Functions: t\dyna

Mon Nov 28 04:33:41 2011 CHORNY [...] cpan.org - Ticket created

Subject:

t\dyna_low.t fails on Windows

undefined reference to `boot_t__dyna_low' See http://www.cpantesters.org/cpan/report/71bba2ef-6c9b-1014-8db7-8f18dd1fd8d8 -- Alexandr Ciornii, http://chorny.net

Tue Feb 07 09:11:32 2012 ZEFRAM [...] cpan.org - Correspondence added

Thanks. Think that's fixed in DynaLoader-Functions-0.001.

Tue Feb 07 09:11:33 2012 The RT System itself - Status changed from 'new' to 'open'

Tue Feb 07 09:11:33 2012 ZEFRAM [...] cpan.org - Status changed from 'open' to 'resolved'

Tue Feb 07 11:47:30 2012 CHORNY [...] cpan.org - Correspondence added

On Tue Feb 07 09:11:32 2012, ZEFRAM wrote: Show quoted text

> Thanks. Think that's fixed in DynaLoader-Functions-0.001.

New error: t\dyna_low.t .. 2/6 # Failed test at t\dyna_low.t line 43. # Failed test at t\dyna_low.t line 44. # Looks like you failed 2 tests of 6. t\dyna_low.t .. Dubious, test returned 2 (wstat 512, 0x200) Failed 2/6 subtests -- Alexandr Ciornii, http://chorny.net

Tue Feb 07 11:47:31 2012 The RT System itself - Status changed from 'resolved' to 'open'

Tue Feb 07 12:17:51 2012 zefram [...] fysh.org - Correspondence added

Subject:	Re: [rt.cpan.org #72770] t\dyna_low.t fails on Windows
Date:	Tue, 7 Feb 2012 17:17:38 +0000
To:	Alexandr Ciornii via RT <bug-DynaLoader-Functions [...] rt.cpan.org>
From:	Zefram <zefram [...] fysh.org>

Alexandr Ciornii via RT wrote: Show quoted text

>t\dyna_low.t .. 2/6 # Failed test at t\dyna_low.t line 43.

Damn, need to mark the symbols for export. Should have remembered that. Please try replacing t/dyna_low.c with the attached file, and see whether that passes. -zefram

Message body is not shown because sender requested not to inline it.

Tue Feb 07 13:01:18 2012 CHORNY [...] cpan.org - Correspondence added

On Tue Feb 07 12:17:51 2012, zefram@fysh.org wrote: Show quoted text

> Alexandr Ciornii via RT wrote:

> >t\dyna_low.t .. 2/6 # Failed test at t\dyna_low.t line 43.

> > Damn, need to mark the symbols for export. Should have remembered that. > Please try replacing t/dyna_low.c with the attached file, and see whether > that passes.

Same errors :(. -- Alexandr Ciornii, http://chorny.net

Tue Feb 07 14:05:06 2012 zefram [...] fysh.org - Correspondence added

Subject:	Re: [rt.cpan.org #72770] t\dyna_low.t fails on Windows
Date:	Tue, 7 Feb 2012 19:04:52 +0000
To:	Alexandr Ciornii via RT <bug-DynaLoader-Functions [...] rt.cpan.org>
From:	Zefram <zefram [...] fysh.org>

Alexandr Ciornii via RT wrote: Show quoted text

>Same errors :(.

I'd better be more systematic about this. Going to review the whole build process for that module, compare it against the standard module build process. Still pretty convinced that it's the test that's buggy, not DL:F. -zefram

Wed Oct 03 03:47:14 2012 shay [...] cpan.org - Correspondence added

On Tue Feb 07 14:05:06 2012, zefram@fysh.org wrote: Show quoted text

> Alexandr Ciornii via RT wrote:

> >Same errors :(.

> > I'd better be more systematic about this. Going to review the whole > build process for that module, compare it against the standard module > build process. Still pretty convinced that it's the test that's buggy, > not DL:F. > > -zefram

I think it's partly the test (needing to define exports explicitly on Win32) and partly ExtUtils::CBuilder, which doesn't pass those exports through when linking. The latter is fixed by perl commit 4ae07704fb, so if you depend on ExtUtils::CBuilder 0.280209 (the $VERSION bump was commit 5fe0597566) and apply the change to the test file found here: http://www.xray.mpe.mpg.de/mailing-lists/perl5-porters/2012-10/msg00097.html then the test does now succeed. (I guess you'll need to get EU::CB's maintainer to roll a new CPAN release before depending on the new $VERSION, though.)

Sat Aug 17 09:53:37 2013 ZEFRAM [...] cpan.org - Correspondence added

Fixed in DynaLoader-Functions-0.002, now on CPAN.

Sat Aug 17 09:53:38 2013 ZEFRAM [...] cpan.org - Status changed from 'open' to 'resolved'

Bug #72770 for DynaLoader-Functions: t\dyna_low.t fails on Windows