develooper Front page | perl.perl5.porters | Postings from March 2019

[perl #133948] Blead breaks CPAN: 2abbd513b8 breaks MP3::Tag

Thread Next
From:
James E Keenan
Date:
March 22, 2019 17:51
Subject:
[perl #133948] Blead breaks CPAN: 2abbd513b8 breaks MP3::Tag
Message ID:
rt-4.0.24-519-1553277114-1871.133948-75-0@perl.org
# New Ticket Created by  James E Keenan 
# Please include the string:  [perl #133948]
# in the subject line of all future correspondence about this issue. 
# <URL: https://rt.perl.org/Ticket/Display.html?id=133948 >


As reported to the perl5-porters newsgroup earlier today, I noticed test 
failures in MP3-Tag when building it as part of the CPAN-River-3000 run 
kicked off yesterday after the release of perl-5.29.9.  Since I was not 
immediately certain that the fault lay in blead, and since this module 
is throwing a lot of "deprecated in perl-5.32" warnings, I began by 
simply opening this bug ticket:

https://rt.cpan.org/Ticket/Display.html?id=128915

The module proved difficult to bisect because its build process requires 
the user to respond to prompts.

Eventually, with the following bisection command:

#####
perl Porting/bisect.pl -Uuseithreads --start=v5.29.8 --end=v5.29.9 
--module=MP3::Tag
#####

... I was able to identify a problematic commit:

#####
HEAD is now at 2abbd513b8 Implement variable length lookbehind in regex 
patterns
bad - non-zero exit from /tmp/YJl89Q_BUW/bin/perl -I /home/jkeenan/.cpan 
-MCPAN::MyConfig -MCPAN -e 
$CPAN::Config->{build_dir}=q{/tmp/hAZueFfERu}; -e install('MP3::Tag'); 
die unless CPAN::Shell->expand(Module => 'MP3::Tag')->uptodate;
2abbd513b87245ddb806e6bc4f59945ecb46dced is the first bad commit
commit 2abbd513b87245ddb806e6bc4f59945ecb46dced
Author: Karl Williamson <khw@cpan.org>
Date:   Sun Mar 17 21:06:10 2019 -0600

     Implement variable length lookbehind in regex patterns

     See [perl #132367].

:040000 040000 7448ad0bab5d8be805d1e8909c1a10339fafa173 
8a633adba6c841e8a9ba2cbda1e6e660d9a1d3dc M	pod
:100644 100644 658896f6150002907ba830c3743457189523b51f 
275945c388f51a00c8692b849e4007efbf64610b M	regcomp.c
:100644 100644 4b9a42c3381f1f340c5210634b61308ce8ec3522 
11cf43f4293368558c39902d4137ee4f4f470e6a M	regcomp.sym
:100644 100644 f72503e54ae3fc44735991ad88f3b637a19abd87 
cafc6b795a204d471caa5cb95eee41ad0003e981 M	regexec.c
:100644 100644 9d43083a4098781e2faa15a3a9d14400d3c72ca4 
45f64e8cdb4ab3b82d6a3aa337990c40fae07445 M	regexp.h
:100644 100644 3b53c1715fe9868506e44eedbe50c33870c1bc94 
803938ac480bf73e9ccd0721e9f8e1445e9c92b1 M	regnodes.h
:040000 040000 b87dcbd6cf55a137f71309b57ac7047965984a9c 
47b04175b15a8325d52c64f9f230ce18291b4eaf M	t
bisect run success
That took 1277 seconds.
#####

I then manually built perl and cpanm at this commit and tried to install 
MP3::Tag.  I got a FAIL.  Doing the same at the immediately preceding 
commit, I got a PASS.

Note that these were threaded builds, which is how I customarily test on 
this platform.  I have not yet tried unthreaded builds.

Karl, can you take a look?

Thank you very much.
Jim Keenan

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