Skip Menu |

This queue is for tickets about the Parse-Taxonomy CPAN distribution.

Report information
The Basics
Id: 106750
Status: resolved
Priority: 0/
Queue: Parse-Taxonomy
People
Owner: jkeenan [...] cpan.org
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.11
Fixed in: 0.12

History Show all quoted text
  Sat Aug 29 08:37:48 2015 SREZIC [...] cpan.org - Ticket created
Subject: t/001-new-file.t fails (if Text::CSV_XS is installled?)
http://fast-matrix.cpantesters.org/?dist=Parse-Taxonomy%200.11 shows a number of failed tests. On my smokers the test failure looks like this: Each node in the taxonomy must have a parent. The following nodes lack the expected parent: , Alpha: , , , Alpha , Epsilon: , , , Alpha , , , Alpha , Epsilon , Kappa: , , , Alpha , , , Epsilon , , , Alpha , Zeta: , , , Alpha , , , Alpha , Zeta , Lambda: , , , Alpha , , , Zeta , , , Alpha , Zeta , Mu: , , , Alpha , , , Zeta , , , Beta: , , , Beta , Eta: , , , Beta , , , Beta , Theta: , , , Beta , , , Delta: , , , Gamma: , , , Gamma , Iota: , , , Gamma , , , Gamma , Iota , Nu Extra Wordspace: , , , Gamma , , , Iota , , , Nu , Extra at t/001-new-file.t line 200. # Looks like you planned 30 tests but ran 25. # Looks like your test exited with 255 just after 25. t/001-new-file.t .......... Dubious, test returned 255 (wstat 65280, 0xff00) Failed 5/30 subtests When explicitly hiding Text::CSV_XS, then the test passes.
  Tue Sep 01 07:08:15 2015 jkeenan [...] cpan.org - Correspondence added
El Sáb Ago 29 08:37:48 2015, SREZIC escribió: Show quoted text
> http://fast-matrix.cpantesters.org/?dist=Parse-Taxonomy%200.11 shows a > number of failed tests. On my smokers the test failure looks like > this: > > Each node in the taxonomy must have a parent. > The following nodes lack the expected parent: > , Alpha: , , > , Alpha , Epsilon: , , , Alpha , , > , Alpha , Epsilon , Kappa: , , , Alpha , , , Epsilon , , > , Alpha , Zeta: , , , Alpha , , > , Alpha , Zeta , Lambda: , , , Alpha , , , Zeta , , > , Alpha , Zeta , Mu: , , , Alpha , , , Zeta , , > , Beta: , , > , Beta , Eta: , , , Beta , , > , Beta , Theta: , , , Beta , , > , Delta: , , > , Gamma: , , > , Gamma , Iota: , , , Gamma , , > , Gamma , Iota , Nu Extra Wordspace: , , , Gamma , , , Iota , , , > Nu , Extra > at t/001-new-file.t line 200. > # Looks like you planned 30 tests but ran 25. > # Looks like your test exited with 255 just after 25. > t/001-new-file.t .......... > Dubious, test returned 255 (wstat 65280, 0xff00) > Failed 5/30 subtests > > When explicitly hiding Text::CSV_XS, then the test passes.
Slaven, thank you very much for this report. In v0.11 I added only tests -- but tests which I thought might break in certain environments that I myself did not know how to reproduce. Could you tell me how you "explicitly hide Text::CSV_XS" for testing purposes? Also, do you think this is just a problem with Parse::Taxonomy -- or more generally a problem with Text::CSV? I´ll need to decide whether to require Text::CSV_XS in Makefile.PL or create a work-around. I expect I won´t get much time to work on this until Sep 08, but thanks again for this bug report -- particularly given that cpantesters is still mostly inoperative. Thank you very much. Jim Keenan
  Tue Sep 01 07:08:15 2015 The RT System itself - Status changed from 'new' to 'open'
  Tue Sep 01 16:12:27 2015 SREZIC [...] cpan.org - Correspondence added
