Skip Menu |

This queue is for tickets about the Perl-Dist-Strawberry CPAN distribution.

Report information
The Basics
Id: 54729
Status: resolved
Priority: 0/
Queue: Perl-Dist-Strawberry
People
Owner: csjewell [...] cpan.org
Requestors: csjewell [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Wed Feb 17 16:53:05 2010 csjewell [...] cpan.org - Ticket created
Subject: Prefer (and include) YAML::XS in April 2010.
<DrHyde> klapperl: CPAN.pm can be astonishingly slow building Metadata on some of my testers, particularly the netbsd/alpha box, but i don't consider it to be a problem. if anyone thinks i should be outraged, i can time it if you like :-) <klapperl> if memory is low we need DBD::SQLite, this much is known; the other dangerous thing is to use YAML.pm instead of YAML::{XS,Syck} So I want to set CPAN to use YAML::XS instead.
  Wed Feb 17 16:54:04 2010 csjewell [...] cpan.org - Severity Wishlist added
  Wed Feb 17 16:54:04 2010 csjewell [...] cpan.org - Dependency by ticket #54728 added
  Tue Feb 23 02:53:44 2010 csjewell [...] cpan.org - Correspondence added
Subject: Prefer (and include) YAML::Syck in April 2010.
Changed to YAML::Syck, patched in r11435 & r11436.
  Tue Feb 23 02:53:50 2010 csjewell [...] cpan.org - Subject changed from 'Prefer (and include) YAML::XS in April 2010.' to 'Prefer (and include) YAML::Syck in April 2010.'
  Tue Feb 23 02:53:50 2010 csjewell [...] cpan.org - Status changed from 'open' to 'patched'
  Tue Feb 23 02:53:53 2010 csjewell [...] cpan.org - Given to CSJEWELL
  Tue Feb 23 02:53:55 2010 csjewell [...] cpan.org - Reference to http://fisheye2.atlassian.com/changelog/cpan/?cs=11435 added
  Tue Feb 23 02:53:56 2010 csjewell [...] cpan.org - Reference to http://fisheye2.atlassian.com/changelog/cpan/?cs=11436 added
  Tue Feb 23 08:58:52 2010 adam [...] ali.as - Correspondence added
Subject: Re: [rt.cpan.org #54729] Prefer (and include) YAML::Syck in April 2010.
Date: Tue, 23 Feb 2010 23:11:01 +1100
To: bug-Perl-Dist-Strawberry [...] rt.cpan.org
From: Adam Kennedy <adamkennedybackup [...] gmail.com>
I'd prefer to see YAML::XS instead of Syck, if we can (and if it works). Syck is buggier and the upstream code isn't as well maintained. Adam K On 23 February 2010 18:53, Curtis Jewell via RT <bug-Perl-Dist-Strawberry@rt.cpan.org> wrote: Show quoted text
>       Queue: Perl-Dist-Strawberry >  Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=54729 > > > Changed to YAML::Syck, patched in r11435 & r11436. >
  Tue Feb 23 08:58:56 2010 The RT System itself - Status changed from 'patched' to 'open'
  Tue Feb 23 10:31:43 2010 csjewell [...] cpan.org - Correspondence added
On Tue Feb 23 08:58:52 2010, adam@ali.as wrote: Show quoted text
> I'd prefer to see YAML::XS instead of Syck, if we can (and if it works). > > Syck is buggier and the upstream code isn't as well maintained.
That can be done... I was just looking at the dependencies for ::XS and going "Filter::Util::Call? Source filtering?" and feeling a little uncomfortable about THAT - that's why I had done Syck first. But if ::Syck is buggier, that's a trump card. ::XS it is.
  Tue Feb 23 17:24:54 2010 csjewell [...] cpan.org - Correspondence added
Subject: Prefer (and include) YAML::XS in April 2010.
On Tue Feb 23 10:31:43 2010, CSJEWELL wrote: Show quoted text
> On Tue Feb 23 08:58:52 2010, adam@ali.as wrote:
> > I'd prefer to see YAML::XS instead of Syck, if we can (and if it works). > > > > Syck is buggier and the upstream code isn't as well maintained.
> > That can be done... I was just looking at the dependencies for ::XS and > going "Filter::Util::Call? Source filtering?" and feeling a little > uncomfortable about THAT - that's why I had done Syck first. > > But if ::Syck is buggier, that's a trump card. ::XS it is.
Had to leave for a while... when I came back, I saw an error, so I'm reporting a bug on CPAN.pm (RT#54919) about ::XS not working. (::Syck didn't work, either.)
  Tue Feb 23 17:24:56 2010 csjewell [...] cpan.org - Subject changed from 'Prefer (and include) YAML::Syck in April 2010.' to 'Prefer (and include) YAML::XS in April 2010.'
  Tue Feb 23 17:24:57 2010 csjewell [...] cpan.org - Dependency on ticket #54919 added
  Tue Feb 23 22:01:48 2010 csjewell [...] cpan.org - Correspondence added
Unfortunately, we're stalled on RT#54919, but I'm putting patches up to "hit the switch" once that bug DOES get fixed.
Subject: Perl-Dist-Strawberry.patch
Index: share/bootstrap/perl-5.10.0/lib/CPAN/Config.pm =================================================================== --- share/bootstrap/perl-5.10.0/lib/CPAN/Config.pm (revision 11434) +++ share/bootstrap/perl-5.10.0/lib/CPAN/Config.pm (working copy) @@ -79,7 +79,7 @@ use_sqlite => q[1], wget => q[], yaml_load_code => q[0], - yaml_module => q[YAML], + yaml_module => q[YAML::XS], }; 1; Index: share/strawberry/perl-5.10.0/lib/CPAN/Config.pm.tt =================================================================== --- share/strawberry/perl-5.10.0/lib/CPAN/Config.pm.tt (revision 11434) +++ share/strawberry/perl-5.10.0/lib/CPAN/Config.pm.tt (working copy) @@ -81,7 +81,7 @@ use_sqlite => q[1], wget => q[], yaml_load_code => q[0], - yaml_module => q[YAML], + yaml_module => q[YAML::XS], }; 1; Index: share/strawberry/perl-5.10.1/lib/CPAN/Config.pm.tt =================================================================== --- share/strawberry/perl-5.10.1/lib/CPAN/Config.pm.tt (revision 11434) +++ share/strawberry/perl-5.10.1/lib/CPAN/Config.pm.tt (working copy) @@ -81,7 +81,7 @@ use_sqlite => q[1], wget => q[], yaml_load_code => q[0], - yaml_module => q[YAML], + yaml_module => q[YAML::XS], }; 1; Index: share/strawberry/perl-5.8.9/lib/CPAN/Config.pm.tt =================================================================== --- share/strawberry/perl-5.8.9/lib/CPAN/Config.pm.tt (revision 11434) +++ share/strawberry/perl-5.8.9/lib/CPAN/Config.pm.tt (working copy) @@ -71,7 +71,7 @@ use_sqlite => q[1], wget => q[], yaml_load_code => q[0], - yaml_module => q[YAML], + yaml_module => q[YAML::XS], }; 1;
Subject: Perl-Dist-WiX.patch
Index: lib/Perl/Dist/WiX/Toolchain.pm =================================================================== --- lib/Perl/Dist/WiX/Toolchain.pm (revision 11434) +++ lib/Perl/Dist/WiX/Toolchain.pm (working copy) @@ -179,6 +179,7 @@ HTML::Tagset HTML::Parser LWP::UserAgent + YAML::XS } ], ); Index: share/default/perl-5.10.0/lib/CPAN/Config.pm.tt =================================================================== --- share/default/perl-5.10.0/lib/CPAN/Config.pm.tt (revision 11434) +++ share/default/perl-5.10.0/lib/CPAN/Config.pm.tt (working copy) @@ -23,6 +23,7 @@ 'unzip' => q[ ], 'wget' => q[ ], 'load_module_verbosity' => 'none', + 'yaml_module' => 'YAML::XS', }; 1; __END__ Index: share/default/perl-5.10.1/lib/CPAN/Config.pm.tt =================================================================== --- share/default/perl-5.10.1/lib/CPAN/Config.pm.tt (revision 11434) +++ share/default/perl-5.10.1/lib/CPAN/Config.pm.tt (working copy) @@ -23,6 +23,7 @@ 'unzip' => q[ ], 'wget' => q[ ], 'load_module_verbosity' => 'none', + 'yaml_module' => 'YAML::XS', }; 1; __END__ Index: share/default/perl-5.11.5/lib/CPAN/Config.pm.tt =================================================================== --- share/default/perl-5.11.5/lib/CPAN/Config.pm.tt (revision 11434) +++ share/default/perl-5.11.5/lib/CPAN/Config.pm.tt (working copy) @@ -23,6 +23,7 @@ 'unzip' => q[ ], 'wget' => q[ ], 'load_module_verbosity' => 'none', + 'yaml_module' => 'YAML::XS', }; 1; __END__ Index: share/default/perl-5.8.9/lib/CPAN/Config.pm.tt =================================================================== --- share/default/perl-5.8.9/lib/CPAN/Config.pm.tt (revision 11434) +++ share/default/perl-5.8.9/lib/CPAN/Config.pm.tt (working copy) @@ -23,6 +23,7 @@ 'unzip' => q[ ], 'wget' => q[ ], 'load_module_verbosity' => 'none', + 'yaml_module' => 'YAML::XS', }; 1; __END__ Index: share/default/perl-git/lib/CPAN/Config.pm.tt =================================================================== --- share/default/perl-git/lib/CPAN/Config.pm.tt (revision 11434) +++ share/default/perl-git/lib/CPAN/Config.pm.tt (working copy) @@ -23,6 +23,7 @@ 'unzip' => q[ ], 'wget' => q[ ], 'load_module_verbosity' => 'none', + 'yaml_module' => 'YAML::XS', }; 1; __END__
  Tue Feb 23 22:01:49 2010 csjewell [...] cpan.org - Status changed from 'open' to 'stalled'
  Tue Feb 23 22:01:50 2010 csjewell [...] cpan.org - Reference to http://fisheye2.atlassian.com/changelog/cpan/?cs=11435 deleted
  Tue Feb 23 22:01:50 2010 csjewell [...] cpan.org - Reference to http://fisheye2.atlassian.com/changelog/cpan/?cs=11436 deleted
  Thu Apr 22 09:33:11 2010 csjewell [...] cpan.org - Dependency by ticket #56822 added
  Thu Apr 22 09:34:22 2010 csjewell [...] cpan.org - Dependency by ticket #54728 deleted
  Thu Aug 19 23:56:28 2010 csjewell [...] cpan.org - Dependency by ticket #56822 deleted
  Thu Aug 19 23:56:28 2010 csjewell [...] cpan.org - Dependency by ticket #60539 added
  Sun Aug 29 23:04:40 2010 CHORNY [...] cpan.org - Correspondence added
Filter::Util::Call is a dependency of Test::Base, so it is a build dependency. I submitted a patch so it it will be shown in build_requires. -- Alexandr Ciornii, http://chorny.net
  Sun Aug 29 23:04:41 2010 The RT System itself - Status changed from 'stalled' to 'open'
  Sun Feb 26 16:54:32 2012 kmx [...] cpan.org - Dependency by ticket #60539 deleted
  Wed Sep 17 11:11:33 2014 kmx [...] cpan.org - Dependency on ticket #54919 deleted
  Wed Sep 17 11:11:53 2014 kmx [...] cpan.org - Status changed from 'open' to 'resolved'