Bug #104467 for Perl-Dist-Strawberry: portable.perl files contain duplicate entries

Sat May 16 04:37:34 2015 haarg [...] haarg.org - Ticket created

Subject:

portable.perl files contain duplicate entries

The portable.perl file, which is used to adjust Config values, has a duplicate entry for installbin. This was a larger problem with older strawberry versions, where Parse::CPAN::Meta was used to load that file: https://github.com/Perl-Toolchain-Gang/Parse-CPAN-Meta/issues/11#issuecomment-102557882 That has since been resolved, but the file still isn't a valid YAML file.

Sat May 16 13:01:20 2015 ether [...] cpan.org - Severity Important added

Sat May 16 13:01:44 2015 ether [...] cpan.org - Cc ETHER added

Sun May 17 03:47:31 2015 dolmen [...] cpan.org - Requestor DOLMEN added

Wed May 20 07:21:48 2015 kmx [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #104467] portable.perl files contain duplicate entries
Date:	Wed, 20 May 2015 13:21:26 +0200
To:	bug-Perl-Dist-Strawberry [...] rt.cpan.org
From:	kmx <kmx [...] cpan.org>

Fixed in git repo, the next release should be fine.

Wed May 20 07:21:49 2015 The RT System itself - Status changed from 'new' to 'open'

Wed May 20 12:33:37 2015 ether [...] cpan.org - Correspondence added

On 2015-05-20 04:21:48, KMX wrote: Show quoted text

> Fixed in git repo, the next release should be fine. >

Would it be possible to do that release soon? This is an important fix and it should go out ASAP. Thanks!

Wed May 20 12:53:40 2015 kmx [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #104467] portable.perl files contain duplicate entries
Date:	Wed, 20 May 2015 18:53:24 +0200
To:	bug-Perl-Dist-Strawberry [...] rt.cpan.org
From:	kmx <kmx [...] cpan.org>

I am currently working on strawberry perl 5.22.0.1 which will be released shortly after 5.22.0 is released (within a week or two). Or do you mean to make a strawberry perl 5.20.2.2 release ?

Wed May 20 15:00:35 2015 ether [...] cpan.org - Correspondence added

On 2015-05-20 09:53:40, KMX wrote: Show quoted text

> I am currently working on strawberry perl 5.22.0.1 which will be released > shortly after 5.22.0 is released (within a week or two). > > Or do you mean to make a strawberry perl 5.20.2.2 release ?

Just my opinion, but I think this is serious enough to justify a 5.20.2.2, for people who may not want to jump to 5.22 right away.

Wed May 20 15:06:14 2015 haarg [...] haarg.org - Correspondence added

This doesn't actually cause any real problems with the current way strawberry handles the file. The YAML parser it uses, Portable::LoadYaml, doesn't cause any warnings or errors when loading the file even if it has duplicates.

Wed May 20 15:09:03 2015 ether [...] cpan.org - Correspondence added

On 2015-05-20 12:06:14, haarg wrote: Show quoted text

> This doesn't actually cause any real problems with the current way > strawberry handles the file. The YAML parser it uses, > Portable::LoadYaml, doesn't cause any warnings or errors when loading > the file even if it has duplicates.

aha, right, this was only a big problem with *earlier* versions of strawberry, that actually used Parse::CPAN::Meta instead of its own yaml parser. Ok then, no extra release is needed then!

Wed May 20 15:16:46 2015 kmx [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #104467] portable.perl files contain duplicate entries
Date:	Wed, 20 May 2015 21:16:20 +0200
To:	bug-Perl-Dist-Strawberry [...] rt.cpan.org
From:	kmx <kmx [...] cpan.org>

Show quoted text

> aha, right, this was only a big problem with *earlier* versions of strawberry, that actually used Parse::CPAN::Meta instead of its own yaml parser. Ok then, no extra release is needed then!

That's a good news as preparing for 5.22.0 is quite a lot of work.

Fri Jun 05 11:26:09 2015 kmx [...] cpan.org - Correspondence added

fixed in 5.22.0.1

Fri Jun 05 11:26:10 2015 kmx [...] cpan.org - Status changed from 'open' to 'resolved'