On 2015-09-01 07:08:15, JKEENAN wrote: Show quoted text
> El Sáb Ago 29 08:37:48 2015, SREZIC escribió:
> > http://fast-matrix.cpantesters.org/?dist=Parse-Taxonomy%200.11 shows > > a > > number of failed tests. On my smokers the test failure looks like > > this: > > > > Each node in the taxonomy must have a parent. > > The following nodes lack the expected parent: > > , Alpha: , , > > , Alpha , Epsilon: , , , Alpha , , > > , Alpha , Epsilon , Kappa: , , , Alpha , , , Epsilon , , > > , Alpha , Zeta: , , , Alpha , , > > , Alpha , Zeta , Lambda: , , , Alpha , , , Zeta , , > > , Alpha , Zeta , Mu: , , , Alpha , , , Zeta , , > > , Beta: , , > > , Beta , Eta: , , , Beta , , > > , Beta , Theta: , , , Beta , , > > , Delta: , , > > , Gamma: , , > > , Gamma , Iota: , , , Gamma , , > > , Gamma , Iota , Nu Extra Wordspace: , , , Gamma , , , Iota , , > > , > > Nu , Extra > > at t/001-new-file.t line 200. > > # Looks like you planned 30 tests but ran 25. > > # Looks like your test exited with 255 just after 25. > > t/001-new-file.t .......... > > Dubious, test returned 255 (wstat 65280, 0xff00) > > Failed 5/30 subtests > > > > When explicitly hiding Text::CSV_XS, then the test passes.
> > Slaven, thank you very much for this report. In v0.11 I added only > tests -- but tests which I thought might break in certain environments > that I myself did not know how to reproduce. > > Could you tell me how you "explicitly hide Text::CSV_XS" for testing > purposes?
By using Devel::Hide. In my smoker wrapper I have the following lines to "hide" the modules listed in @hidemod: $ENV{PERL5OPT} .= " -MDevel::Hide"; $ENV{DEVEL_HIDE_VERBOSE} = 0; $ENV{DEVEL_HIDE_PM} = "@hidemod"; Show quoted text
> Also, do you think this is just a problem with Parse::Taxonomy -- or > more generally a problem with Text::CSV? I´ll need to decide whether > to require Text::CSV_XS in Makefile.PL or create a work-around.
Sorry, no idea. I also don't know whether Text::CSV and Text::CSV_XS are supposed to be 100% compatible. Regards, Slaven
  Mon Sep 07 11:23:45 2015 jkeenan [...] cpan.org - Correspondence added
On Sat Aug 29 08:37:48 2015, SREZIC wrote: Show quoted text
> http://fast-matrix.cpantesters.org/?dist=Parse-Taxonomy%200.11 shows a > number of failed tests. On my smokers the test failure looks like > this: > > Each node in the taxonomy must have a parent. > The following nodes lack the expected parent: > , Alpha: , , > , Alpha , Epsilon: , , , Alpha , , > , Alpha , Epsilon , Kappa: , , , Alpha , , , Epsilon , , > , Alpha , Zeta: , , , Alpha , , > , Alpha , Zeta , Lambda: , , , Alpha , , , Zeta , , > , Alpha , Zeta , Mu: , , , Alpha , , , Zeta , , > , Beta: , , > , Beta , Eta: , , , Beta , , > , Beta , Theta: , , , Beta , , > , Delta: , , > , Gamma: , , > , Gamma , Iota: , , , Gamma , , > , Gamma , Iota , Nu Extra Wordspace: , , , Gamma , , , Iota , , , > Nu , Extra > at t/001-new-file.t line 200. > # Looks like you planned 30 tests but ran 25. > # Looks like your test exited with 255 just after 25. > t/001-new-file.t .......... > Dubious, test returned 255 (wstat 65280, 0xff00) > Failed 5/30 subtests > > When explicitly hiding Text::CSV_XS, then the test passes.
Slaven, I have just uploaded Parse::Taxonomy version 0.12 to CPAN. This version requires Text::CSV_XS and uses one feature present therein to handle the real-world-test case that was breaking on version 0.11 when Text::CSV_XS was available. Can you give it a spin? Thank you very much. Jim Keenan
  Mon Sep 07 14:04:08 2015 SREZIC [...] cpan.org - Correspondence added
