Skip Menu |
Report information
Id: 133278
Status: resolved
Priority: 0/
Queue: perl5

Owner: Nobody
Requestors: rehsack [at] gmail.com
Cc:
AdminCc:

Operating System: (no value)
PatchStatus: (no value)
Severity: low
Type: unknown
Perl Version: (no value)
Fixed In: (no value)



To: perlbug [...] perl.org
From: Jens Rehsack <rehsack [...] gmail.com>
Subject: Missing av_top_index in Devel::PPPort
Date: Wed, 13 Jun 2018 18:19:26 +0200
Download (untitled) / with headers
text/plain 481b
perlapi reports AvFILL is deprecated and encourages using of av_top_index() instead. Unfortunately av_top_index has been introduced with 5.18 - ideally ppport.h supports a fallback for older Perls. See https://travis-ci.org/perl5-utils/List-MoreUtils-XS/builds/391774513 for the failures (with up-to-date ppport.h) The 5.18 failure comes from unfortunate recover from RT#123868/RT#96343 (cpan) and needs rework from author (me). Best regards -- Jens Rehsack - rehsack@gmail.com
Download signature.asc
application/pgp-signature 833b

Message body not shown because it is not plain text.

RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 628b
On Wed, 13 Jun 2018 09:19:39 -0700, rehsack@gmail.com wrote: Show quoted text
> perlapi reports AvFILL is deprecated and encourages using of > av_top_index() instead. > Unfortunately av_top_index has been introduced with 5.18 - ideally > ppport.h supports a fallback for older Perls.
I agree, and will work on it shortly Show quoted text
> > See https://travis-ci.org/perl5-utils/List-MoreUtils- > XS/builds/391774513 for the failures (with up-to-date ppport.h) > The 5.18 failure comes from unfortunate recover from > RT#123868/RT#96343 (cpan) and needs rework from author (me). > > Best regards > -- > Jens Rehsack - rehsack@gmail.com
-- Karl Williamson
To: perl5-porters [...] perl.org
From: Tony Cook <tony [...] develop-help.com>
Subject: Re: [perl #133278] Missing av_top_index in Devel::PPPort
Date: Fri, 15 Jun 2018 16:02:07 +1000
Download (untitled) / with headers
text/plain 816b
On Wed, Jun 13, 2018 at 09:19:39AM -0700, Jens Rehsack (via RT) wrote: Show quoted text
> # New Ticket Created by Jens Rehsack > # Please include the string: [perl #133278] > # in the subject line of all future correspondence about this issue. > # <URL: https://rt.perl.org/Ticket/Display.html?id=133278 > > > > perlapi reports AvFILL is deprecated and encourages using of av_top_index() instead. > Unfortunately av_top_index has been introduced with 5.18 - ideally ppport.h supports a fallback for older Perls. > > See https://travis-ci.org/perl5-utils/List-MoreUtils-XS/builds/391774513 for the failures (with up-to-date ppport.h) > The 5.18 failure comes from unfortunate recover from RT#123868/RT#96343 (cpan) and needs rework from author (me).
AvFILL() shouldn't deprecated, maybe discouraged, but not deprecated. Tony
RT-Send-CC: perl5-porters [...] perl.org
On Thu, 14 Jun 2018 23:02:25 -0700, tonyc wrote: Show quoted text
> On Wed, Jun 13, 2018 at 09:19:39AM -0700, Jens Rehsack (via RT) wrote:
> > # New Ticket Created by Jens Rehsack > > # Please include the string: [perl #133278] > > # in the subject line of all future correspondence about this issue. > > # <URL: https://rt.perl.org/Ticket/Display.html?id=133278 > > > > > > > perlapi reports AvFILL is deprecated and encourages using of > > av_top_index() instead. > > Unfortunately av_top_index has been introduced with 5.18 - ideally > > ppport.h supports a fallback for older Perls. > > > > See https://travis-ci.org/perl5-utils/List-MoreUtils- > > XS/builds/391774513 for the failures (with up-to-date ppport.h) > > The 5.18 failure comes from unfortunate recover from > > RT#123868/RT#96343 (cpan) and needs rework from author (me).
> > AvFILL() shouldn't deprecated, maybe discouraged, but not deprecated. > > Tony
I changed perlapi to not say that this was deprecated (it isn't) in commit d29fa84aee8bfff9c27f68ec2d9292e1370e0924 Still working on Devel::PPPort -- Karl Williamson
RT-Send-CC: perl5-porters [...] perl.org
This is now fixed in blead, and available for download from cpan -- Karl Williamson
Download (untitled) / with headers
text/plain 313b
Thank you for filing this report. You have helped make Perl better. With the release today of Perl 5.30.0, this and 160 other issues have been resolved. Perl 5.30.0 may be downloaded via: https://metacpan.org/release/XSAWYERX/perl-5.30.0 If you find that the problem persists, feel free to reopen this ticket.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

For issues related to this RT instance (aka "perlbug"), please contact perlbug-admin at perl.org