From 523b30316ccbf7957aa8da41729ba68b3a5f47b5 Mon Sep 17 00:00:00 2001
From: Nicholas Clark <nick@ccl4.org>
Date: Mon, 14 Apr 2008 14:38:34 +0000
Subject: Move all the logic to print the "Changed" message into regen_lib.pl,
 which removes the need for any checksums. Arrange for regen.pl to run
 autodoc.pl last, as it wants to open files generated by reentr.pl.

p4raw-id: //depot/perl@33675
---
 regen.pl | 18 +++++-------------
 1 file changed, 5 insertions(+), 13 deletions(-)

(limited to 'regen.pl')

diff --git a/regen.pl b/regen.pl
index 1a479cbed6..6670e17ad3 100644
--- a/regen.pl
+++ b/regen.pl
@@ -15,6 +15,9 @@ my $perl = $^X;
 # changes.  Um, what ?
 # safer_unlink ("warnings.h", "lib/warnings.pm");
 
+# We no longer need the values on this mapping, as the "changed" message is
+# now generated by regen_lib.pl, so should we just drop them?
+
 my %gen = (
 	   'autodoc.pl'  => [qw[pod/perlapi.pod pod/perlintern.pod]],
 	   'embed.pl'    => [qw[proto.h embed.h embedvar.h global.sym
@@ -43,20 +46,9 @@ sub do_cksum {
     return %cksum;
 }
 
-foreach my $pl (keys %gen) {
+# this puts autodoc.pl last, which can be useful as it reads reentr.c
+foreach my $pl (reverse sort keys %gen) {
   my @command =  ($^X, $pl, @ARGV);
   print "@command\n";
-  my %cksum0;
-  %cksum0 = do_cksum($pl) unless $pl eq 'warnings.pl'; # the files were removed
   system @command;
-  next if $pl eq 'warnings.pl'; # the files were removed
-  my %cksum1 = do_cksum($pl);
-  my @chg;
-  for my $f (@{ $gen{$pl} }) {
-      push(@chg, $f)
-	  if !defined($cksum0{$f}) ||
-	     !defined($cksum1{$f}) ||
-	     $cksum0{$f} ne $cksum1{$f};
-  }
-  print "Changed: @chg\n" if @chg;
 }
-- 
cgit v1.2.1