-
Notifications
You must be signed in to change notification settings - Fork 589
Open
Labels
cpan-dual-lifeissues regarding dual-life cpan-first distributionsissues regarding dual-life cpan-first distributionsdo not mergeDon't merge this PR, at least for nowDon't merge this PR, at least for now
Description
As discussed in #15866 (comment), we plan (at some date TBD) to move the code for AutoLoader to be fully "blead-upstream." Currently the code resides under the cpan/
directory within the core distribution; that would imply that the code is maintained "cpan-upstream." It turns out that that has never been the case. From the documentation to lib/AutoSplit.pm
we see:
This package has been part of the perl core since the first release
of perl5. It has been released separately to CPAN so older installations
can benefit from bug fixes.
So AutoLoader's code should have been moved to reside under dist/
years ago. Moreover, its CPAN release contains no separate repository information, so there's probably no version control history that we need to retain.
This ticket exists to track that correction. We need to:
- Move the AutoLoader distribution from
cpan/
todist/
. - Review the code -- particularly with respect to operating systems we no longer support -- and its test coverage.
- Address the issue originally raised in AutoLoader implies suitability for constants, fails to mention inlining incompatibility [patch] #15866: "AutoLoader implies suitability for constants, fails to mention inlining incompability". Evaluate the patch submitted therein.
- Evaluate the sole remaining bug report in RT: https://rt.cpan.org/Ticket/Display.html?id=45679
- Recruit someone to be the releasor of AutoLoader to CPAN (assuming we're going to maintain compatibility with older perls).
Thank you very much.
Jim Keenan
Metadata
Metadata
Assignees
Labels
cpan-dual-lifeissues regarding dual-life cpan-first distributionsissues regarding dual-life cpan-first distributionsdo not mergeDon't merge this PR, at least for nowDon't merge this PR, at least for now