develooper Front page | perl.perl5.porters | Postings from January 2011

Re: [rt.cpan.org #65284] Module broken with 5.13.x development perl versions

Thread Previous | Thread Next
From:
David Golden
Date:
January 31, 2011 12:12
Subject:
Re: [rt.cpan.org #65284] Module broken with 5.13.x development perl versions
Message ID:
AANLkTi=Nk9vVMOmywnvHkF=oRuAZRH_z6UXYaTGHXWy2@mail.gmail.com
It's a bug in base.  It shouldn't set the $VERSION to something that
doesn't pass a "lax" version number test.

If it *has* to set VERSION for some reason, (and I'm not convinced it
should) it should just set it to '0'.

-- David

On Mon, Jan 31, 2011 at 8:19 AM, Philippe Bruhat (BooK)
<philippe.bruhat@free.fr> wrote:
> This looks like a bug in either base.pm or VERSION to me.
>
> Or should I really patch it with this?
>
> -               my $version = $mod->{module}->VERSION;
> +               my $version = ${"$mod->{module}\::VERSION"};
>
>
> ----- Forwarded message from Peter Rabbitson via RT <bug-Devel-TraceUse@rt.cpan.org> -----
>
> From: Peter Rabbitson via RT <bug-Devel-TraceUse@rt.cpan.org>
> Date: Mon, 31 Jan 2011 07:15:20 -0500
> Subject: [rt.cpan.org #65284] Module broken with 5.13.x development perl
>        versions
> To: undisclosed-recipients: ;
> Reply-To: bug-Devel-TraceUse@rt.cpan.org
> Content-Type: text/plain; charset="utf-8"
> In-Reply-To:
> References: <RT-Ticket-65284@rt.cpan.org>
> Message-ID: <rt-3.8.HEAD-17549-1296476118-222.65284-4-0@rt.cpan.org>
>
> Mon Jan 31 07:15:18 2011: Request 65284 was acted upon.
> Transaction: Ticket created by RIBASUSHI
>       Queue: Devel-TraceUse
>     Subject: Module broken with 5.13.x development perl versions
>   Broken in: (no value)
>    Severity: Important
>       Owner: Nobody
>  Requestors: ribasushi@cpan.org
>      Status: new
>  Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=65284 >
>
>
> base.pm adds a "-1, set by base.pm" to unversioned packages.
> Devel::TraceUse does a ->VERSION which throws with non-numeric version
> numbers starting somewhere in 5.13. Subsequently the following is observed:
>
> perl -d:TraceUse -e 'package Foo; sub foo{}; package Bar; use base "Foo"'
> Modules used from -e:
>   1.  base 2.15, -e line 1 [Bar]
>   2.    strict 1.04, base.pm line 3
>   3.    vars 1.02, base.pm line 4
>   4.      warnings::register 1.02, vars.pm line 7
>   5.        warnings 1.11, warnings/register.pm line 24
> Invalid version format (non-numeric data) at
> /home/rabbit/perl5/perlbrew/perls/5.13.9_rc/lib/site_perl/5.13.9/Devel/TraceUse.pm
> line 121.
>
> ----- End forwarded message -----
>
> --
>  Philippe Bruhat (BooK)
>
>  The surest path to chaos is too much organization.
>                                                 (Moral from Groo #4 (Image))
>

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