From 5c6d4121244650834aaa0e80ea273869589596d3 Mon Sep 17 00:00:00 2001 From: Russell Jenkins Date: Wed, 30 Oct 2024 21:48:42 +1100 Subject: [PATCH 1/2] Add Module::Pluggable as a requirement, excluding broken versions Module::Pluggable added checks for unreadable files in v6.1 and v6.2. Those tests fail when installed as root, which may happen in a docker file, such as under github actions. While we have no explicit dependency on Module::Pluggable, our dependencies do, such as Config::Any. This is a bit of a crutch until Module::Plugable is patched. Upstream issue: https://github.com/simonwistow/Module-Pluggable/issues/27 --- cpanfile | 1 + 1 file changed, 1 insertion(+) diff --git a/cpanfile b/cpanfile index 07f3246bd..c092e9a3d 100644 --- a/cpanfile +++ b/cpanfile @@ -22,6 +22,7 @@ requires 'Import::Into'; requires 'JSON::MaybeXS'; requires 'List::Util', '1.29'; # 1.29 has the pair* functions requires 'MIME::Base64', '3.13'; # 3.13 has the URL safe variants +requires 'Module::Pluggable', '!= 6.1, != 6.2'; # 6.1 and 6.2 fail their test suite when run as root requires 'Module::Runtime'; requires 'Moo', '2.000000'; requires 'Moo::Role'; From e5790b9f5cf8e8e063d798677707e848408d6ed0 Mon Sep 17 00:00:00 2001 From: Russell Jenkins Date: Wed, 30 Oct 2024 22:08:53 +1100 Subject: [PATCH 2/2] Alternate way to specify we do not want Module::Pluggable 6.1 or 6.2 This way we do not have to bump EUMM --- cpanfile | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/cpanfile b/cpanfile index c092e9a3d..6638c4e2c 100644 --- a/cpanfile +++ b/cpanfile @@ -22,7 +22,6 @@ requires 'Import::Into'; requires 'JSON::MaybeXS'; requires 'List::Util', '1.29'; # 1.29 has the pair* functions requires 'MIME::Base64', '3.13'; # 3.13 has the URL safe variants -requires 'Module::Pluggable', '!= 6.1, != 6.2'; # 6.1 and 6.2 fail their test suite when run as root requires 'Module::Runtime'; requires 'Moo', '2.000000'; requires 'Moo::Role'; @@ -48,6 +47,12 @@ requires 'File::Which'; requires 'Role::Tiny', '2.000000'; conflicts 'Role::Tiny', '== 2.000007'; +# Module::Pluggable 6.1 and 6.2 fail their test suite when run as root, +# such as under docker in a github action +requires 'Module::Pluggable'; +conflicts 'Module::Pluggable', '== 6.1'; +conflicts 'Module::Pluggable', '== 6.2'; + # Minimum version of YAML is needed due to: # - https://github.com/PerlDancer/Dancer2/issues/899 # Excluded 1.16 is needs due to: