xslate / p5-Text-Xslate

Scalable template engine for Perl5
https://metacpan.org/release/Text-Xslate
Other
121 stars 47 forks source link

BUG FIX: Arguments list too long #203

Closed greeneg closed 4 years ago

greeneg commented 4 years ago

This PR addresses Issue #202 by using the blib/lib directory instead of inheriting the lengthy @INC that can be found on a machine that has numerous module build directories in it's CPAN build cache.

Closes #202

Signed-off-by: Gary Greene greeneg@tolharadys.net

skaji commented 4 years ago

I think we may remove $libs. Does this work for you?

diff --git t/900_bugs/032_issue79.t t/900_bugs/032_issue79.t
index 88f37c07..ba81f715 100644
--- t/900_bugs/032_issue79.t
+++ t/900_bugs/032_issue79.t
@@ -15,13 +15,7 @@ my $dir = "$Bin/issue79";

 rmtree "$dir/cache";

-my %std_inc = map { $_ => 1 } (".", @Config::Config{qw(
-    sitelibexp sitearchexp
-    privlibexp archlibexp
-)});
-my $libs = join " ", map { qq{"-I$_" } } grep { !$std_inc{$_} } @INC;
-
-my $run_cmd = qq{$^X $libs "$dir/xslate.pl"};
+my $run_cmd = qq{$^X "$dir/xslate.pl"};
 note $run_cmd;

 my @tmpls = qw/ contentA.tt contentB.tt /;
greeneg commented 4 years ago

Yes, that also works