develooper Front page | perl.perl5.porters | Postings from April 2015

Re: [perl #124364] Some version strings in %Module::CoreList::deltacontain trailing space and cause warning with version->parse

Thread Previous | Thread Next
From:
Perl Ancar
Date:
April 24, 2015 04:44
Subject:
Re: [perl #124364] Some version strings in %Module::CoreList::deltacontain trailing space and cause warning with version->parse
Message ID:
CAMyyfzBP0KvrfwLfQ8GjL-B8oKmCxvpJTmU=6hR0=BuZ--_LVQ@mail.gmail.com
I'd love to get it integrated, but we should probably get at least a couple
of other people looking at the code first to check its correctness first.

On Thu, Apr 23, 2015 at 7:24 PM, Ricardo Signes via RT <
perlbug-followup@perl.org> wrote:

> * perlancar <perlbug-followup@perl.org> [2015-04-23T02:15:31]
> > Some version strings in %Module::CoreList::delta contain trailing space
> and
> > cause warning with version->parse, e.g. there is one '1.15 ' which when
> fed
> > to version->parse under warnings:
> >
> >  % perl -wE'say version->parse("1.15 ")'
> >  Version string '1.15 ' contains invalid data; ignoring: ' ' at -e line
> 1.
> >  1.15
>
> Thanks, I believe I have just fixed all this with b7c1f08.
>
> I see that you probably found this while working on Module::CoreList::More,
> which contains, among other things, a "faster is_core."  Is there a reason
> we
> shouldn't use your implementation in Module::CoreList?
>
> It seems a shame to leave it in an add-on module that many users would
> never
> see.
>
> --
> rjbs
>
>

Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About