Subject: | t/storable-test.pl coerces Storable to runtime load modules |
We patch Storable to NOT runtime load modules when it thaws something. This is an old back and forth with p5p where they basically stated that it's not safe to use Storable for this sorta thing (yes it's unsafe. don't do it)
As a result, t/storable.t is failing on our perl.
This patch would make the test pass for us if you are interested in taking it. If not feel free to reject:
diff --git a/modules/URI/URI/t/storable-test.pl b/modules/URI/URI/t/storable-test.pl
index 33deb6f..51ce137 100644
--- a/modules/URI/URI/t/storable-test.pl
+++ b/modules/URI/URI/t/storable-test.pl
@@ -2,6 +2,9 @@ use strict;
use warnings;
use Storable;
+use URI::http ();
+use URI ();
+
if (@ARGV && $ARGV[0] eq "store") {
require URI;
require URI::URL;