]> git.ipfire.org Git - thirdparty/automake.git/commitdiff
automake: just use warnings, with no => FATAL.
authorKarl Berry <karl@freefriends.org>
Fri, 31 Jan 2025 17:20:47 +0000 (09:20 -0800)
committerKarl Berry <karl@freefriends.org>
Fri, 31 Jan 2025 17:20:47 +0000 (09:20 -0800)
Per Perl recommendation:
  https://perldoc.perl.org/warnings#Fatal-Warnings
Suggested by Collin Funk:
  https://lists.gnu.org/archive/html/automake/2025-01/msg00003.html
And in the Perl discussion:
  https://github.com/Perl/perl5/issues/22954#issuecomment-2622966302

* bin/aclocal.in: just use warnings, not making them fatal.
* bin/automake.in:
* contrib/tap-driver.pl:
* gen-testsuite-part:
* lib/Automake/ChannelDefs.pm:
* lib/Automake/Channels.pm:
* lib/Automake/Condition.pm:
* lib/Automake/Config.in:
* lib/Automake/Configure_ac.pm:
* lib/Automake/DisjConditions.pm:
* lib/Automake/FileUtils.pm:
* lib/Automake/General.pm:
* lib/Automake/Getopt.pm:
* lib/Automake/Item.pm:
* lib/Automake/ItemDef.pm:
* lib/Automake/Language.pm:
* lib/Automake/Location.pm:
* lib/Automake/Options.pm:
* lib/Automake/Rule.pm:
* lib/Automake/RuleDef.pm:
* lib/Automake/VarDef.pm:
* lib/Automake/Variable.pm:
* lib/Automake/Version.pm:
* lib/Automake/Wrap.pm:
* lib/Automake/XFile.pm:
* t/ax/deltree.pl:
* t/ax/extract-testsuite-summary.pl:
* t/check-fd-redirect.sh:
* t/tap-signal.tap:
* t/tests-environment-fd-redirect.sh:
* t/testsuite-summary-count-many.sh:

31 files changed:
bin/aclocal.in
bin/automake.in
contrib/tap-driver.pl
gen-testsuite-part
lib/Automake/ChannelDefs.pm
lib/Automake/Channels.pm
lib/Automake/Condition.pm
lib/Automake/Config.in
lib/Automake/Configure_ac.pm
lib/Automake/DisjConditions.pm
lib/Automake/FileUtils.pm
lib/Automake/General.pm
lib/Automake/Getopt.pm
lib/Automake/Item.pm
lib/Automake/ItemDef.pm
lib/Automake/Language.pm
lib/Automake/Location.pm
lib/Automake/Options.pm
lib/Automake/Rule.pm
lib/Automake/RuleDef.pm
lib/Automake/VarDef.pm
lib/Automake/Variable.pm
lib/Automake/Version.pm
lib/Automake/Wrap.pm
lib/Automake/XFile.pm
t/ax/deltree.pl
t/ax/extract-testsuite-summary.pl
t/check-fd-redirect.sh
t/tap-signal.tap
t/tests-environment-fd-redirect.sh
t/testsuite-summary-count-many.sh

