Skip Menu |

This queue is for tickets about the aliased CPAN distribution.

Report information
The Basics
Id: 100359
Status: resolved
Priority: 0/
Queue: aliased

People
Owner: Nobody in particular
Requestors: ether [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: (no value)
Fixed in: 0.33



Subject: deprecation warnings under 5.21.6
Under 5.21.6 (still in blead, but due to be released next week), I'm seeing warnings like: Constants from lexical variables potentially modified elsewhere are deprecated at /home/sand/src/perl/repoperls/installed-perls/perl/v5.21.5-426-gb255a11/a2da/lib/site_perl/5.21.6/aliased.pm line 46. I'm told this is from this change: http://perl5.git.perl.org/perl.git/commitdiff/0ac016fc68a8b6f210ddacb62494e53da203c571 In this commit you can see the 'perldiag' explanation for the warning.
On Sat Nov 15 13:15:47 2014, ETHER wrote: Show quoted text
> Under 5.21.6 (still in blead, but due to be released next week), I'm > seeing warnings like: > > Constants from lexical variables potentially modified elsewhere are > deprecated at /home/sand/src/perl/repoperls/installed- > perls/perl/v5.21.5-426-gb255a11/a2da/lib/site_perl/5.21.6/aliased.pm > line 46. > > I'm told this is from this change: > > http://perl5.git.perl.org/perl.git/commitdiff/0ac016fc68a8b6f210ddacb62494e53da203c571 > > In this commit you can see the 'perldiag' explanation for the warning.
See attachment.
Subject: open_tk7QsBwv.txt
diff -rup aliased-0.31-OsgpDT-orig/lib/aliased.pm aliased-0.31-OsgpDT/lib/aliased.pm --- aliased-0.31-OsgpDT-orig/lib/aliased.pm 2014-11-15 12:06:25.000000000 -0800 +++ aliased-0.31-OsgpDT/lib/aliased.pm 2014-11-15 12:06:32.000000000 -0800 @@ -42,8 +42,11 @@ sub _make_alias { ? $alias : "$callpack\::$alias"; + # need a scalar not referenced elsewhere to make the sub inlinable + my $pack2 = $package; + no strict 'refs'; - *{ $destination } = sub () { $package }; + *{ $destination } = sub () { $pack2 }; } sub _load_alias {
I'd be happy to ship this if you want to give me comaint. I'm keen to see this resolved soon, as I have a module that fails tests due to this warning.
On 2014-11-16 18:33:28, ETHER wrote: Show quoted text
> I'd be happy to ship this if you want to give me comaint. I'm keen to > see this resolved soon, as I have a module that fails tests due to > this warning.
*gentle poke*
On 2014-12-13 13:21:48, ETHER wrote: Show quoted text
> On 2014-11-16 18:33:28, ETHER wrote:
> > I'd be happy to ship this if you want to give me comaint. I'm keen to > > see this resolved soon, as I have a module that fails tests due to > > this warning.
> > > *gentle poke*
Hi, 5.21.7 is now out. I'm still happy to take care of this issue for you.
Thanks all, 0.33 has been uploaded to PAUSE (unauthorized for now, until the perms get sorted out).