This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
In buildtoc, only have entries in %Build for targets being built.
[perl5.git] / pod / buildtoc
index 7b770f2..41ca6f8 100644 (file)
@@ -1,9 +1,9 @@
 #!/usr/bin/perl -w
 
 use strict;
-use vars qw($masterpodfile %Build %Targets $Verbose $Quiet $Up %Ignore
+use vars qw($masterpodfile %Build %Targets $Verbose $Quiet %Ignore
            @Master %Readmes %Pods %Aux %Readmepods %Pragmata %Modules
-           %Copies %Generated);
+           %Copies %Generated $Test);
 use File::Spec;
 use File::Find;
 use FindBin;
@@ -14,8 +14,17 @@ use Carp;
 
 no locale;
 
-$Up = File::Spec->updir;
-$masterpodfile = File::Spec->catfile($Up, "pod.lst");
+{
+  my $Top = File::Spec->catdir($FindBin::Bin, File::Spec->updir);
+
+  sub abs_from_top {
+    my $path = shift;
+    return File::Spec->catdir($Top, split /\//, $path) if $path =~ s!/\z!!;
+    return File::Spec->catfile($Top, split /\//, $path);
+  }
+}
+
+$masterpodfile = abs_from_top('pod.lst');
 
 # Generate any/all of these files
 # --verbose gives slightly more output
@@ -23,25 +32,32 @@ $masterpodfile = File::Spec->catfile($Up, "pod.lst");
 # --build-all tries to build everything
 # --build-foo updates foo as follows
 # --showfiles shows the files to be changed
+# --test exit early, exit if perl.pod, pod.lst, MANIFEST are
+#   consistent, die otherwise.
 
 %Targets
   = (
-     toc => "perltoc.pod",
-     manifest => File::Spec->catdir($Up, "MANIFEST"),
-     perlpod => "perl.pod",
-     vms => File::Spec->catfile($Up, "vms", "descrip_mms.template"),
-     nmake => File::Spec->catfile($Up, "win32", "Makefile"),
-     dmake => File::Spec->catfile($Up, "win32", "makefile.mk"),
-     podmak => File::Spec->catfile($Up, "win32", "pod.mak"),
-     # plan9 =>  File::Spec->catdir($Up, "plan9", "mkfile"),
-     unix => File::Spec->catfile($Up, "Makefile.SH"),
+     toc => 'pod/perltoc.pod',
+     manifest => 'MANIFEST',
+     perlpod => 'pod/perl.pod',
+     vms => 'vms/descrip_mms.template',
+     nmake => 'win32/Makefile',
+     dmake => 'win32/makefile.mk',
+     podmak => 'win32/pod.mak',
+     # plan9 =>  'plan9/mkfile'),
+     unix => 'Makefile.SH',
      # TODO: add roffitall
     );
 
+foreach (values %Targets) {
+  $_ = abs_from_top($_);
+}
+
 {
   my @files = keys %Targets;
   my $filesopts = join(" | ", map { "--build-$_" } "all", sort @files);
   my $showfiles;
+  my %build_these;
   die <<__USAGE__
 $0: Usage: $0 [--verbose] [--showfiles] $filesopts
 __USAGE__
@@ -49,9 +65,15 @@ __USAGE__
        && GetOptions (verbose => \$Verbose,
                       quiet => \$Quiet,
                       showfiles => \$showfiles,
-                      map {+"build-$_", \$Build{$_}} @files, 'all');
-  # Set them all to true
-  @Build{@files} = @files if ($Build{all});
+                      test => \$Test,
+                      map {+"build-$_", \$build_these{$_}} @files, 'all');
+  if ($build_these{all}) {
+    %Build = %Targets;
+  } else {
+    while (my ($file, $want) = each %build_these) {
+      $Build{$file} = $Targets{$file} if $want;
+    }
+  }
   if ($showfiles) {
       print
          join(" ",
@@ -64,7 +86,7 @@ __USAGE__
                   File::Spec->catfile(@d ?
                                       (@d == 1 && $d[0] eq '' ? () : @d)
                                       : "pod", $f);
-              } @Targets{grep { $_ ne 'all' && $Build{$_} } keys %Build}),
+              } @Targets{@files}),
           "\n";
       exit(0);
   }
@@ -78,11 +100,9 @@ __USAGE__
      );
 
 if ($Verbose) {
-  print "I'm building $_\n" foreach grep {$Build{$_}} keys %Build;
+  print "I'm building $_\n" foreach keys %Build;
 }
 
-chdir $FindBin::Bin or die "$0: Can't chdir $FindBin::Bin: $!";
-
 open MASTER, $masterpodfile or die "$0: Can't open $masterpodfile: $!";
 
 my ($delta_source, $delta_target);
@@ -111,11 +131,11 @@ foreach (<MASTER>) {
     $flags{aux} = 1 if $flags =~ tr/a//d;
 
     if ($flags =~ tr/D//d) {
-      $flags{perlpod_omit} = 1;
+      $flags{manifest_omit} = 1;
       $delta_source = "$filename.pod";
     }
     if ($flags =~ tr/d//d) {
-      $flags{manifest_omit} = 1;
+      $flags{perlpod_omit} = 1;
       $delta_target = "$filename.pod";
     }
     $Generated{"$filename.pod"}++ if $flags =~ tr/g//d;
@@ -148,7 +168,7 @@ if (defined $delta_source) {
     die "$0: delta source defined but not target";
   }
 } elsif (defined $delta_target) {
-  die "$0: delta target defined but not target";
+  die "$0: delta target defined but not source";
 }
 
 close MASTER;
@@ -160,23 +180,25 @@ close MASTER;
   my (@manireadmes, %manireadmes);
   my (@perlpods, %perlpods);
   my (%our_pods);
-  my (%sources);
 
   # Convert these to a list of filenames.
   foreach (keys %Pods, keys %Readmepods) {
     $our_pods{"$_.pod"}++;
   }
 
-  # None of these filenames will be boolean false
-  @disk_pods = glob("*.pod");
-  @disk_pods{@disk_pods} = @disk_pods;
+  opendir my $dh, abs_from_top('pod/');
+  while (readdir $dh) {
+    next unless /\.pod\z/;
+    push @disk_pods, $_;
+    ++$disk_pods{$_};
+  }
 
   # Things we copy from won't be in perl.pod
   # Things we copy to won't be in MANIFEST
-  @sources{values %Copies} = ();
 