index f69a355bef6b3c2e94b89928e5eb79e6302fb45d..05ec14a30c71ae3176bdf1afe0b2b7102119db07 100644 (file)
@@ -19,9 +19,7 @@
 # Written by Tom Tromey <tromey@redhat.com>, and
 # Alexandre Duret-Lutz <adl@gnu.org>.
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
+use 5.006; use strict; use warnings;
 
 BEGIN
 {
index a3b74f4859a091ce652a06e2656acb72983242c2..c08b83972f8560e570c274cdcb745bcf3d40140e 100644 (file)
@@ -22,9 +22,7 @@
 
 package Automake;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
+use 5.006; use strict; use warnings;
 
 BEGIN
 {
index 86a5874d21f22fdab283b4cf231092ea3995e743..8706deb92f494982c5603d1fb05ca5c096a405da 100755 (executable)
 #  Imports, static data, and setup.  #
 # ---------------------------------- #
 
-use warnings FATAL => 'all';
-use strict;
+use strict; use warnings;
 use Getopt::Long ();
+
 use TAP::Parser;
 
-my $VERSION = '2024-12-03.03'; # UTC
+my $VERSION = '2025-01-31.17'; # UTC
 
 my $ME = "tap-driver.pl";
 
index 9bc3efa05de8be8ca3791e8a88cf8469ef89ec88..b60626ae18a410305cec448bb4279aec5907c682 100755 (executable)
@@ -20,8 +20,8 @@
 
 #--------------------------------------------------------------------------
 
-use warnings FATAL => "all";
 use strict;
+use warnings;
 use File::Basename ();
 use constant TRUE => 1;
 use constant FALSE => 0;
index d9c43174d2cd8d11f2687f3c3f1b371c787b8d19..2a3e7d284dbcfc27e72ca23d671b3e8ff6f92572 100644 (file)
@@ -49,10 +49,7 @@ shorthand function to output on specific channels.
 
 =cut
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
-
+use 5.006; use strict; use warnings;
 use Exporter;
 
 use Automake::Channels;
index 3d848c45b711cb71b6eb75f74def7e6aeec773b9..0fba4fbdbca8486fef46eb2e374a6740a9ef61bb 100644 (file)
@@ -66,10 +66,7 @@ etc.) that can also be overridden on a per-message basis.
 
 =cut
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
-
+use 5.006; use strict; use warnings;
 use Carp;
 use Exporter;
 use File::Basename;
index c376ee377d21de06215a2e5bdc8d435b72226876..1bf66eb2abf099887aadc691172d312c561b1114 100644 (file)
 
 package Automake::Condition;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
-
+use 5.006; use strict; use warnings;
 use Carp;
 use Exporter;
 
index 2a0db49ee06e376cbafb1c24a1a19f9277bac677..35112c22c357a65d6bc9de2fff97801a0266c4cd 100644 (file)
@@ -17,9 +17,7 @@
 
 package Automake::Config;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
+use 5.006; use strict; use warnings;
 
 use Exporter;
 
index 598a8d91fec338d344c01d3bff7f7b5f98693378..e9cb921a693ee5583187f315d8e84534ac1a5b80 100644 (file)
@@ -20,9 +20,7 @@
 
 package Automake::Configure_ac;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
+use 5.006; use strict; use warnings;
 
 use Exporter;
 
index 4310e4df638c6ea3d567502593f6a7e3a6e6f81c..dd179782c7be5c5dad3b9e3c2e750f68ec1d0e58 100644 (file)
@@ -15,9 +15,7 @@
 
 package Automake::DisjConditions;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
+use 5.006; use strict; use warnings;
 
 use Carp;
 use Automake::Condition qw (TRUE FALSE);
index c91c36a03dd2047da547ea284141b6039486cc1a..49658bda20beef9fe38f3c32f1a5538195c24055 100644 (file)
@@ -34,9 +34,7 @@ This perl module provides various general purpose file handling functions.
 
 =cut
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
+use 5.006; use strict; use warnings;
 
 BEGIN
 {
index d544468b5e090422fff8397acb3441d3358b249f..528931f9de1b63c73cd949a7c130ad9f7c41a971 100644 (file)
@@ -15,9 +15,7 @@
 
 package Automake::General;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
+use 5.006; use strict; use warnings;
 
 use Exporter;
 use File::Basename;
index c55d6bad5d21f414098a26862feb2acf8601e0f8..445bade27e353c4d75d58b134ac97942e2ae1858 100644 (file)
@@ -35,9 +35,7 @@ line options in conformance to the GNU Coding standards.
 
 =cut
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
+use 5.006; use strict; use warnings;
 
 use Carp qw (confess croak);
 use Exporter ();
index 9f2d31bd809174514179e6b04b413842c04d655e..e8d06d6b89fc2e743fbc5d03f02a8aa0dde54b7d 100644 (file)
@@ -15,9 +15,7 @@
 
 package Automake::Item;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
+use 5.006; use strict; use warnings;
 
 use Carp;
 
index 2dae1240a9c945bff7dff02c210ce323b7bc007e..de9b9b494da695daace146ffa8fd8fb35ed2c2f8 100644 (file)
 
 package Automake::ItemDef;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
-
+use 5.006; use strict; use warnings;
 use Carp;
 
 =head1 NAME
index 77cc6b7100c51da78c1251a3a1a49fc3b4fe674b..1a4c6eee438c025a003eb9235baf455cbf204040 100644 (file)
@@ -15,9 +15,7 @@
 
 package Automake::Language;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
+use 5.006; use strict; use warnings;
 
 use Class::Struct ();
 
index 365efd320499466541012528fdadb1869d4b28e7..f504016388cd9d865b9ce040321110abf5fae73d 100644 (file)
@@ -15,9 +15,7 @@
 
 package Automake::Location;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
+use 5.006; use strict; use warnings;
 
 =head1 NAME
 
index afd476011684101ccf50c0b3c715f2b9373da93b..dd0f17a15d81b62b63b6dd630365384cd7df8927 100644 (file)
 
 package Automake::Options;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
-
+use 5.006; use strict; use warnings;
 use Exporter;
 
 use Automake::Config;
index ebdd217e99e9288b418d3d172845bea4947f84e3..6a59ce4a3b09ab6ff8b5d1a5af8fa1e0252699e0 100644 (file)
 
 package Automake::Rule;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
-
+use 5.006; use strict; use warnings;
 use Carp;
 use Exporter;
 
index 79ca298400872d95237dc79b7fe1c8fd230bc3a4..04fcdea00cf5cc37364ea8c074ca72c79b97e5d0 100644 (file)
 
 package Automake::RuleDef;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
-
+use 5.006; use strict; use warnings;
 use Carp;
 use Exporter;
 
index 7508598d81079e78351c6b51d6219ca6088a640b..f4f8950217b4dfe3d45b448ad9f2ec7600727c43 100644 (file)
 
 package Automake::VarDef;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
-
+use 5.006; use strict; use warnings;
 use Carp;
 use Exporter;
 
index 6e90b81f232cc78c0f1a2b440e11502ae3412d34..a949f68d9b12a1c2b4ac4eaca0318eaa44e65c96 100644 (file)
 
 package Automake::Variable;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
-
+use 5.006; use strict; use warnings;
 use Carp;
 use Exporter;
 
index 5de709aca1085752adb3f4d5ca4875c3562a0f7b..cb73532b4f543a8114f2ad4367c9c525dd20a521 100644 (file)
@@ -15,9 +15,7 @@
 
 package Automake::Version;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
+use 5.006; use strict; use warnings;
 
 use Automake::ChannelDefs;
 
index 4f611c6e2969c4b2f08e2e878db10d3be40852ae..aef131c4541c85a786c33596c93079a0cb9bcf80 100644 (file)
 
 package Automake::Wrap;
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
-
+use 5.006; use strict; use warnings;
 use Exporter;
 
 our @ISA = qw (Exporter);
index 544df4ddb4363009b0ca09d1761a68381f57c1a4..85153362e7e2bccce1929d016fad532cf32e3b48 100644 (file)
@@ -69,10 +69,7 @@ and C<getlines> methods to translate C<\r\n> to C<\n>.
 
 =cut
 
-use 5.006;
-use strict;
-use warnings FATAL => 'all';
-
+use 5.006; use strict; use warnings;
 use Errno;
 use Exporter;
 use IO::File;
index d8590abb928250ebb6b3def5254889530436b809..53f1fb85dd0300d4332202d38f440ab336d3f30f 100644 (file)
@@ -17,8 +17,7 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
-use strict;
-use warnings FATAL => 'all';
+use strict; use warnings;
 use File::Path qw/rmtree/;
 
 my $exit_status = 0;
index 3881cea328ed0ea6b9803c8ac5c70b52fffd073f..b1bedcdee2a62a7fb18cb742607e4afe314c9237 100644 (file)
@@ -17,8 +17,7 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
-use warnings FATAL => 'all';
-use strict;
+use strict; use warnings;
 
 my $br = '=' x 76;
 my @sections = ('');
index f76b2fa7d1ae3a71d4a12ca2adae7350a66caa63..f2eab71dc9540ba20f4e6c674c875eb5452a4f87 100644 (file)
@@ -81,8 +81,7 @@ do_check
 
 echo "#! $PERL -w" > foo.test
 cat >> foo.test <<'END'
-use warnings FATAL => 'all';
-use strict;
+use strict; use warnings;
 
 open (FD3, "<&=3") or die "opening FD3: $!";
 open (FD4, ">&=4") or die "opening FD4: $!";
index db47b892e9b2fac0adf61a1183d8f3b428361a31..3484a46ef59db5e1d1ee071d73f22f74e6784286 100644 (file)
@@ -52,7 +52,7 @@ for sig in $all_signals; do
     # We need autoflush to avoid losing output, which could cause spurious
     # "no test plan seen" in the TAP driver.
     BEGIN { $| = 1 }
-    use warnings FATAL => "all";
+    use warnings;
     print "1..1\\n";
     print "ok 1\\n";
     kill $sig, \$\$;
index 4de70ec2b9e20f27048763ff6bb84a34a67ff909..5a4e5a03edfed6fad604cd975224401dfcd3fbec 100644 (file)
@@ -44,8 +44,8 @@ END
 
 echo "#! $PERL -w" > bar.test
 cat >>bar.test <<'END'
-use warnings FATAL => 'all';
 use strict;
+use warnings;
 open(FD8, ">&=8") or die "$!";
 open(FD9, ">&=9") or die "$!";
 print FD8 "  $0: 8888\n";
index 0b9358f2d09995d970e4f999d67af5ed3bf60176..9e74f358675de56bb9c56ba706e2db7da478fcc9 100644 (file)
@@ -52,8 +52,8 @@ END
 chmod a+x all.test
 
 $PERL -w -e '
-  use warnings FATAL => "all";
   use strict;
+  use warnings;
 
   my $base = 1000;
   my %count = (