Skip to content

Commit

Permalink
Don't start Configure.pl without ExtUtils::Command
Browse files Browse the repository at this point in the history
Not all distros include this module, but it is needed to run the Makefile.
If a user would try and install Rakudo Star and they don't have the module,
this would obscure the missing module and they likely would not know why
the Rakudo Star compilation/installation was failing.

`use` the module in Configure.pl so the user immediately knows they must
install this module if they do not have it.
  • Loading branch information
samcv committed Sep 13, 2017
1 parent ec18efa commit 3f4a9ff
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions Configure.pl
Expand Up @@ -18,8 +18,10 @@
my $uclang = uc $lang;
my $win = $^O eq 'MSWin32';
my $slash = $win ? '\\' : '/';


# We don't use ExtUtils::Command in Configure.pl, but it is used in the Makefile
# Try `use`ing it here so users know if they need to install this module
# (not included with *every* Perl installation)
use ExtUtils::Command;
MAIN: {
if (-r 'config.default') {
unshift @ARGV, shellwords(slurp('config.default'));
Expand Down

0 comments on commit 3f4a9ff

Please sign in to comment.