From de7c8dbea1a17a0e1709c4b12371615d28e21c13 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Sat, 2 Jan 2016 14:45:21 -0500 Subject: [PATCH] Make copyright.pl cope with nonstandard case choices in copyright notices. The need for this is shown by the files it missed in Bruce's recent run. I fixed it so that it will actually adjust the case when needed. In passing, also make it skip .po files, since those will just get overwritten anyway from the translation repository. --- src/tools/copyright.pl | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/src/tools/copyright.pl b/src/tools/copyright.pl index f52150b124..07d260b0b1 100755 --- a/src/tools/copyright.pl +++ b/src/tools/copyright.pl @@ -17,7 +17,8 @@ use File::Basename; use Tie::File; my $pgdg = 'PostgreSQL Global Development Group'; -my $cc = 'Copyright \(c\) '; +my $cc = 'Copyright \(c\)'; +my $ccliteral = 'Copyright (c)'; # year-1900 is what localtime(time) puts in element 5 my $year = 1900 + ${ [ localtime(time) ] }[5]; @@ -40,7 +41,7 @@ sub wanted # skip file names with binary extensions # How are these updated? bjm 2012-01-02 - return if ($_ =~ m/\.(ico|bin)$/); + return if ($_ =~ m/\.(ico|bin|po)$/); my @lines; tie @lines, "Tie::File", $File::Find::name; @@ -49,16 +50,16 @@ sub wanted { # We only care about lines with a copyright notice. - next unless $line =~ m/$cc.*$pgdg/; + next unless $line =~ m/$cc.*$pgdg/i; # Skip line if already matches the current year; if not # we get $year-$year, e.g. 2012-2012 - next if $line =~ m/$cc$year, $pgdg/; + next if $line =~ m/$cc $year, $pgdg/i; # We process all lines because some files have copyright # strings embedded in them, e.g. src/bin/psql/help.c - $line =~ s/($cc\d{4})(, $pgdg)/$1-$year$2/; - $line =~ s/($cc\d{4})-\d{4}(, $pgdg)/$1-$year$2/; + $line =~ s/$cc (\d{4})-\d{4}, $pgdg/$ccliteral $1-$year, $pgdg/i; + $line =~ s/$cc (\d{4}), $pgdg/$ccliteral $1-$year, $pgdg/i; } untie @lines; } -- 2.40.0