Minor improvements to perlcc

[perl5.git] / installhtml

diff --git a/installhtml b/installhtml
index d42faa1..fd11ee6 100755 (executable)
--- a/installhtml
+++ b/installhtml
@@ -1,4 +1,8 @@
-#!/usr/bin/perl -w
+#!./perl -w
+
+# This file should really be a extracted from a .PL
+
+use lib 'lib';         # use source library if present
 
 use Config;            # for config options in the makefile
 use Getopt::Long;      # for command-line parsing
@@ -32,15 +36,20 @@ pages found in the perl distribution.
 
 Displays the usage.
 
+=item B<--podroot> POD search path base directory
+
+The base directory to search for all .pod and .pm files to be converted.
+Default is current directory.
+
 =item B<--podpath> POD search path
 
 The list of directories to search for .pod and .pm files to be converted.
 Default is `podroot/.'.
 
-=item B<--podroot> POD search path base directory
+=item B<--recurse> recurse on subdirectories
 
-The base directory to search for all .pod and .pm files to be converted.
-Default is current directory.
+Whether or not to convert all .pm and .pod files found in subdirectories
+too.  Default is to not recurse.
 
 =item B<--htmldir> HTML destination directory
 
@@ -52,22 +61,22 @@ be a path relative to the filesystem, not the resulting URL.
 The base directory which all resulting HTML files will be visible at in
 a URL.  The default is `/'.
 
-=item B<--recurse> recurse on subdirectories
-
-Whether or not to convert all .pm and .pod files found in subdirectories
-too.  Default is to not recurse.
-
 =item B<--splithead> POD files to split on =head directive
 
-Colon-separated list of pod files to split by the =head directive.  These
-files should have names specified relative to podroot.
+Colon-separated list of pod files to split by the =head directive.  The
+.pod suffix is optional. These files should have names specified
+relative to podroot.
 
 =item B<--splititem> POD files to split on =item directive
 
 Colon-separated list of all pod files to split by the =item directive.
-I<installhtml> does not do the actual split, rather it invokes I<splitpod>
-to do the dirty work.  As with --splithead, these files should have names
-specified relative to podroot.
+The .pod suffix is optional.  I<installhtml> does not do the actual
+split, rather it invokes I<splitpod> to do the dirty work.  As with
+--splithead, these files should have names specified relative to podroot.
+
+=item B<--splitpod> Directory containing the splitpod program
+
+The directory containing the splitpod program. The default is `podroot/pod'.
 
 =item B<--libpods> library PODs for LE<lt>E<gt> links
 
@@ -86,14 +95,14 @@ The following command-line is an example of the one we use to convert
 perl documentation:
 
     ./installhtml --podpath=lib:ext:pod:vms   \
-                 --podroot=/usr/src/perl     \
-                  --htmldir=/perl/nmanual     \
-                 --htmlroot=/perl/nmanual    \
-                 --splithead=pod/perlipc.pod \
-                 --splititem=pod/perlfunc    \
-                 --libpods=perlfunc:perlguts:perlvar:perlrun:perlop \
-                  --recurse \
-                 --verbose
+                       --podroot=/usr/src/perl     \
+                       --htmldir=/perl/nmanual     \
+                       --htmlroot=/perl/nmanual    \
+                       --splithead=pod/perlipc     \
+                       --splititem=pod/perlfunc    \
+                       --libpods=perlfunc:perlguts:perlvar:perlrun:perlop \
+                       --recurse \
+                       --verbose
 
 =head1 AUTHOR
 
@@ -129,8 +138,8 @@ Usage: $0 --help --podpath=<name>:...:<name> --podroot=<name>
                   of a pod =head[1-6] directive.
     --splititem - comma-separated list of .pod or .pm files to split using
                   splitpod.
-    --splitpod  - where the program splitpod can be found (\$podroot/pod by
-                  default).
+    --splitpod  - directory where the program splitpod can be found
+                  (\$podroot/pod by default).
     --verbose   - self-explanatory.
 
 END_OF_USAGE
@@ -150,6 +159,7 @@ $verbose = 0;               # whether or not to print debugging info
 
 $pod2html = "pod/pod2html";
 
+usage("") unless @ARGV;
 
 # parse the command-line
 $result = GetOptions( qw(
@@ -191,7 +201,7 @@ $splitpod = "$podroot/pod" unless $splitpod;
 #  it may effect some of the links
 @splitdirs = ();    # files in these directories won't get an index
 split_on_head($podroot, $htmldir, \@splitdirs, \@ignore, @splithead);
-split_on_item($podroot, \@splitdirs, \@ignore, @splititem);
+split_on_item($podroot,           \@splitdirs, \@ignore, @splititem);
 
 
 # convert the pod pages found in @poddirs
@@ -269,6 +279,15 @@ sub parse_command_line {
 }
 
 
+sub absolute_path {
+    my($cwd, $path) = @_;
+       return "$cwd/$path" unless $path =~ m:/:;
+    # add cwd if path is not already an absolute path
+    $path = "$cwd/$path" if (substr($path,0,1) ne '/');
+    return $path;
+}
+
+
 sub create_index {
     my($html, $dir) = @_;
     my(@files, @filedata, @index, $file);
@@ -276,7 +295,7 @@ sub create_index {
     # get the list of .html files in this directory
     opendir(DIR, $dir) ||
        die "$0: error opening directory $dir for reading: $!\n";
-    @files = sort(grep(/\.html$/, readdir(DIR)));
+    @files = sort(grep(/\.html?$/, readdir(DIR)));
     closedir(DIR);
 
     open(HTML, ">$html") ||
@@ -346,6 +365,7 @@ sub split_on_item {
 
     print "splitting files by item.\n" if $verbose && $#splititem >= 0;
     $pwd = getcwd();
+       my $splitter = absolute_path($pwd, "$splitpod/splitpod");
     foreach $pod (@splititem) {
        # figure out the directory to split into
        $pod      =~ s,^([^/]*)$,/$1,;
@@ -364,8 +384,10 @@ sub split_on_item {
        }
        chdir("$podroot/$dirname") ||
            die "$0: error changing to directory $podroot/$dirname: $!\n";
-       system("../splitpod", "../$filename") &&
-           warn "$0: error running '../splitpod ../$filename'"
+       die "$splitter not found. Use '-splitpod dir' option.\n"
+           unless -f $splitter;
+       system("perl", $splitter, "../$filename") &&
+           warn "$0: error running '$splitter ../$filename'"
                 ." from $podroot/$dirname";
     }
     chdir($pwd);