Remove incorrect function import from pgindent
authorDaniel Gustafsson
Wed, 9 Oct 2024 07:34:34 +0000 (09:34 +0200)
committerDaniel Gustafsson
Wed, 9 Oct 2024 07:34:34 +0000 (09:34 +0200)
Commit 149ac7d4559 which re-implemented pgindent in Perl explicitly
imported the devnull function from File::Spec, but the module does
not export anything.  In recent versions of Perl calling a missing
import function cause a warning, which combined with warnings being
fatal cause pgindent to error out.

Backpatch to all supported versions.

Author: Erik Wienhold 
Reviewed-by: Andrew Dunstan
Reviewed-by: Daniel Gustafsson
Discusson: https://postgr.es/m/2372cd74-11b0-46f9-b28e-8f9627215d19@ewie.name
Backpatch-through: v12

src/tools/pgindent/pgindent

index a3afb9db6453d696300911d850f6da21f752e4c7..0477a6675ad6741eff257e9a4e2c7fff91bdba6d 100755 (executable)
@@ -7,7 +7,7 @@ use warnings;
 
 use Cwd qw(abs_path getcwd);
 use File::Find;
-use File::Spec qw(devnull);
+use File::Spec;
 use File::Temp;
 use IO::Handle;
 use Getopt::Long;