Integrate:
authorNicholas Clark <nick@ccl4.org>
Mon, 17 Nov 2008 11:58:17 +0000 (11:58 +0000)
committerNicholas Clark <nick@ccl4.org>
Mon, 17 Nov 2008 11:58:17 +0000 (11:58 +0000)
[ 34856]
Integrate:
[ 34814]
add -t (tabular option) to Porting/corecpan.pl
This allows multiple source trees to be compared
p4raw-link: @34856 on //depot/maint-5.10/perl: 336cca2641dd9041bbe56cc0012c22aa4962f153
p4raw-link: @34814 on //depot/perl: b78893c9935e40731b0134c5996886a293086d32

p4raw-id: //depot/maint-5.8/perl@34868
p4raw-integrated: from //depot/maint-5.10/perl@34863 'copy in'
Porting/corecpan.pl (@32694..)

Porting/corecpan.pl

index 0bf3913..df2a70a 100644 (file)
@@ -1,6 +1,7 @@
 #!perl
 # Reports, in a perl source tree, which dual-lived core modules have not the
 # same version than the corresponding module on CPAN.
+# with -t option, can compare multiple source trees in tabular form.
 
 use 5.9.0;
 use strict;
@@ -8,18 +9,26 @@ use Getopt::Std;
 use ExtUtils::MM_Unix;
 use lib 'Porting';
 use Maintainers qw(get_module_files %Modules);
+use Cwd;
+
+use List::Util qw(max);
 
 our $packagefile = '02packages.details.txt';
 
 sub usage () {
     die <<USAGE;
-$0 - report which core modules are outdated.
+$0
+$0 -t home1[:label] home2[:label] ...
+
+Report which core modules are outdated.
 To be run at the root of a perl source tree.
+
 Options :
 -h : help
 -v : verbose (print all versions of all files, not only those which differ)
 -f : force download of $packagefile from CPAN
      (it's expected to be found in the current directory)
+-t : display in tabular form CPAN vs one or more perl source trees
 USAGE
 }
 
@@ -30,8 +39,21 @@ sub get_package_details () {
        or die "Failed to get package details\n";
 }
 
-getopts('fhv');
+getopts('fhvt');
 our $opt_h and usage;
+our $opt_t;
+
+my @sources = @ARGV ? @ARGV : '.';
+die "Too many directories speficied without -t option\n"
+    if @sources != 1 and ! $opt_t;
+
+@sources = map {
+               # handle /home/user/perl:bleed style labels
+               my ($dir,$label) = split /:/;
+               $label = $dir unless defined $label;
+               [ $dir, $label ];
+           } @sources;
+
 our $opt_f || !-f $packagefile and get_package_details;
 
 # Load the package details. All of them.
@@ -39,31 +61,97 @@ my %cpanversions;
 open my $fh, $packagefile or die $!;
 while (<$fh>) {
     my ($p, $v) = split ' ';
+    next if 1../^\s*$/; # skip header
     $cpanversions{$p} = $v;
 }
 close $fh;
 
-for my $dist (sort keys %Modules) {
-    next unless $Modules{$dist}{CPAN};
-    print "Module $dist...\n";
-    for my $file (get_module_files($dist)) {
-       next if $file !~ /\.pm\z/ or $file =~ m{^t/} or $file =~ m{/t/};
-       my $vcore = MM->parse_version($file) // 'undef';
-       my $module = $file;
-       $module =~ s/\.pm\z//;
-       # some heuristics to figure out the module name from the file name
-       $module =~ s{^(lib|ext)/}{}
-           and $1 eq 'ext'
-           and ( $module =~ s{^(.*)/lib/\1\b}{$1},
-                 $module =~ s{(\w+)/\1\b}{$1},
-                 $module =~ s{^Encode/encoding}{encoding},
-                 $module =~ s{^MIME/Base64/QuotedPrint}{MIME/QuotedPrint},
-                 $module =~ s{^List/Util/lib/Scalar}{Scalar},
-               );
-       $module =~ s{/}{::}g;
-       my $vcpan = $cpanversions{$module} // 'not found';
-       if (our $opt_v or $vcore ne $vcpan) {
-           print "    $file: core=$vcore, cpan=$vcpan\n";
+my %results;
+
+# scan source tree(s) and CPAN module list, and put results in %results
+
+foreach my $source (@sources) {
+    my ($srcdir, $label) = @$source;
+    my $olddir = getcwd();
+    chdir $srcdir or die "chdir $srcdir: $!\n";
+
+    for my $dist (sort keys %Modules) {
+       next unless $Modules{$dist}{CPAN};
+       for my $file (get_module_files($dist)) {
+           next if $file !~ /\.pm\z/ or $file =~ m{^t/} or $file =~ m{/t/};
+           my $vcore = '!EXIST';
+           $vcore = MM->parse_version($file) // 'undef' if -f $file;
+           my $module = $file;
+           $module =~ s/\.pm\z//;
+           # some heuristics to figure out the module name from the file name
+           $module =~ s{^(lib|ext)/}{}
+               and $1 eq 'ext'
+               and ( $module =~ s{^(.*)/lib/\1\b}{$1},
+                     $module =~ s{(\w+)/\1\b}{$1},
+                     $module =~ s{^Encode/encoding}{encoding},
+                     $module =~ s{^MIME/Base64/QuotedPrint}{MIME/QuotedPrint},
+                     $module =~ s{^List/Util/lib/Scalar}{Scalar},
+                   );
+           $module =~ s{/}{::}g;
+           my $vcpan = $cpanversions{$module} // 'undef';
+           $results{$dist}{$file}{$label} = $vcore;
+           $results{$dist}{$file}{CPAN} = $vcpan;
+       }
+    }
+
+    chdir $olddir or die "chdir $olddir: $!\n";
+}
+
+# output %results in the requested format
+
+my @labels = ((map $_->[1], @sources), 'CPAN' );
+
+if ($opt_t) {
+    my %changed;
+    my @fields;
+    for my $dist (sort keys %results) {
+       for my $file (sort keys %{$results{$dist}}) {
+           my @versions = @{$results{$dist}{$file}}{@labels};
+           for (0..$#versions) {
+               $fields[$_] = max($fields[$_],
+                                 length $versions[$_],
+                                 length $labels[$_],
+                                 length '!EXIST'
+                               );
+           }
+           if (our $opt_v or grep $_ ne $versions[0], @versions) {
+               $changed{$dist} = 1;
+           }
+       }
+    }
+    printf "%*s ", $fields[$_], $labels[$_] for 0..$#labels;
+    print "\n";
+    printf "%*s ", $fields[$_], '-' x length $labels[$_] for 0..$#labels;
+    print "\n";
+
+    my $field_total;
+    $field_total += $_ + 1 for @fields;
+
+    for my $dist (sort keys %results) {
+       next unless $changed{$dist};
+       print " " x $field_total, " $dist\n";
+       for my $file (sort keys %{$results{$dist}}) {
+           my @versions = @{$results{$dist}{$file}}{@labels};
+           for (0..$#versions) {
+               printf "%*s ", $fields[$_], $versions[$_]//'!EXIST'
+           }
+           print "    $file\n";
+       }
+    }
+}
+else {
+    for my $dist (sort keys %results) {
+       print "Module $dist...\n";
+       for my $file (sort keys %{$results{$dist}}) {
+           my ($vcpan, $vcore) = @{$results{$dist}{$file}}{@labels};
+           if (our $opt_v or $vcore ne $vcpan) {
+               print "    $file: core=$vcore, cpan=$vcpan\n";
+           }
        }
     }
 }