Parcourir la source

Simplify asciidoc-helper

We don't need sed for our string manipulation, so let's get rid of
it. Suggested by weasel.
Sebastian Hahn il y a 14 ans
Parent
commit
0cd1b499b4
3 fichiers modifiés avec 11 ajouts et 8 suppressions
  1. 1 1
      ChangeLog
  2. 2 2
      doc/Makefile.am
  3. 8 5
      doc/asciidoc-helper.sh

+ 1 - 1
ChangeLog

@@ -30,7 +30,7 @@ Changes in version 0.2.2.10-alpha - 2010-??-??
   o Code simplifications and refactoring:
     - Fix some urls in the exit notice file and make it XHTML1.1 strict
       compliant. Based on a patch from Christian Kujau.
-
+    - Don't use sed in asciidoc-helper anymore.
 
 Changes in version 0.2.2.9-alpha - 2010-02-22
   o Directory authority changes:

+ 2 - 2
doc/Makefile.am

@@ -39,7 +39,7 @@ DIST_SUBDIRS = spec
 # Generate the html documentation from asciidoc, but don't do
 # machine-specific replacements yet
 $(html_in) :
-	$(top_srcdir)/doc/asciidoc-helper.sh html @ASCIIDOC@ @SED@ $(top_srcdir)/doc/$@
+	$(top_srcdir)/doc/asciidoc-helper.sh html @ASCIIDOC@ $(top_srcdir)/doc/$@
 
 tor.html.in : tor.1.txt
 torify.html.in : torify.1.txt
@@ -49,7 +49,7 @@ tor-resolve.html.in : tor-resolve.1.txt
 # Generate the manpage from asciidoc, but don't do
 # machine-specific replacements yet
 $(man_in) :
-	$(top_srcdir)/doc/asciidoc-helper.sh man @A2X@ @SED@ $(top_srcdir)/doc/$@
+	$(top_srcdir)/doc/asciidoc-helper.sh man @A2X@ $(top_srcdir)/doc/$@
 
 tor.1.in : tor.1.txt
 torify.1.in : torify.1.txt

+ 8 - 5
doc/asciidoc-helper.sh

@@ -4,19 +4,19 @@
 # See LICENSE for licensing information
 # Run this to generate .html.in or .1.in files from asciidoc files.
 # Arguments:
-# html|man asciidocpath sedpath outputfile
+# html|man asciidocpath outputfile
 
 set -e
 
-if [ $# != 4 ]; then
+if [ $# != 3 ]; then
   exit 1;
 fi
 
-output=$4
-input=`echo $output | $3 -e 's/html\.in$/1\.txt/g' -e 's/1\.in$/1\.txt/g'`
-base=`echo $output | $3 -e 's/\.html\.in$//g' -e 's/\.1\.in$//g'`
+output=$3
 
 if [ "$1" = "html" ]; then
+    input=${output%%.html.in}.1.txt
+    base=${output%%.html.in}
     if [ "$2" != none ]; then
       "$2" -d manpage -o $output $input;
     else
@@ -29,6 +29,9 @@ if [ "$1" = "html" ]; then
       echo "==================================";
     fi
 elif [ "$1" = "man" ]; then
+    input=${output%%.1.in}.1.txt
+    base=${output%%.1.in}
+    
     if test "$2" != none; then
       if $2 -f manpage $input; then
         mv $base.1 $output;