On 2015-09-07 11:23:45, JKEENAN wrote: Show quoted text
> On Sat Aug 29 08:37:48 2015, SREZIC wrote:
> > http://fast-matrix.cpantesters.org/?dist=Parse-Taxonomy%200.11 shows > > a > > number of failed tests. On my smokers the test failure looks like > > this: > > > > Each node in the taxonomy must have a parent. > > The following nodes lack the expected parent: > > , Alpha: , , > > , Alpha , Epsilon: , , , Alpha , , > > , Alpha , Epsilon , Kappa: , , , Alpha , , , Epsilon , , > > , Alpha , Zeta: , , , Alpha , , > > , Alpha , Zeta , Lambda: , , , Alpha , , , Zeta , , > > , Alpha , Zeta , Mu: , , , Alpha , , , Zeta , , > > , Beta: , , > > , Beta , Eta: , , , Beta , , > > , Beta , Theta: , , , Beta , , > > , Delta: , , > > , Gamma: , , > > , Gamma , Iota: , , , Gamma , , > > , Gamma , Iota , Nu Extra Wordspace: , , , Gamma , , , Iota , , > > , > > Nu , Extra > > at t/001-new-file.t line 200. > > # Looks like you planned 30 tests but ran 25. > > # Looks like your test exited with 255 just after 25. > > t/001-new-file.t .......... > > Dubious, test returned 255 (wstat 65280, 0xff00) > > Failed 5/30 subtests > > > > When explicitly hiding Text::CSV_XS, then the test passes.
> > Slaven, > > I have just uploaded Parse::Taxonomy version 0.12 to CPAN. This > version requires Text::CSV_XS and uses one feature present therein to > handle the real-world-test case that was breaking on version 0.11 when > Text::CSV_XS was available. > > Can you give it a spin? > > Thank you very much. > Jim Keenan
Looks good: http://fast-matrix.cpantesters.org/?dist=Parse-Taxonomy%200.12
  Mon Sep 07 14:04:10 2015 SREZIC [...] cpan.org - Status changed from 'open' to 'resolved'
  Mon Sep 07 14:04:11 2015 SREZIC [...] cpan.org - Fixed in 0.12 added
  Wed Sep 09 21:17:10 2015 jkeenan [...] cpan.org - Correspondence added
I'm going to keep this ticket open at least until cpantesters comes back to life. The fast-matrix version of cpantesters is reporting problems on Windows for older versions of Perl (see: http://fast-matrix.cpantesters.org/?dist=Parse-Taxonomy%200.12). But I probably won't be able to diagnose these until we get "traditional" CPANtesters failure reports.
  Wed Sep 09 21:17:11 2015 jkeenan [...] cpan.org - Status changed from 'resolved' to 'open'
  Wed Sep 09 21:17:11 2015 jkeenan [...] cpan.org - Taken
  Sat Sep 19 19:24:08 2015 jkeenan [...] cpan.org - Correspondence added
On Wed Sep 09 21:17:10 2015, JKEENAN wrote: Show quoted text
> I'm going to keep this ticket open at least until cpantesters comes > back to life. The fast-matrix version of cpantesters is reporting > problems on Windows for older versions of Perl (see: http://fast- > matrix.cpantesters.org/?dist=Parse-Taxonomy%200.12). But I probably > won't be able to diagnose these until we get "traditional" CPANtesters > failure reports.
CPANtesters.org is back: http://matrix.cpantesters.org/?dist=Parse-Taxonomy+0.12 Which demonstrates that the problem discussed in this ticket has been addressed. (There are some test failures, but they're being addressed in v0.13.) Re-closing the ticket. Thank you very much. Jim Keenan
  Sat Sep 19 19:24:09 2015 jkeenan [...] cpan.org - Status changed from 'open' to 'resolved'