Skip to content

Commit f4a775e

Browse files
committed
test-dist-modules.pl: make separate installation the default
This avoids the dangerous default of overwriting installed modules.
1 parent 1f92987 commit f4a775e

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

.github/workflows/testsuite.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -853,5 +853,5 @@ jobs:
853853
run: /usr/bin/perl -V
854854
- name: Build and test dist modules
855855
run:
856-
/usr/bin/perl Porting/test-dist-modules.pl -continue -separate
856+
/usr/bin/perl Porting/test-dist-modules.pl -continue
857857

Porting/test-dist-modules.pl

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,16 @@
1111

1212
my $continue;
1313
my $separate;
14+
my $install;
1415
GetOptions("c|continue" => \$continue,
1516
"s|separate" => \$separate,
17+
"i|install" => \$install,
1618
"h|help" => \&usage)
1719
or die "Unknown options\n";
1820

21+
$separate
22+
and warn "-s / -separate is now the default\n";
23+
1924
$|++;
2025

2126
-f "Configure"
@@ -28,7 +33,7 @@
2833

2934
my @config;
3035
my $install_path;
31-
if ($separate) {
36+
unless ($install) {
3237
# require EU::MM 6.31 or later
3338
my $install_base = tempdir( CLEANUP => 1 );
3439
push @config, "INSTALL_BASE=$install_base";

0 commit comments

Comments
 (0)