Skip to content
This repository has been archived by the owner on Aug 4, 2024. It is now read-only.

Module::Pluggable is a deprecation error on 5.18.4 #270

Closed
timlegge opened this issue May 9, 2023 · 2 comments
Closed

Module::Pluggable is a deprecation error on 5.18.4 #270

timlegge opened this issue May 9, 2023 · 2 comments

Comments

@timlegge
Copy link

timlegge commented May 9, 2023

@exodist as discussed in IRC:

5.18.4 presents a deprecated warning that breaks some downstream module and caused failed tests for Test2::Plugin::NoWarnings fail with the following error :houseabsolute/Test2-Plugin-NoWarnings#5. It seems that installing Module::Plugable first fixes the issue.

Test Summary Report

t/basic.t (Wstat: 256 Tests: 3 Failed: 1)
Failed test: 1
Non-zero exit status: 1
Files=5, Tests=9, 0 wallclock secs ( 0.01 usr 0.00 sys + 0.54 cusr 0.07 csys = 0.62 CPU)
Result: FAIL

Looking at the results of the test:

blib/lib/ t/basic.t

Seeded srand with seed '20230508' from local date.

not ok 1 - Unexpected warning: Module::Pluggable will be removed from the Perl core distribution in the next major release. Please install it from CPAN. It is being used at /.../perl-5.18.4/lib/site_perl/5.18.4/Test2/API/InterceptResult/Event.pm, line 27.
ok 2
ok 3
1..3

cpanm Module::Pluggable
--> Working on Module::Pluggable
Fetching https://cpan.metacpan.org/authors/id/S/SI/SIMONW/Module-Pluggable-5.2.tar.gz ... OK
Configuring Module-Pluggable-5.2 ... OK
Building and testing Module-Pluggable-5.2 ... OK
Successfully installed Module-Pluggable-5.2 (upgraded from 4.7)
1 distribution installed

perl -I blib/lib/ t/basic.t

Seeded srand with seed '20230508' from local date.

ok 1
ok 2
1..2

Seems to work fine

timlegge added a commit to timlegge/perl-Net-SAML2 that referenced this issue May 11, 2023
Test-More/Test2-Suite#270

Required until the issue is fixed in Test2::Suite
Basically on 5.18.4 Module::Plugable is deprecated and
prints a warning that is eventually caught by
Test2::Plugin::NoWarnings and that causes the install of
Test2::Plugin::NoWarnings to fail.
@exodist
Copy link
Member

exodist commented Apr 25, 2024

Is this an issue with Test2-Suite? or was this a Test2::Plugin::NoWarnings that has been fixed?

@timlegge
Copy link
Author

Is this an issue with Test2-Suite? or was this a Test2::Plugin::NoWarnings that has been fixed?

According to houseabsolute/Test2-Plugin-NoWarnings#5 (comment) it is in Test2-Suite

exodist added a commit that referenced this issue Apr 25, 2024
    - Fix #292 and #270
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants