Skip Menu |

This queue is for tickets about the YAML-Tiny CPAN distribution.

Report information
The Basics
Id: 40370
Status: resolved
Priority: 0/
Queue: YAML-Tiny

People
Owner: jkeenan [...] cpan.org
Requestors: wyp3rlx02 [...] sneakemail.com
Cc:
AdminCc:

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



CC: Gabor Szabo <szabgab [...] gmail.com>, cpan-workers <cpan-workers [...] perl.org>, bug-yaml-tiny [...] rt.cpan.org
Subject: Re: did I break something or just old YAML ?
Date: Sat, 25 Oct 2008 11:06:09 +0200
To: "(Andreas J. Koenig)" <andreas.koenig.7os6VVqR [...] franz.ak.mind.de>
From: Steffen Mueller <wyp3rlx02 [...] sneakemail.com>
Hi Andreas, Hi Gabor, (Andreas J. Koenig) wrote: Show quoted text
>>>>>> On Fri, 24 Oct 2008 15:12:12 +0200, "Gabor Szabo" <szabgab@gmail.com> said:
>
> > Here is the output from installing Spreadsheet-Read-0.29: > > $ module_version YAML > > 0.66 > > $ module_version YAML::Tiny > > 1.32
> > Nowadays one must set the CPAN.pm config option to > yaml_module=YAML::Syck. Both YAML.pm and YAML::Tiny cannot read the > perfectly valid META.yml file that comes with Spreadsheet-Read-0.29 > > YAML::Syck does not have this issue.
Since YAML::Tiny is specifically aimed at being able to at least parse all META.yml's on CPAN, this should be reported as a bug. Do not forget that the META.yml reader in core also uses YAML::Tiny's code, so this is important to fix. CCing the YAML::Tiny bug tracker. Best regards, Steffen
RT-Send-CC: andreas.koenig.7os6VVqR [...] franz.ak.mind.de, szabgab [...] gmail.com
On Sat Oct 25 05:06:34 2008, wyp3rlx02@sneakemail.com wrote: Show quoted text
> Hi Andreas, Hi Gabor, > > (Andreas J. Koenig) wrote:
> >>>>>> On Fri, 24 Oct 2008 15:12:12 +0200, "Gabor Szabo" > >>>>>> <szabgab@gmail.com> said:
> >
> > > Here is the output from installing Spreadsheet-Read-0.29: > > > $ module_version YAML > > > 0.66 > > > $ module_version YAML::Tiny > > > 1.32
> > > > Nowadays one must set the CPAN.pm config option to > > yaml_module=YAML::Syck. Both YAML.pm and YAML::Tiny cannot read the > > perfectly valid META.yml file that comes with Spreadsheet-Read-0.29 > > > > YAML::Syck does not have this issue.
> > Since YAML::Tiny is specifically aimed at being able to at least parse > all META.yml's on CPAN, this should be reported as a bug. Do not > forget > that the META.yml reader in core also uses YAML::Tiny's code, so this > is > important to fix. > > CCing the YAML::Tiny bug tracker. > > Best regards, > Steffen
I have been asked by David Golden to review tickets in YAML::Tiny's rt.cpan.org bug queue. AFAICT, YAML::Tiny has no problem parsing Spreadsheet::Read's META.yml file. Testing Spreadsheet-Read v0.49, I get: ########## my $y = YAML::Tiny->read($meta); say Dumper $y; ########## $VAR1 = bless( [ { 'requires' => { 'perl' => '5.006', 'Carp' => '0', 'File::Temp' => '0.22', 'Test::Harness' => '0', 'Exporter' => '0', 'Test::More' => '0.88', 'Data::Dumper' => '0', 'Test::NoWarnings' => '0' }, 'provides' => { 'Spreadsheet::Read' => { 'file' => 'Read.pm', 'version' => '0.49' } }, 'version' => '0.49', 'name' => 'Spreadsheet-Read', 'author' => [ 'H.Merijn Brand <h.m.brand@xs4all.nl>' ], 'recommends' => { 'Test::More' => '0.98', 'File::Temp' => '0.2301', 'perl' => '5.016003', 'IO::Scalar' => '0' }, 'optional_features' => { 'opt_tools' => { 'description' => 'Spreadsheet tools' }, 'opt_excelx' => { 'requires' => { 'Spreadsheet::XLSX' => '0.13', 'Spreadsheet::XLSX::Fmt2007' => '0' }, 'description' => 'Provides parsing of Microsoft Excel 2007 files' }, 'opt_excel' => { 'description' => 'Provides parsing of Microsoft Excel files', 'requires' => { 'Spreadsheet::ParseExcel' => '0.26', 'Spreadsheet::ParseExcel::FmtDefault' => '0' } }, 'opt_csv' => { 'requires' => { 'Text::CSV_XS' => '0.69' }, 'description' => 'Provides parsing of CSV streams' }, 'opt_oo' => { 'description' => 'Provides parsing of OpenOffice spreadsheets', 'requires' => { 'Spreadsheet::ReadSXC' => '0.20' } } }, 'generated_by' => 'Author, CPAN::Meta::Converter version 2.130880', 'dynamic_config' => '1', 'meta-spec' => { 'url' => 'http://module-build.sourceforge.net/META-spec-v1.4.html', 'version' => '1.4' }, 'configure_requires' => { 'ExtUtils::MakeMaker' => '0' }, 'abstract' => 'Meta-Wrapper for reading spreadsheet data', 'resources' => { 'license' => 'http://dev.perl.org/licenses/', 'repository' => 'http://repo.or.cz/r/Spreadsheet-Read.git' }, 'license' => 'perl' } ], 'YAML::Tiny' ); ########## I will close this ticket within 7 days unless someone offers a significant objection. Thank you very much. Jim Keenan
RT-Send-CC: andreas.koenig.7os6VVqR [...] franz.ak.mind.de, szabgab [...] gmail.com
On Fri Sep 27 22:47:11 2013, JKEENAN wrote: Show quoted text
> > AFAICT, YAML::Tiny has no problem parsing Spreadsheet::Read's META.yml > file. > > Testing Spreadsheet-Read v0.49, I get: > > ########## > my $y = YAML::Tiny->read($meta); > say Dumper $y; > ########## > $VAR1 = bless( [ > { > 'requires' => { > 'perl' => '5.006', > 'Carp' => '0', > 'File::Temp' => '0.22', > 'Test::Harness' => '0', > 'Exporter' => '0', > 'Test::More' => '0.88', > 'Data::Dumper' => '0', > 'Test::NoWarnings' => '0' > }, > 'provides' => { > 'Spreadsheet::Read' => { > 'file' => 'Read.pm', > 'version' => '0.49' > } > }, > 'version' => '0.49', > 'name' => 'Spreadsheet-Read', > 'author' => [ > 'H.Merijn Brand <h.m.brand@xs4all.nl>' > ], > 'recommends' => { > 'Test::More' => '0.98', > 'File::Temp' => '0.2301', > 'perl' => '5.016003', > 'IO::Scalar' => '0' > }, > 'optional_features' => { > 'opt_tools' => { > 'description' => 'Spreadsheet tools' > }, > 'opt_excelx' => { > 'requires' => { > 'Spreadsheet::XLSX' => '0.13', > 'Spreadsheet::XLSX::Fmt2007' => '0' > }, > 'description' => 'Provides parsing of Microsoft Excel 2007 > files' > }, > 'opt_excel' => { > 'description' => 'Provides parsing of Microsoft Excel files', > 'requires' => { > 'Spreadsheet::ParseExcel' => '0.26', > 'Spreadsheet::ParseExcel::FmtDefault' => '0' > } > }, > 'opt_csv' => { > 'requires' => { > 'Text::CSV_XS' => '0.69' > }, > 'description' => 'Provides parsing of CSV streams' > }, > 'opt_oo' => { > 'description' => 'Provides parsing of OpenOffice > spreadsheets', > 'requires' => { > 'Spreadsheet::ReadSXC' => '0.20' > } > } > }, > 'generated_by' => 'Author, CPAN::Meta::Converter version > 2.130880', > 'dynamic_config' => '1', > 'meta-spec' => { > 'url' => 'http://module-build.sourceforge.net/META-spec- > v1.4.html', > 'version' => '1.4' > }, > 'configure_requires' => { > 'ExtUtils::MakeMaker' => '0' > }, > 'abstract' => 'Meta-Wrapper for reading spreadsheet data', > 'resources' => { > 'license' => 'http://dev.perl.org/licenses/', > 'repository' => 'http://repo.or.cz/r/Spreadsheet-Read.git' > }, > 'license' => 'perl' > } > ], 'YAML::Tiny' ); > ########## > > I will close this ticket within 7 days unless someone offers a > significant objection. > > Thank you very much. > Jim Keenan
No objection raised. Hence, closing as per schedule.