Remove incorrect function import from pgindent
authorDaniel Gustafsson <[email protected]>
Wed, 9 Oct 2024 07:34:34 +0000 (09:34 +0200)
committerDaniel Gustafsson <[email protected]>
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 <[email protected]>
Reviewed-by: Andrew Dunstan <[email protected]>
Reviewed-by: Daniel Gustafsson <[email protected]>
Discusson: https://postgr.es/m/2372cd74-11b0-46f9-b28e-8f9627215d19@ewie.name
Backpatch-through: v12

src/tools/pgindent/pgindent

index 77c8118e5d797d20f13f5993ff23cb7d9ab0f583..e889af6b1e49240994b4969dcd3a33ab770bf30b 100755 (executable)
@@ -7,7 +7,7 @@ use warnings FATAL => 'all';
 
 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;