-  open(MANI, "../MANIFEST") || die "$0: opening ../MANIFEST failed: $!";
-  while (<MANI>) {
+  my $filename = abs_from_top('MANIFEST');
+  open my $mani, '<', $filename or die "$0: opening $filename failed: $!";
+  while (<$mani>) {
     if (m!^pod/([^.]+\.pod)\s+!i) {
       push @manipods, $1;
     } elsif (m!^README\.(\S+)\s+!i) {
@@ -184,56 +206,73 @@ close MASTER;
       push @manireadmes, "perl$1.pod";
     }
   }
-  close(MANI);
+  close $mani or die $!;
   @manipods{@manipods} = @manipods;
   @manireadmes{@manireadmes} = @manireadmes;
 
-  open(PERLPOD, "perl.pod") || die "$0: opening perl.pod failed: $!\n";
-  while (<PERLPOD>) {
+  $filename = abs_from_top('pod/perl.pod');
+  open my $perlpod, '<', $filename or die "$0: opening $filename failed: $!\n";
+  while (<$perlpod>) {
     if (/^For ease of access, /../^\(If you're intending /) {
       if (/^\s+(perl\S*)\s+\w/) {
        push @perlpods, "$1.pod";
       }
     }
   }
-  close(PERLPOD);
+  close $perlpod or die $!;
   die "$0: could not find the pod listing of perl.pod\n"
     unless @perlpods;
   @perlpods{@perlpods} = @perlpods;
 
+  my @inconsistent;
   foreach my $i (sort keys %disk_pods) {
-    warn "$0: $i exists but is unknown by buildtoc\n"
+    push @inconsistent, "$0: $i exists but is unknown by buildtoc\n"
       unless $our_pods{$i};
-    warn "$0: $i exists but is unknown by ../MANIFEST\n"
+    push @inconsistent, "$0: $i exists but is unknown by ../MANIFEST\n"
       if !$manipods{$i} && !$manireadmes{$i} && !$Copies{$i} && !$Generated{$i};
-    warn "$0: $i exists but is unknown by perl.pod\n"
-       if !$perlpods{$i} && !exists $sources{$i};
+    push @inconsistent, "$0: $i exists but is unknown by perl.pod\n"
+       if !$perlpods{$i} && !exists $Copies{$i};
   }
+  my %BuildFiles;
+  ++$BuildFiles{$_} foreach values %Build;
+
   foreach my $i (sort keys %our_pods) {
-    warn "$0: $i is known by buildtoc but does not exist\n"
-      unless $disk_pods{$i};
+    push @inconsistent, "$0: $i is known by buildtoc but does not exist\n"
+      unless $disk_pods{$i} or $BuildFiles{$i};
   }
   foreach my $i (sort keys %manipods) {
-    warn "$0: $i is known by ../MANIFEST but does not exist\n"
+    push @inconsistent, "$0: $i is known by ../MANIFEST but does not exist\n"
       unless $disk_pods{$i};
-    warn "$0: $i is known by ../MANIFEST but is marked as generated\n"
+    push @inconsistent, "$0: $i is known by ../MANIFEST but is marked as generated\n"
       if $Generated{$i};
   }
   foreach my $i (sort keys %perlpods) {
-    warn "$0: $i is known by perl.pod but does not exist\n"
-      unless $disk_pods{$i};
+    push @inconsistent, "$0: $i is known by perl.pod but does not exist\n"
+      unless $disk_pods{$i} or $BuildFiles{$i};
+  }
+  if ($Test) {
+    print "1..1\n";
+    if (@inconsistent) {
+      print "not ok 1\n";
+      die @inconsistent
+    }
+    print "ok 1\n";
+    exit;
+  }
+  else {
+    warn @inconsistent if @inconsistent;
   }
 }
 
 # Find all the modules
-{
+if ($Build{toc}) {
   my @modpods;
-  find \&getpods => qw(../lib ../ext);
+  find \&getpods => abs_from_top('lib/');
 
   sub getpods {
     if (/\.p(od|m)$/) {
       my $file = $File::Find::name;
-      return if $file eq '../lib/Pod/Functions.pm'; # Used only by pod itself
+      return if $file =~ qr!/lib/Pod/Functions.pm\z!; # Used only by pod itself
       return if $file =~ m!(?:^|/)t/!;
       return if $file =~ m!lib/Attribute/Handlers/demo/!;
       return if $file =~ m!lib/Net/FTP/.+\.pm!; # Hi, Graham! :-)
@@ -242,7 +281,6 @@ close MASTER;
       return if $file =~ m!XS/(?:APItest|Typemap)!;
       my $pod = $file;
       return if $pod =~ s/pm$/pod/ && -e $pod;
-      die "$0: tut $File::Find::name" if $file =~ /TUT/;
       unless (open (F, "< $_\0")) {
        warn "$0: bogus <$file>: $!";
        system "ls", "-l", $file;
@@ -252,7 +290,6 @@ close MASTER;
        while ($line = <F>) {
          if ($line =~ /^=head1\s+NAME\b/) {
            push @modpods, $file;
-           #warn "GOOD $file\n";
            close F;
            return;
          }
@@ -267,15 +304,15 @@ close MASTER;
 
   my %done;
   for (@modpods) {
-    #($name) = /(\w+)\.p(m|od)$/;
-    my $name = path2modname($_);
+    my $name = $_;
+    $name =~ s/\.p(m|od)$//;
+    $name =~ s-.*?/lib/--;
+    $name =~ s-/-::-g;
+    next if $done{$name}++;
+
     if ($name =~ /^[a-z]/) {
       $Pragmata{$name} = $_;
     } else {
-      if ($done{$name}++) {
-       # warn "already did $_\n";
-       next;
-      }
       $Modules{$name} = $_;
     }
   }
@@ -284,23 +321,12 @@ close MASTER;
 # OK. Now a lot of ancillary function definitions follow
 # Main program returns at "Do stuff"
 
-sub path2modname {
-    local $_ = shift;
-    s/\.p(m|od)$//;
-    s-.*?/(lib|ext)/--;
-    s-/-::-g;
-    s/(\w+)::\1/$1/;
-    return $_;
-}
-
-sub output ($);
-
-sub output_perltoc {
-  open(OUT, ">perltoc.pod") || die "$0: creating perltoc.pod failed: $!";
+my $OUT;
 
-  local $/ = '';
+sub do_toc {
+  my $filename = shift;
 
-  ($_= <<"EOPOD2B") =~ s/^\t//gm && output($_);
+  ($_= <<"EOPOD2B") =~ s/^\t//gm and $OUT .= $_;
 
        # !!!!!!!   DO NOT EDIT THIS FILE   !!!!!!!
        # This file is autogenerated by buildtoc from all the other pods.
@@ -322,10 +348,12 @@ EOPOD2B
 #' make emacs happy
 
   # All the things in the master list that happen to be pod filenames
-  podset(map {"$_->[1].pod"} grep {defined $_ && @$_ == 3 && !$_->[0]{toc_omit}} @Master);
+  foreach (grep {defined $_ && @$_ == 3 && !$_->[0]{toc_omit}} @Master) {
+    podset($_->[1], abs_from_top("pod/$_->[1].pod"));
+  }
 
 
-  ($_= <<"EOPOD2B") =~ s/^\t//gm && output($_);
+  ($_= <<"EOPOD2B") =~ s/^\t//gm and $OUT .= $_;
 
 
 
@@ -333,9 +361,11 @@ EOPOD2B
 
 EOPOD2B
 
-  podset(sort values %Pragmata);
+  foreach (sort keys %Pragmata) {
+    podset($_, $Pragmata{$_});
+  }
 
-  ($_= <<"EOPOD2B") =~ s/^\t//gm && output($_);
+  ($_= <<"EOPOD2B") =~ s/^\t//gm and $OUT .= $_;
 
 
 
@@ -343,7 +373,9 @@ EOPOD2B
 
 EOPOD2B
 
-  podset( @Modules{ sort keys %Modules } );
+  foreach (sort keys %Modules) {
+    podset($_, $Modules{$_});
+  }
 
   $_= <<"EOPOD2B";
 
@@ -371,9 +403,14 @@ EOPOD2B
 EOPOD2B
 
   s/^\t//gm;
-  output $_;
-  output "\n";                    # flush $LINE
-  close OUT;
+  $OUT .= "$_\n";
+
+  $OUT =~ s/\n\s+\n/\n\n/gs;
+  $OUT =~ s/\n{3,}/\n\n/g;
+
+  $OUT =~ s/([^\n]+)/wrap('', '', $1)/ge;
+
+  return $OUT;
 }
 
 # Below are all the auxiliary routines for generating perltoc.pod
@@ -381,42 +418,39 @@ EOPOD2B
 my ($inhead1, $inhead2, $initem);
 
 sub podset {
-    local @ARGV = @_;
-    my $pod;
+    my ($pod, $file) = @_;
+
+    local $/ = '';
 
-    return unless scalar(@ARGV);
+    open my $fh, '<', $file or die "Can't open file '$file' for $pod: $!";
 
-    while(<>) {
+    while(<$fh>) {
        tr/\015//d;
        if (s/^=head1 (NAME)\s*/=head2 /) {
-           $pod = path2modname($ARGV);
            unhead1();
-           output "\n \n\n=head2 ";
-           $_ = <>;
+           $OUT .= "\n\n=head2 ";
+           $_ = <$fh>;
            # Remove svn keyword expansions from the Perl FAQ
            s/ \(\$Revision: \d+ \$\)//g;
-           if ( /^\s*$pod\b/ ) {
+           if ( /^\s*\Q$pod\E\b/ ) {
                s/$pod\.pm/$pod/;       # '.pm' in NAME !?
-               output $_;
            } else {
                s/^/$pod, /;
-               output $_;
            }
-           next;
        }
-       if (s/^=head1 (.*)/=item $1/) {
+       elsif (s/^=head1 (.*)/=item $1/) {
            unhead2();
-           output "=over 4\n\n" unless $inhead1;
+           $OUT .= "=over 4\n\n" unless $inhead1;
            $inhead1 = 1;
-           output $_; nl(); next;
+           $_ .= "\n";
        }
-       if (s/^=head2 (.*)/=item $1/) {
+       elsif (s/^=head2 (.*)/=item $1/) {
            unitem();
-           output "=over 4\n\n" unless $inhead2;
+           $OUT .= "=over 4\n\n" unless $inhead2;
            $inhead2 = 1;
-           output $_; nl(); next;
+           $_ .= "\n";
        }
-       if (s/^=item ([^=].*)/$1/) {
+       elsif (s/^=item ([^=].*)/$1/) {
            next if $pod eq 'perldiag';
            s/^\s*\*\s*$// && next;
            s/^\s*\*\s*//;
@@ -424,24 +458,23 @@ sub podset {
            s/\s+$//;
            next if /^[\d.]+$/;
            next if $pod eq 'perlmodlib' && /^ftp:/;
-           ##print "=over 4\n\n" unless $initem;
-           output ", " if $initem;
+           $OUT .= ", " if $initem;
            $initem = 1;
            s/\.$//;
            s/^-X\b/-I<X>/;
-           output $_; next;
        }
-       if (s/^=cut\s*\n//) {
-           unhead1();
+       else {
+           unhead1() if /^=cut\s*\n/;
            next;
        }
+       $OUT .= $_;
     }
 }
 
 sub unhead1 {
     unhead2();
     if ($inhead1) {
-       output "\n\n=back\n\n";
+       $OUT .= "\n\n=back\n\n";
     }
     $inhead1 = 0;
 }
@@ -449,45 +482,18 @@ sub unhead1 {
 sub unhead2 {
     unitem();
     if ($inhead2) {
-       output "\n\n=back\n\n";
+       $OUT .= "\n\n=back\n\n";
     }
     $inhead2 = 0;
 }
 
 sub unitem {
     if ($initem) {
-       output "\n\n";
-       ##print "\n\n=back\n\n";
+       $OUT .= "\n\n";
     }
     $initem = 0;
 }
 
-sub nl {
-    output "\n";
-}
-
-my $NEWLINE = 0;       # how many newlines have we seen recently
-my $LINE;              # what remains to be printed
-
-sub output ($) {
-    for (split /(\n)/, shift) {
-       if ($_ eq "\n") {
-           if ($LINE) {
-               print OUT wrap('', '', $LINE);
-               $LINE = '';
-           }
-           if (($NEWLINE) < 2) {
-               print OUT;
-               $NEWLINE++;
-           }
-       }
-       elsif (/\S/ && length) {
-           $LINE .= $_;
-           $NEWLINE = 0;
-       }
-    }
-}
-
 # End of original buildtoc. From here on are routines to generate new sections
 # for and inplace edit other files
 
@@ -524,10 +530,10 @@ sub generate_perlpod {
 
 
 sub generate_manifest {
-  # Annyoingly unexpand doesn't consider it good form to replace a single
+  # Annoyingly, unexpand doesn't consider it good form to replace a single
   # space before a tab with a tab
   # Annoyingly (2) it returns read only values.
-  my @temp = unexpand (map {sprintf "%-32s%s\n", @$_} @_);
+  my @temp = unexpand (map {sprintf "%-32s%s", @$_} @_);
   map {s/ \t/\t\t/g; $_} @temp;
 }
 sub generate_manifest_pod {
@@ -569,11 +575,6 @@ SNIP
    sort keys %Pods, keys %Readmepods;
 }
 
-sub generate_descrip_mms_3 {
-  map qq{\t- If F\$Search("[.pod]$_").nes."" Then Delete/NoConfirm/Log [.pod]$_;*},
-    sort keys %Generated, keys %Copies;
-}
-
 sub generate_nmake_1 {
   # XXX Fix this with File::Spec
   (map {sprintf "\tcopy ..\\README.%-8s ..\\pod\\perl$_.pod\n", $_}
@@ -589,6 +590,7 @@ sub generate_nmake_2 {
                   join " ", sort keys %Copies, keys %Generated,
                                  map {"perl$_.pod"} keys %Readmes);
   $line =~ s/$/ \\/mg;
+  $line =~ s/ \\$//;
   $line;
 }
 
@@ -610,31 +612,30 @@ sub verify_contiguous {
 }
 
 sub do_manifest {
-  my $name = shift;
+  my ($name, $prev) = @_;
   my @manifest =
-    grep {! m!^pod/[^.]+\.pod.*\n!}
-      grep {! m!^README\.(\S+)! || $Ignore{$1}} @_;
-  # Dictionary order - fold and handle non-word chars as nothing
-  map  { $_->[0] }
-  sort { $a->[1] cmp $b->[1] || $a->[0] cmp $b->[0] }
-  map  { my $f = lc $_; $f =~ s/[^a-z0-9\s]//g; [ $_, $f ] }
-    @manifest,
-      &generate_manifest_pod(),
-       &generate_manifest_readme();
+    grep {! m!^pod/[^.]+\.pod.*!}
+      grep {! m!^README\.(\S+)! || $Ignore{$1}} split "\n", $prev;
+  join "\n", (
+             # Dictionary order - fold and handle non-word chars as nothing
+             map  { $_->[0] }
+             sort { $a->[1] cmp $b->[1] || $a->[0] cmp $b->[0] }
+             map  { my $f = lc $_; $f =~ s/[^a-z0-9\s]//g; [ $_, $f ] }
+             @manifest,
+             &generate_manifest_pod(),
+             &generate_manifest_readme()), '';
 }
 
 sub do_nmake {
-  my $name = shift;
-  my $makefile = join '', @_;
-  die "$0: $name contains NUL bytes" if $makefile =~ /\0/;
+  my ($name, $makefile) = @_;
   $makefile =~ s/^\tcopy \.\.\\README.*\n/\0/gm;
   verify_contiguous($name, $makefile, 'README copies');
   # Now remove the other copies that follow
   1 while $makefile =~ s/\0\tcopy .*\n/\0/gm;
   $makefile =~ s/\0+/join ("", &generate_nmake_1)/se;
 
-  $makefile =~ s{(del /f [^\n]+podchecker[^\n]+).*?(pod2html)}
-    {"$1\n" . &generate_nmake_2."\n\t    $2"}se;
+  $makefile =~ s{(-cd \$\(PODDIR\) && del /f[^\n]+).*?(-cd \.\.\\utils && del /f)}
+    {"$1\n" . &generate_nmake_2."\n\t$2"}se;
   $makefile;
 }
 
@@ -642,8 +643,7 @@ sub do_nmake {
 *do_dmake = *do_dmake = \&do_nmake;
 
 sub do_perlpod {
-  my $name = shift;
-  my $pod = join '', @_;
+  my ($name, $pod) = @_;
 
   unless ($pod =~ s{(For\ ease\ of\ access,\ .*\n)
                    (?:\s+[a-z]{4,}.*\n #   fooo
@@ -658,8 +658,7 @@ sub do_perlpod {
 }
 
 sub do_podmak {
-  my $name = shift;
-  my $body = join '', @_;
+  my ($name, $body) = @_;
   foreach my $variable (qw(pod man html tex)) {
     die "$0: could not find $variable in $name"
       unless $body =~ s{\n\U$variable\E = (?:[^\n]*\\\n)*[^\n]*}
@@ -669,15 +668,21 @@ sub do_podmak {
 }
 
 sub do_vms {
-  my $name = shift;
-  my $makefile = join '', @_;
-  die "$0: $name contains NUL bytes" if $makefile =~ /\0/;
+  my ($name, $makefile) = @_;
   $makefile =~ s/\npod\d* =[^\n]*/\0/gs;
   verify_contiguous($name, $makefile, 'pod assignments');
   $makefile =~ s/\0+/join "\n", '', &generate_descrip_mms_1/se;
 
   die "$0: $name contains NUL bytes" if $makefile =~ /\0/;
 
+# Looking for the macro defining the current perldelta:
+#PERLDELTA_CURRENT = [.pod]perl5139delta.pod
+
+  $makefile =~ s/\nPERLDELTA_CURRENT\s+=\s+\Q[.pod]perl\E\d+delta\.pod\n
+                   /\0/sx;
+  verify_contiguous($name, $makefile, 'current perldelta macro');
+  $makefile =~ s/\0+/join "\n", '', "PERLDELTA_CURRENT = [.pod]$delta_target", ''/se;
+
 # Looking for rules like this
 # [.lib.pods]perl.pod : [.pod]perl.pod
 #      @ If F$Search("[.lib]pods.dir").eqs."" Then Create/Directory [.lib.pods]
@@ -690,27 +695,19 @@ sub do_vms {
   verify_contiguous($name, $makefile, 'copy rules');
   $makefile =~ s/\0+/join "\n", '', &generate_descrip_mms_2/se;
 
-# Looking for rules like this:
-#      - If F$Search("[.pod]perldelta.pod").nes."" Then Delete/NoConfirm/Log [.pod]perldelta.pod;*
-  $makefile =~ s!(?:\t- If F\$Search\("\[\.pod\]perl[a-z]+\Q.pod").nes."" Then Delete/NoConfirm/Log [.pod]perl\E[a-z]+\.pod;\*\n)+!\0!sg;
-  verify_contiguous($name, $makefile, 'delete rules');
-  $makefile =~ s/\0+/join "\n", &generate_descrip_mms_3, ''/se;
-
   $makefile;
 }
 
 sub do_unix {
-  my $name = shift;
-  my $makefile_SH = join '', @_;
-  die "$0: $name contains NUL bytes" if $makefile_SH =~ /\0/;
+  my ($name, $makefile_SH) = @_;
 
   $makefile_SH =~ s{^(perltoc_pod_prereqs = extra.pods).*}
                   {join ' ', $1, map "pod/$_",
                        sort keys %Copies, grep {!/perltoc/} keys %Generated
                    }mge;
 
-# pod/perldelta.pod: pod/perl511delta.pod
-#      cd pod && $(LNS) perl511delta.pod perldelta.pod
+# pod/perl511delta.pod: pod/perldelta.pod
+#      cd pod && $(LNS) perldelta.pod perl511delta.pod
 
   $makefile_SH =~ s!(
 pod/perl[a-z0-9_]+\.pod: pod/perl[a-z0-9_]+\.pod
@@ -733,32 +730,41 @@ pod/$_: pod/$Copies{$_}
 
 my $built;
 while (my ($target, $name) = each %Targets) {
+  print "Working on target $target\n" if $Verbose;
   next unless $Build{$target};
   $built++;
-  if ($target eq "toc") {
-    print "Now processing $name\n" if $Verbose;
-    &output_perltoc;
-    print "Finished\n" if $Verbose;
-    next;
-  }
+  my ($orig, $mode);
   print "Now processing $name\n" if $Verbose;
-  open THING, $name or die "Can't open $name: $!";
-  my @orig = <THING>;
-  my $orig = join '', @orig;
-  close THING;
-  my @new = do {
+  if ($target ne "toc") {
+    local $/;
+    open THING, $name or die "Can't open $name: $!";
+    binmode THING;
+    $orig = <THING>;
+    close THING;
+    die "$0: $name contains NUL bytes" if $orig =~ /\0/;
+  }
+
+  my $new = do {
     no strict 'refs';
-    &{"do_$target"}($target, @orig);
+    &{"do_$target"}($target, $orig);
   };
-  my $new = join '', @new;
-  if ($new eq $orig) {
-    print "Was not modified\n" if $Verbose;
-    next;
+
+  if (defined $orig) {
+    if ($new eq $orig) {
+      print "Was not modified\n" if $Verbose;
+      next;
+    }
+    $mode = (stat $name)[2] // die "$0: Can't stat $name: $!";
+    rename $name, "$name.old" or die "$0: Can't rename $name to $name.old: $!";
   }
-  rename $name, "$name.old" or die "$0: Can't rename $name to $name.old: $!";
+
   open THING, ">$name" or die "$0: Can't open $name for writing: $!";
+  binmode THING;
   print THING $new or die "$0: print to $name failed: $!";
-  close THING or die die "$0: close $name failed: $!";
+  close THING or die "$0: close $name failed: $!";
+  if (defined $mode) {
+    chmod $mode & 0777, $name or die "$0: can't chmod $mode $name: $!";
+  }
 }
 
 warn "$0: was not instructed to build anything\n" unless $built;