This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Pass the git tag to cmpVERSION.pl with a --tag argument.
[perl5.git] / Porting / cmpVERSION.pl
old mode 100644 (file)
new mode 100755 (executable)
index 22241c4..ec6b869
@@ -1,13 +1,11 @@
 #!/usr/bin/perl -w
 
-#
 # cmpVERSION - compare the current Perl source tree and a given tag
 # for modules that have identical version numbers but different contents.
 #
 # with -d option, output the diffs too
 # with -x option, exclude dual-life modules (after all, there are tools
 #                 like core-cpan-diff that can already deal with them)
-#                 With this option, one of the directories must be '.'.
 #
 # Original by slaven@rezic.de, modified by jhi and matt.w.johnson@gmail.com.
 #
@@ -16,49 +14,48 @@ use strict;
 
 use ExtUtils::MakeMaker;
 use File::Compare;
-use File::Find;
-use File::Spec::Functions qw(rel2abs abs2rel catfile catdir curdir);
-use Getopt::Std;
+use File::Spec::Functions qw(devnull);
+use Getopt::Long;
+
+my ($diffs, $exclude_dual, $tag_to_compare);
+unless (GetOptions('diffs' => \$diffs,
+                  'exclude|x' => \$exclude_dual,
+                  'tag=s' => \$tag_to_compare,
+                  ) && @ARGV == 0) {
+    die "usage: $0 [ -d -x --tag TAG]";
+}
 
-use lib 'Porting';
-use Maintainers;
+die "$0: This does not look like a Perl directory\n"
+    unless -f "perl.h" && -d "Porting";
+die "$0: 'This is a Perl directory but does not look like Git working directory\n"
+    unless -d ".git";
 
-sub usage {
-die <<"EOF";
-usage: $0 [ -d -x ] source_dir tag_to_compare
-EOF
-}
+my $null = devnull();
 
-my %opts;
-getopts('dx', \%opts) or usage;
-@ARGV == 2 or usage;
+unless (defined $tag_to_compare) {
+    # Thanks to David Golden for this suggestion.
 
-my ($source_dir, $tag_to_compare) = @ARGV[0,1];
-die "$0: '$source_dir' does not look like a Perl directory\n"
-    unless -f catfile($source_dir, "perl.h") && -d catdir($source_dir, "Porting");
-die "$0: '$source_dir' is a Perl directory but does not look like Git working directory\n"
-    unless -d catdir($source_dir, ".git");
+    $tag_to_compare = `git describe --abbrev=0`;
+    chomp $tag_to_compare;
+}
 
-my $tag_exists = `git --no-pager tag -l $tag_to_compare 2>/dev/null`;
+my $tag_exists = `git --no-pager tag -l $tag_to_compare 2>$null`;
 chomp $tag_exists;
 
 die "$0: '$tag_to_compare' is not a known Git tag\n"
     unless $tag_exists eq $tag_to_compare;
 
 my %dual_files;
-if ($opts{x}) {
-    die "With -x, the directory must be '.'\n"
-       unless $source_dir eq '.';
-    for my $m (grep $Maintainers::Modules{$_}{CPAN},
-                               keys %Maintainers::Modules)
-    {
+if ($exclude_dual) {
+    unshift @INC, 'Porting';
+    require Maintainers;
 
+    for my $m (grep $Maintainers::Modules{$_}{CPAN},
+                               keys %Maintainers::Modules) {
        $dual_files{$_} = 1 for Maintainers::get_module_files($m);
     }
 }
 
-chdir $source_dir or die "$0: chdir '$source_dir' failed: $!\n";
-
 # Files to skip from the check for one reason or another,
 # usually because they pull in their version from some other file.
 my %skip;
@@ -78,7 +75,7 @@ my @module_diffs = grep {
     $this_dir = $1 if m/^(.*)\//;
     /\.pm$/ &&
     (!defined($this_dir) || ($this_dir !~ $skip_dirs)) &&
-    !exists $skip{$_} && 
+    !exists $skip{$_} &&
     !exists $dual_files{$_}
 } @all_diffs;
 
@@ -97,10 +94,9 @@ foreach my $pm_file (@module_diffs) {
     my $pm_version = eval {MM->parse_version($pm_file)};
     my $orig_pm_content = get_file_from_git($pm_file, $tag_to_compare);
     my $orig_pm_version = eval {MM->parse_version(\$orig_pm_content)};
-    next unless
-       defined $pm_version &&
-       defined $orig_pm_version &&
-        $pm_version eq $orig_pm_version;
+    next if ( ! defined $pm_version || ! defined $orig_pm_version );
+    next if ( $pm_version eq 'undef' || $orig_pm_version eq 'undef' ); # sigh
+    next if $pm_version ne $orig_pm_version;
     push @output_files, $pm_file;
     push @output_diffs, $pm_file unless $pm_eq;
     push @output_diffs, $xs_file unless $xs_eq;
@@ -108,7 +104,7 @@ foreach my $pm_file (@module_diffs) {
 
 sub compare_git_file {
     my ($file, $tag) = @_;
-    open(my $orig_fh, "-|", "git --no-pager show $tag:$file 2>/dev/null");
+    open(my $orig_fh, "-|", "git --no-pager show $tag:$file 2>$null");
     return undef if eof($orig_fh);
     my $is_eq = compare($file, $orig_fh) == 0;
     close($orig_fh);
@@ -118,7 +114,7 @@ sub compare_git_file {
 sub get_file_from_git {
     my ($file, $tag) = @_;
     local $/ = undef;
-    my $file_content = `git --no-pager show $tag:$file 2>/dev/null`;
+    my $file_content = `git --no-pager show $tag:$file 2>$null`;
     return $file_content;
 }
 
@@ -126,7 +122,7 @@ for (sort @output_files) {
     print "$_\n";
 }
 
-exit unless $opts{d};
+exit unless $diffs;
 
 for (sort @output_diffs) {
     print "\n";