Open jkeenan opened 2 months ago
karenetheridge
commented
2 months ago Try-Tiny contained some tests for smartmatch which would fail on 5.41.3, and it's at the very top of the cpan river; I've done a release for this to fix the tests, so if you try again you should see a substantial improvement (certainly everything in the Moose ecosystem depends on Try::Tiny, so many of those distributions should disappear from the failure list now).
jkeenan
commented
2 months ago Try-Tiny contained some tests for smartmatch which would fail on 5.41.3, and it's at the very top of the cpan river; I've done a release for this to fix the tests, so if you try again you should see a substantial improvement (certainly everything in the Moose ecosystem depends on Try::Tiny, so many of those distributions should disappear from the failure list now).
@karenetheridge, thanks very much for that. I also see that @atoomic has released a new version of Clone which was the missing prerequisite most often cited in the cpan logs. I have now kicked off a new screen session testing 530 CPAN distributions.
jkeenan
commented
2 months ago Try-Tiny contained some tests for smartmatch which would fail on 5.41.3, and it's at the very top of the cpan river; I've done a release for this to fix the tests, so if you try again you should see a substantial improvement (certainly everything in the Moose ecosystem depends on Try::Tiny, so many of those distributions should disappear from the failure list now).
@karenetheridge, thanks very much for that. I also see that @atoomic has released a new version of Clone which was the missing prerequisite most often cited in the
cpanlogs. I have now kicked off a new screen session testing 530 CPAN distributions.
Swift action on the part of several CPAN distribution maintainers has greatly alleviated the problems we were facing just two days ago. As a consequence, I no longer feel that we need to consider reverting either of the two big code removal projects or postponing Tuesday's scheduled release of perl-5.41.3, nor do we need a code freeze in advance of that release.
I say this on the basis of data gathered yesterday in comparison to similar data gathered last month and earlier this month. I often try to install a monthly dev release, then install Task::CPAN::Reporter against that perl, then try to install 500+ distributions, generating CPANtesters reports along the way. The result for each distro is logged in ~/.cpanreporter/reports-sent.db. This plain-text file will grow indefinitely, but if you rename the file on a regular basis and compress it, you get something more useful. So yesterday I set out to compare two files, one (reports-sent.db.before.20240808.gz) which held results generated between the perl-5.41.2 release in July and smaller CPANtesters runs done up till August 8, the other (reports-sent.db) which held the results of a fresh run begun yesterday after better versions of Try-Tiny and Clone had been uploaded.
The distros I test include (i) approximately 500 from the top of the CPAN River; (ii) most of my own CPAN distros; (iii) a list of reverse dependencies on Variable::Magic chosen because that module is particularly sensitive to changes in Perl's internals. Note: When testing the CPAN River, I do not customarily test distributions that are shipped with core. Hence, the data reported below does not detect or report the current failures in the CPAN distribution itself. These tests were conducted on FreeBSD-13; there are a handful of distros which FAIL on that platform but PASS on Linux.
Month-to-Month Comparisons
earlier) but had a worse result yesterday (later). They are presumably adversely affected by either the "apostrophe" removal or the "switch/smartmatch" removal and need our continued attention.
{
"common-sense" => { earlier => "test PASS", later => "make UNKNOWN" },
"Crypt-CBC" => { earlier => "test PASS", later => "test DISCARD" },
"Crypt-PBKDF2" => { earlier => "test PASS", later => "test DISCARD" },
"DBIx-Class" => { earlier => "test PASS", later => "test FAIL" },
"Dist-Zilla" => { earlier => "test PASS", later => "test DISCARD" },
"JSON-XS" => { earlier => "test PASS", later => "test DISCARD" },
"Modern-Perl" => { earlier => "test PASS", later => "test FAIL" },
"Perl-PrereqScanner" => { earlier => "test PASS", later => "test DISCARD" },
"Sub-Delete" => { earlier => "test PASS", later => "test FAIL" },
"Syntax-Keyword-Junction" => { earlier => "test PASS", later => "test FAIL" },
"Test-utf8" => { earlier => "test PASS", later => "test FAIL" },
"Type-Tiny" => { earlier => "test PASS", later => "test FAIL" },
"Types-Path-Tiny" => { earlier => "test PASS", later => "test DISCARD" },
"Types-Serialiser" => { earlier => "test PASS", later => "test DISCARD" },
}{
"AnyEvent-AIO" => "test DISCARD",
"CBOR-XS" => "test DISCARD",
"common-header" => "test UNKNOWN",
"Coro" => "test DISCARD",
"DBD-mysql" => "PL UNKNOWN",
"DBD-Pg" => "PL DISCARD",
"DBICx-TestDatabase" => "test DISCARD",
"DBIx-Class-DynamicDefault" => "test DISCARD",
"DBIx-Class-InflateColumn-Object-Enum" => "test DISCARD",
"DBIx-Class-TimeStamp" => "test DISCARD",
"Devel-Events" => "test FAIL",
"Devel-Events-Objects" => "test DISCARD",
"Devel-FindRef" => "test DISCARD",
"Dios" => "test FAIL",
"Exporter-Tidy" => "test FAIL",
"IO-AIO" => "test DISCARD",
"Kavorka" => "test DISCARD",
"Lexical-Accessor" => "test DISCARD",
"Lexical-Types" => "test FAIL",
"Lexical-TypeTiny" => "test DISCARD",
"Logger-Fq" => "make UNKNOWN",
"Moops" => "test DISCARD",
"MooseX-Lexical-Types" => "test DISCARD",
"MooseX-Params" => "test FAIL",
"MooX-late" => "test DISCARD",
"MRO-Magic" => "test FAIL",
"Net--RabbitMQ" => "test FAIL",
"Parse-Keyword" => "test FAIL",
"Pcore" => "make DISCARD",
"Pinto" => "test DISCARD",
"Proc-Fork" => "test DISCARD",
"Protocol-HTTP2" => "test DISCARD",
"Ref-Store" => "make DISCARD",
"Return-Type" => "test DISCARD",
"Scope-Upper" => "make UNKNOWN",
"Sub-HandlesVia" => "test DISCARD",
"Sub-Op" => "make UNKNOWN",
"subs-auto" => "test FAIL",
"Tapper-Schema" => "test DISCARD",
"Test-Fixture-DBIC-Schema" => "test DISCARD",
"Tie-RefHash-Weak" => "test FAIL",
"TryCatch" => "test DISCARD",
"Variable-Temp" => "test DISCARD",
"X11-Protocol" => "test FAIL",
} Tuesday's dev release will trigger a new round of "testing all of CPAN" on the part of some of our heavy-duty CPANtesters. I fully expect that there will be a lot of new failures for distributions farther down the CPAN River, e.g., distros that, for whatever reason, bundled their own copies of Test::More. Such failures are of concern not just for themselves, but also for the fact that their reverse dependencies cease to be reachable in dependency-order automated testing. So while the pre-5.41.3 crisis is mostly past, we will soon have a lot of storm damage to clean up.
Tux
commented
2 months ago common::sense is part of LEHMANNWARE. Most of hist distributions are protected by Canary::Stability which causes a FAIL on any perl 5.022 and up. Most recent update was April 2020. Do not expect any fixes for modern perl.
I think this is like a 'I don't care what happens after 5.022 and you are not allowed to update anything from me that depends on this'.
https://metacpan.org/dist/Canary-Stability/source/Stability.pm#L145
https://metacpan.org/module/Canary::Stability/requires
jkeenan
commented
2 months ago
common::senseis part of LEHMANNWARE. Most of hist distributions are protected byCanary::Stabilitywhich causes a FAIL on any perl 5.022 and up. Most recent update was April 2020. Do not expect any fixes for modern perl. I think this is like a 'I don't care what happens after 5.022 and you are not allowed to update anything from me that depends on this'. https://metacpan.org/dist/Canary-Stability/source/Stability.pm#L145 https://metacpan.org/module/Canary::Stability/requires
It should be noted that Canary-Stability continues to PASS on CPANtesters; see: http://fast-matrix.cpantesters.org/?dist=Canary-Stability;perl=5.41.3;reports=1#sl=7,1
A small patch to common::sense gets it working on blead: https://rt.cpan.org/Ticket/Display.html?id=154926.
Tux
commented
2 months ago Marc disgusts RT, so I do not think he'll notice. The fact that Canary::Stability itself works does not take away the fact that using it will cause build failures. See https://metacpan.org/release/MLEHMANN/JSON-XS-4.03/source/Makefile.PL#L3
karenetheridge
commented
1 month ago re File::HomeDir::Tiny and ' as a package separator: I did https://github.com/garu/App-cpanminus-reporter/pull/42 and https://github.com/dagolden/Path-Tiny/issues/294 and https://github.com/genehack/app-gitgot/pull/78
Leont
commented
1 month ago I have several distributions that I know are broken by the smartmatch change that haven't been detected by the usual mechanisms yet. I'm worried the problem is larger than we might think it is.
jkeenan
commented
1 month ago I have several distributions that I know are broken by the smartmatch change that haven't been detected by the usual mechanisms yet. I'm worried the problem is larger than we might think it is.
Can you name them? (I've been keeping an off-list log and would like to add them if needed.)
karenetheridge
commented
1 month ago Now that 5.41.3 is out (and 5.41.4 is just a few days away), have we got an updated list of known-affected distributions?
jkeenan
commented
1 month ago Now that 5.41.3 is out (and 5.41.4 is just a few days away), have we got an updated list of known-affected distributions?
Just ask, "Is this particular distribution dependent on DBIx::Class, Type::Tiny, or common::sense?" (Type::Tiny's author has been working on a fix, but as yet there has been no CPAN release.)
Leont
commented
1 month ago Can you name them? (I've been keeping an off-list log and would like to add them if needed.)
Smart::Match, Syntax-Infix-Smartmatch (working on a fix, but that depends on #22612), threads-lite, SmartMatch-Sugar
Leont
commented
1 month ago Smart::Match, Syntax-Infix-Smartmatch (working on a fix, but that depends on https://github.com/Perl/perl5/pull/22612), threads-lite, SmartMatch-Sugar
Syntax-Infix-Smartmatch has been fixed.
karenetheridge
commented
3 weeks ago Just ask, "Is this particular distribution dependent on DBIx::Class, Type::Tiny, or common::sense?" (Type::Tiny's author has been working on a https://github.com/tobyink/p5-type-tiny/issues/160, but as yet there has been no CPAN release.)
Type::Tiny has now seen a stable release containing the fix.
It might be possible to get a new DBIx::Class fix out, given that the fix is a simple change in a test file. We would need someone with a good relationship with the author to request a release.
Re common::sense, this module is not widely used and its upstream users have alternate implementations, but it might also be possible to persuade this author to accept a patch fix.
What else is of concern?
We are experiencing considerable "Blead Breaks CPAN (BBC)" breakage on blead. This has already been reported on the Perl 5 Porters mailing list (thread starting here, and as already reported as breakage of individual CPAN distributions: Mojolicious; CPAN; and DBIx::Class.
The two most likely sources of this breakage are commits which implemented two long-scheduled removals of deprecated code. I'll refer to these as "apostrophe in name removal" (https://github.com/Perl/perl5/pull/22303) and "given/when/switch/smartmatch" (https://github.com/Perl/perl5/pull/22370). I believe a strong case can be made for reverting at least one of these two changes -- but first let me sketch the scope of the breakage.
Blead Breakage as of Aug 16 2024
Yesterday I built and installed an unthreaded
perlon Debian GNU/Linux 12 (bookworm). This was built at v5.41.2-289-gc85d5a6718:If this had been a monthly dev release, I would have next installed Task::CPAN::Reporter against that
perland proceeded to attempt to install over 600 CPAN distributions, mostly in "CPAN River" order, and generated reports for CPANtesters.org. When I did this after the perl-5.41.2 release in July, out of 607 distros I got only 19 which were logged as anything other thantest PASSin~/.cpanreporter/reports-sent.db.This was not a monthly dev release, and I did not want results from this run reported to CPANtesters. So I merely attempted to install the same 600+ distributions against c85d5a6718 and logged all the output from
./bin/cpan. The process stalled several times, had to be restarted and never completed entirely. I was, however, able to glean a list of failing distros from the top of the CPAN river (though listed here in AUTHOR/distro order):Analysis
Caveats: I have not thoroughly analyzed the reasons for the breakage on a distro-by-distro basis. I suspect that most are due to "apostrophe in name removal" and a smaller number are due to given/when/switch/smartmatch. Some of these distros could have broken in the past month even before the "apostrophe" changes were applied to blead.
Nonetheless, that's a lot of patching and CPAN-releasing to be done. Some of the distros listed there have been mainstays in production for more than 25 years. And trying to conduct two major code removals in one production cycle -- not to mention in a single monthly development cycle -- is a recipe for a lot of resentment directed toward the Porters. Yes, there have been times when we made changes which we knew would generate "a lot" of CPAN breakage (say, up to 100 distros) and we organized to quickly submit patches upstream. But here we have breakage of over 150 distros -- and that's just at the top of the CPAN river and from a presumably quite diligent subset of all CPAN authors.
Suggestions
Delay the perl-5.41.3 release scheduled for Tuesday, August 20 until we have a more thorough understanding of the scope of the breakage, have come to a consensus as to how to proceed, and have a plan to communicate with affected CPAN authors.
Consider reverting the commits associated with one or the other or both of the two removal projects. Complicating factor: Some CPAN authors have already begun to issue new release to deal with the breakage, and some of those distros that ship with core have already been synched into blead. (I have a branch which can serve as a proof-of-concept for reversion of the "apostrophe" problem.)