Fixes for "installhtml --splithead", based on :

Subject: Re: [PATCH 5.8.0 UTIL] Fix installhtml for splitting and PM/POD conflicts
Message-ID: <3E8ABC85.5060807@uk.radan.com>

This also includes a small patch to Pod::Html : when generating
anchor names, play on the safe side and turn every non-alphanumeric
character into an '_'. Moreover, Pod::Html::anchorify() is now 
optionally exported.

p4raw-id: //depot/perl@19136

1 files changed, 14 insertions, 15 deletions

diff --git a/installhtml b/installhtml
index 97c207917e..9884f27d3f 100755
--- a/installhtml
+++ b/installhtml
@@ -6,7 +6,7 @@ use strict;
 use Config;		# for config options in the makefile
 use Getopt::Long;	# for command-line parsing
 use Cwd;
-use Pod::Html;
+use Pod::Html 'anchorify';
 
 =head1 NAME
 
@@ -227,6 +227,7 @@ foreach my $dir (@splititem) {
 }
 
 foreach my $dir (@splithead) {
+    (my $pod = $dir) =~ s,^.*/,,;
     $dir .= ".pod" unless $dir =~ /(\.pod|\.pm)$/;
     # let pod2html create the file
     runpod2html($dir, 1);
@@ -243,15 +244,15 @@ foreach my $dir (@splithead) {
     $/ = "";
     my @data = ();
     while (<H>) {
-	last if /NAME=/;
-	$_ =~ s{HREF="#(.*)">}{
-	    my $url = "$file/$1.html" ;
-            $url = Pod::Html::relativize_url( $url, "$file.html" )
-                if ( ! defined $Options{htmlroot} || $Options{htmlroot} eq '' );
-            "HREF=\"$url\">" ;
-        }eg;
+	last if /name="name"/i;
+	$_ =~ s{href="#(.*)">}{
+	    my $url = "$pod/$1.html" ;
+	    $url = Pod::Html::relativize_url( $url, "$file.html" )
+	    if ( ! defined $Options{htmlroot} || $Options{htmlroot} eq '' );
+	    "href=\"$url\">" ;
+	}egi;
 	push @data, $_;
-    } 
+    }
     close(H);
 
     # now rewrite the file 
@@ -453,7 +454,7 @@ sub splitpod {
     #  L<> links as necessary.
     my %heads = ();
     foreach $i (0..$#poddata) {
-	$heads{htmlize($1)} = 1 if $poddata[$i] =~ /=head[1-6]\s+(.*)/;
+	$heads{anchorify($1)} = 1 if $poddata[$i] =~ /=head[1-6]\s+(.*)/;
     }
 
     # create a directory of a similar name and store all the
@@ -485,15 +486,15 @@ sub splitpod {
 	# determine an appropriate filename (this must correspond with
 	#  what pod2html will try and guess)
 	# $poddata[$i] =~ /^\s*=head[1-6]\s+(.*)/;
-	$file = "$dir/" . htmlize($section) . ".pod";
+	$file = "$dir/" . anchorify($section) . ".pod";
 
 	# create the new .pod file
 	print "\tcreating $poddir/$file\n" if $verbose;
 	open(SPLITOUT, ">$poddir/$file") ||
 	    die "$0: error opening $poddir/$file for output: $!\n";
 	$poddata[$i] =~ s,L<([^<>]*)>,
-			    defined $heads{htmlize($1)} ? "L<$dir/$1>" : "L<$1>"
-			 ,ge;
+			defined $heads{anchorify($1)} ? "L<$dir/$1>" : "L<$1>"
+		     ,ge;
 	print SPLITOUT $poddata[$i]."\n\n";
 	print SPLITOUT "=over 4\n\n";
 	print SPLITOUT "=item *\n\nBack to L<$dir/\"$prevsec\">\n\n" if $prevsec;
@@ -605,5 +606,3 @@ sub runpod2html {
 	"--infile=$podroot/$pod", "--outfile=$htmldir/$html");
     die "$0: error running $pod2html: $!\n" if $?;
 }
-
-sub htmlize { htmlify(0, @_) }