Bug #129925 for version: incorrect version comparisons

Thu Jun 27 19:45:16 2019 djerius [...] cpan.org - Ticket created

Subject:

incorrect version comparisons

Given the list of versions v2.018 v2.018_01 v2.019 I would expect that a sort using version objects would return the given order, but it doesn't: perl -Mversion -E 'say join qq[\n], sort { $a <=> $b } map { version->parse($_) } qw[ v2.018_01 v2.018 v2.019 ]' v2.018 v2.019 v2.018_01

Fri Jun 28 07:15:14 2019 haarg [...] haarg.org - Correspondence added

On Thu Jun 27 19:45:16 2019, DJERIUS wrote: Show quoted text

> Given the list of versions > > v2.018 > v2.018_01 > v2.019 > > I would expect that a sort using version objects would return the > given order, but it doesn't: > > perl -Mversion -E 'say join qq[\n], sort { $a <=> $b } map { version-

> >parse($_) } qw[ v2.018_01 v2.018 v2.019 ]'

> > v2.018 > v2.019 > v2.018_01

By using a leading v, these are parsed as dotted-decimal version numbers, not numeric version numbers. So the . is a separator, and the _ is ignored. The version number are therefore equivalent to: v2.18.0 v2.19.0 v2.1801.0 So they are being sorted correctly. If you want them to be treated as numeric versions, don't include the leading v.

Fri Jun 28 07:15:15 2019 The RT System itself - Status changed from 'new' to 'open'

Fri Jun 28 08:19:03 2019 djerius [...] cpan.org - Correspondence added

CC:	Diab Jerius <djerius [...] cpan.org>
Subject:	Re: [rt.cpan.org #129925] incorrect version comparisons
Date:	Fri, 28 Jun 2019 08:18:31 -0400
To:	bug-version [...] rt.cpan.org
From:	Diab Jerius <djerius [...] cpan.org>

Thanks for the clarification. Sorry about the noise. On Fri, Jun 28, 2019, 07:15 Graham Knop via RT <bug-version@rt.cpan.org> wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=129925 > > > On Thu Jun 27 19:45:16 2019, DJERIUS wrote:

> > Given the list of versions > > > > v2.018 > > v2.018_01 > > v2.019 > > > > I would expect that a sort using version objects would return the > > given order, but it doesn't: > > > > perl -Mversion -E 'say join qq[\n], sort { $a <=> $b } map { version-

> > >parse($_) } qw[ v2.018_01 v2.018 v2.019 ]'

> > > > v2.018 > > v2.019 > > v2.018_01

> > By using a leading v, these are parsed as dotted-decimal version numbers, > not numeric version numbers. So the . is a separator, and the _ is > ignored. The version number are therefore equivalent to: > > v2.18.0 > v2.19.0 > v2.1801.0 > > So they are being sorted correctly. If you want them to be treated as > numeric versions, don't include the leading v. >