New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bleadperl v5.13.9-540-g0a0b6c9 breaks IROBERTS/Crypt-OpenSSL-RSA-0.26.tar.gz #11140
Comments
From @andkgit bisect: 0a0b6c9 is the first bad commit example fail report: Not yet available on cpantesters... PERL_DL_NONLAZY=1 /home/src/perl/repoperls/installed-perls/perl/v5.13.9-540-g0a0b6c9/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0, 'blib/lib', 'blib/arch')" t/*.t Test Summary Report perl -V: Summary of my perl5 (revision 5 version 13 subversion 9) configuration: Characteristics of this binary (from libperl): -- |
From @nwc10On Sat, Feb 19, 2011 at 10:05:38AM -0800, Andreas J. Koenig via RT wrote:
This is going to be a judgment call. The module in question relies on the Arguably the module is buggy and needs fixing with explicit inheritance. Nicholas Clark |
The RT System itself - Status changed from 'new' to 'open' |
From @nwc10On Sat, Feb 19, 2011 at 06:20:43PM +0000, Nicholas Clark wrote:
And how much notice they need to change it. Nicholas Clark |
From @LeontOn Sat, Feb 19, 2011 at 7:20 PM, Nicholas Clark <nick@ccl4.org> wrote:
I think cleanest short-term solution would be simply reinserting this *AUTOLOAD = \&AutoLoader::AUTOLOAD; It won't make DynaLoader derived classes autoloaded, but it will make Leon |
From vadim.konovalov@alcatel-lucent.com
as a humble perl user, I very much prefer to have as little AutoLoader/AutoSplit as possible. I very much like 0a0b6c9 commit
Best regards, |
From @cpansproutOn Sat Feb 19 12:48:25 2011, LeonT wrote:
I don’t understand. Apart from moving subroutines back below the What’s the different between DynaLoader-derived classes’ being (I, for one, think this new ‘bug’ [not autoloading] is perfectly fine |
From @LeontOn Sun, Feb 20, 2011 at 10:15 PM, Father Chrysostomos via RT
I wasn't thinking when I posted that. You're completely right. This is
I agree. The behavior of AutoLoader is perfectly well described and Leon |
From Mark.Martinec@ijs.siI just came across this bug report, after spending time on fixing it first :) The problem is an inappropriate use of AutoLoader, contrary to its require AutoLoader; whereas it should be: use AutoLoader 'AUTOLOAD'; This is the only change needed for this module to get it working man AutoLoader: Subroutine Stubs The AutoSplit and AutoLoader modules automate the creation of forward Because of this mechanism it is important that AutoLoader is always Using AutoLoader's AUTOLOAD Subroutine use AutoLoader 'AUTOLOAD'; Mark |
From @cpansproutSince this is a problem with the CPAN module, I’m marking it as rejected. There is a CPAN RT ticket at |
@cpansprout - Status changed from 'open' to 'rejected' |
Migrated from rt.perl.org#84444 (status was 'rejected')
Searchable as RT84444$
The text was updated successfully, but these errors were encountered: