Not sure what I was thinking, but 18637ebb never worked :/

While at it work around the newly-discovered RT#106935

diff --git a/lib/DBIx/Class/ResultSet/Pager.pm b/lib/DBIx/Class/ResultSet/Pager.pm
index e8510c3..e606f8a 100644 (file)
--- a/lib/DBIx/Class/ResultSet/Pager.pm
+++ b/lib/DBIx/Class/ResultSet/Pager.pm
@@ -4,6 +4,9 @@ package # hide from pause
 use warnings;
 use strict;
 
+# temporary, to load MRO::Compat, will be soon entirely rewritten anyway
+use DBIx::Class::_Util;
+
 use base 'Data::Page';
 use mro 'c3';
 

diff --git a/xt/dist/strictures.t b/xt/dist/strictures.t
index 7f67779..70efc7c 100644 (file)
--- a/xt/dist/strictures.t
+++ b/xt/dist/strictures.t
@@ -5,6 +5,8 @@ use strict;
 
 use Test::More;
 use File::Find;
+use File::Spec;
+use Config;
 use lib 't/lib';
 use DBICTest;
 
@@ -12,12 +14,24 @@ use DBICTest;
 # that are related to lib/ - then we should be able to run
 # perl -c checks (via syntax_ok), and all should just work
 my $missing_groupdeps_present = grep
-  { DBIx::Class::Optional::Dependencies->req_ok_for($_) }
+  { ! DBIx::Class::Optional::Dependencies->req_ok_for($_) }
   grep
     { $_ !~ /^ (?: test | rdbms | dist ) _ /x }
     keys %{DBIx::Class::Optional::Dependencies->req_group_list}
 ;
 
+# don't test syntax when RT#106935 is triggered (mainly CI)
+# FIXME - remove when RT is resolved
+my $tainted_relpath = (
+  length $ENV{PATH}
+    and
+  ${^TAINT}
+    and
+  grep
+    { ! File::Spec->file_name_is_absolute($_) }
+    split /\Q$Config{path_sep}/, $ENV{PATH}
+) ? 1 : 0;
+
 find({
   wanted => sub {
     -f $_ or return;
@@ -36,8 +50,13 @@ find({
     Test::Strict::strict_ok($f);
     Test::Strict::warnings_ok($f);
 
-    Test::Strict::syntax_ok($f)
-      if ! $missing_groupdeps_present and $f =~ /^ (?: lib  )/x;
+    Test::Strict::syntax_ok($f) if (
+      ! $tainted_relpath
+        and
+      ! $missing_groupdeps_present
+        and
+      $f =~ /^ (?: lib  )/x
+    );
   },
   no_chdir => 1,
 }, (qw(lib t examples maint)) );