Skip Menu |
 

This queue is for tickets about the SVG-Calendar CPAN distribution.

Report information
The Basics
Id: 127463
Status: open
Priority: 0/
Queue: SVG-Calendar

People
Owner: IVANWILLS [...] cpan.org
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Can't locate object method "new" via package "Astro::Coord::ECI::Sun"
Download (untitled) / with headers
text/plain 430b
The test suite started to fail on my smokers: ... Can't locate object method "new" via package "Astro::Coord::ECI::Sun" (perhaps you forgot to load "Astro::Coord::ECI::Sun"?) at /usr/perl5.28.0-RC2p/lib/site_perl/5.28.0/Astro/Coord/ECI.pm line 1605, <DATA> line 129. # Looks like your test exited with 2 before it could output anything. t/moon.t .......... Dubious, test returned 2 (wstat 512, 0x200) Failed 93/93 subtests ...
RT-Send-CC: wyant [...] cpan.org
Download (untitled) / with headers
text/plain 563b
On 2018-10-26 13:52:47, SREZIC wrote: Show quoted text
> The test suite started to fail on my smokers: > > ... > Can't locate object method "new" via package "Astro::Coord::ECI::Sun" > (perhaps you forgot to load "Astro::Coord::ECI::Sun"?) at > /usr/perl5.28.0-RC2p/lib/site_perl/5.28.0/Astro/Coord/ECI.pm line > 1605, <DATA> line 129. > # Looks like your test exited with 2 before it could output anything. > t/moon.t .......... > Dubious, test returned 2 (wstat 512, 0x200) > Failed 93/93 subtests > ...
Maybe this happens if newer versions of Astro-satpass are installed.
Thanks for adding me to this ticket. I will take a look. Tom
Download (untitled) / with headers
text/plain 331b
At this point this looks like my bug. Astro::Coord::ECI::Moon 0.100 was the last good version. In 0.100_01 I put in support for alternate Sun classes (in preparation for the release of Astro::Coord::ECI::VSOP87D), and in that change 'use Astro::Coord::ECI::Sun' was removed from Astro::Coord::ECI::Moon and Astro::Coord::ECI::Star.
I have opened RT 127488 against Astro-satpass
Download (untitled) / with headers
text/plain 217b
Astro-satpass version 0.102_01, which I believe and hope fixes this, just went to PAUSE. FWIW, the minimum code to reproduce the problem (or not) is: use Astro::Coord::ECI; Astro::Coord::ECI->new()->get( 'sun' );
Download (untitled) / with headers
text/plain 434b
On Sat Oct 27 12:51:32 2018, WYANT wrote: Show quoted text
> Astro-satpass version 0.102_01, which I believe and hope fixes this, > just went to PAUSE. FWIW, the minimum code to reproduce the problem > (or not) is: > > use Astro::Coord::ECI; > Astro::Coord::ECI->new()->get( 'sun' );
No test failures in Astro-satpass 0.102_01, versus 473 passes. I have published Astro-satpass 0.103, and closed RT 127488. I suspect this ticket can be closed also.
RT-Send-CC: wyant [...] cpan.org
Download (untitled) / with headers
text/plain 719b
On 2018-11-03 11:30:30, WYANT wrote: Show quoted text
> On Sat Oct 27 12:51:32 2018, WYANT wrote:
> > Astro-satpass version 0.102_01, which I believe and hope fixes this, > > just went to PAUSE. FWIW, the minimum code to reproduce the problem > > (or not) is: > > > > use Astro::Coord::ECI; > > Astro::Coord::ECI->new()->get( 'sun' );
> > > No test failures in Astro-satpass 0.102_01, versus 473 passes. I have > published Astro-satpass 0.103, and closed RT 127488. I suspect this > ticket can be closed also.
A possible action would be to adjust the minimum prereq version in this distribution, so I'll leave the decision about closing this ticket to the author. But I can confirm that the newer version indeed fixes the problem.
Download (untitled) / with headers
text/plain 862b
I've increased the minimum version and released v0.3.13 On Sun Nov 04 03:50:47 2018, SREZIC wrote: Show quoted text
> On 2018-11-03 11:30:30, WYANT wrote:
> > On Sat Oct 27 12:51:32 2018, WYANT wrote:
> > > Astro-satpass version 0.102_01, which I believe and hope fixes > > > this, > > > just went to PAUSE. FWIW, the minimum code to reproduce the problem > > > (or not) is: > > > > > > use Astro::Coord::ECI; > > > Astro::Coord::ECI->new()->get( 'sun' );
> > > > > > No test failures in Astro-satpass 0.102_01, versus 473 passes. I have > > published Astro-satpass 0.103, and closed RT 127488. I suspect this > > ticket can be closed also.
> > A possible action would be to adjust the minimum prereq version in > this distribution, so I'll leave the decision about closing this > ticket to the author. > > But I can confirm that the newer version indeed fixes the problem.
Download (untitled) / with headers
text/plain 1.2k
On 2018-11-04 21:52:41, IVANWILLS wrote: Show quoted text
> I've increased the minimum version and released v0.3.13 > > On Sun Nov 04 03:50:47 2018, SREZIC wrote:
> > On 2018-11-03 11:30:30, WYANT wrote:
> > > On Sat Oct 27 12:51:32 2018, WYANT wrote:
> > > > Astro-satpass version 0.102_01, which I believe and hope fixes > > > > this, > > > > just went to PAUSE. FWIW, the minimum code to reproduce the problem > > > > (or not) is: > > > > > > > > use Astro::Coord::ECI; > > > > Astro::Coord::ECI->new()->get( 'sun' );
> > > > > > > > > No test failures in Astro-satpass 0.102_01, versus 473 passes. I have > > > published Astro-satpass 0.103, and closed RT 127488. I suspect this > > > ticket can be closed also.
> > > > A possible action would be to adjust the minimum prereq version in > > this distribution, so I'll leave the decision about closing this > > ticket to the author. > > > > But I can confirm that the newer version indeed fixes the problem.
> >
I still see failures if an older Astro::satpass is installed. Unfortunately it seems that setting a higher version for a recommended module does not force a reinstall. Maybe the only solution is to also check the version in the test script, and skip if it's the problematic version.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.