Skip Menu |

This queue is for tickets about the Chemistry-OpenSMILES CPAN distribution.

Report information
The Basics
Id: 132582
Status: resolved
Priority: 0/
Queue: Chemistry-OpenSMILES

People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

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



Subject: Can't locate Chemistry/OpenSMILES/Parser.pm
On some of my smoker systems the test suite fails like this: ... Output from './Build test': print() on closed filehandle $out at /usr/perl5.18.4p/lib/site_perl/5.18.4/Module/Build/Parse/Yapp.pm line 51. Can't locate Chemistry/OpenSMILES/Parser.pm in @INC (you may need to install the Chemistry::OpenSMILES::Parser module) (@INC contains: /home/cpansand/.cpan/build/2020051408/Chemistry-OpenSMILES-0.1.1-3/blib/lib /home/cpansand/.cpan/build/2020051408/Chemistry-OpenSMILES-0.1.1-3/blib/arch inc /usr/perl5.18.4p/lib/site_perl/5.18.4/amd64-freebsd /usr/perl5.18.4p/lib/site_perl/5.18.4 /usr/perl5.18.4p/lib/5.18.4/amd64-freebsd /usr/perl5.18.4p/lib/5.18.4 .) at t/01_atoms.t line 5. BEGIN failed--compilation aborted at t/01_atoms.t line 5. t/01_atoms.t ... Dubious, test returned 2 (wstat 512, 0x200) No subtests run Can't locate Chemistry/OpenSMILES/Parser.pm in @INC (you may need to install the Chemistry::OpenSMILES::Parser module) (@INC contains: /home/cpansand/.cpan/build/2020051408/Chemistry-OpenSMILES-0.1.1-3/blib/lib /home/cpansand/.cpan/build/2020051408/Chemistry-OpenSMILES-0.1.1-3/blib/arch inc /usr/perl5.18.4p/lib/site_perl/5.18.4/amd64-freebsd /usr/perl5.18.4p/lib/site_perl/5.18.4 /usr/perl5.18.4p/lib/5.18.4/amd64-freebsd /usr/perl5.18.4p/lib/5.18.4 .) at t/02_chains.t line 6. BEGIN failed--compilation aborted at t/02_chains.t line 6. t/02_chains.t .. Dubious, test returned 2 (wstat 512, 0x200) No subtests run ...
On Thu May 14 11:43:14 2020, SREZIC wrote: Show quoted text
> Output from './Build test': > > print() on closed filehandle $out at > /usr/perl5.18.4p/lib/site_perl/5.18.4/Module/Build/Parse/Yapp.pm line > 51. > Can't locate Chemistry/OpenSMILES/Parser.pm in @INC (you may need to > install the Chemistry::OpenSMILES::Parser module) > [...]
Thanks for the report. This is most likely a bug in Module::Build::Parse::Yapp, which has been fixed in Module::Build::Parse::Yapp v0.1.2. Could you please update it and rerun the tests?
On 2020-05-14 11:47:27, MERKYS wrote: Show quoted text
> On Thu May 14 11:43:14 2020, SREZIC wrote:
> > Output from './Build test': > > > > print() on closed filehandle $out at > > /usr/perl5.18.4p/lib/site_perl/5.18.4/Module/Build/Parse/Yapp.pm line > > 51. > > Can't locate Chemistry/OpenSMILES/Parser.pm in @INC (you may need to > > install the Chemistry::OpenSMILES::Parser module) > > [...]
> > Thanks for the report. This is most likely a bug in > Module::Build::Parse::Yapp, which has been fixed in > Module::Build::Parse::Yapp v0.1.2. Could you please update it and > rerun the tests?
I tried cpan_smoke_modules -skiptestedpass -pre Module::Build::Parse::Yapp MERKYS/Chemistry-OpenSMILES-0.1.1.tar.gz and now it passes on all systems >= perl 5.10.0. So yes, it could be a problem in older Module::Build::Parse::Yapp versions.
On Thu May 14 12:48:10 2020, SREZIC wrote: Show quoted text
> I tried > > cpan_smoke_modules -skiptestedpass -pre Module::Build::Parse::Yapp > MERKYS/Chemistry-OpenSMILES-0.1.1.tar.gz > > and now it passes on all systems >= perl 5.10.0. > So yes, it could be a problem in older Module::Build::Parse::Yapp > versions.
Thanks a lot. I mark this ticket as resolved.
On 2020-05-14 12:50:49, MERKYS wrote: Show quoted text
> On Thu May 14 12:48:10 2020, SREZIC wrote:
> > I tried > > > > cpan_smoke_modules -skiptestedpass -pre Module::Build::Parse::Yapp > > MERKYS/Chemistry-OpenSMILES-0.1.1.tar.gz > > > > and now it passes on all systems >= perl 5.10.0. > > So yes, it could be a problem in older Module::Build::Parse::Yapp > > versions.
> > Thanks a lot. I mark this ticket as resolved.
However, shouldn't the minimum version be specified in your META.json/.yml/Build.PL? My smokers are still generating fail reports for 0.1.2.
On Sun May 17 03:58:54 2020, SREZIC wrote: Show quoted text
> However, shouldn't the minimum version be specified in your > META.json/.yml/Build.PL? My smokers are still generating fail reports > for 0.1.2.
Good point, thank you. Will do that in the next release.