Re: Make doc failure

[Carl Sorensen]

raham Percival <graham <at> percival-music.ca> writes:

> 
> On Thu, Oct 22, 2009 at 07:34:18AM -0600, Carl Sorensen wrote:
> > # fix links from other manuals to general
> > find ./out-www -name "*.html" | 
xargs sed -i 's/\.\.\/general\//\.\.\//'
> > sed: 1: "./out-www/All.html": invalid command code .
> 
> Mao.  IIRC you're on OSX?  Evidently this is a difference between
> GNU sed and BSD sed.
> 
> Out of interest, from Documentation, does
>    sed -i 's/\.\.\/general\//\.\.\//' out-www/All.html
> work?  I'm not certain if that would change anything if it does,
> since the ./out-www comes from expanding $(outdir) in the
> makefile.

I finally got around to looking at this.  I needed to change the arguments
to sed.  I made the following patch to Documentation/GNUmakefile, and
things seem to work properly.

diff --git a/Documentation/GNUmakefile b/Documentation/GNUmakefile
index 4e4bf89..dc1da96 100644
--- a/Documentation/GNUmakefile
+++ b/Documentation/GNUmakefile

 
@@ -146,8 +146,8 @@ local-WWW-1: $(OUT_TEXINFO_MANUALS) 
$(PDF_FILES) info
 
 local-WWW-2: txt-to-html $(OUT_HTML_FILES) $(DEEP_HTML_FILES) 
$(source-links) $(OM
        # fix links from other manuals to general
-       find $(outdir) -name "*.html" | xargs sed -i 's/..\/general\//..\//'
-       ls $(outdir)/*.html | xargs sed -i 's/<a href=\"..\//<a href=\"/'
+       find $(outdir) -name "*.html" | xargs sed  -e 
's/..\/general\//..\//' -i ''
+       ls $(outdir)/*.html | xargs sed  -e 's/<a href=\"..\//<a href=\"/' -i ''
 endif
 

I don't know if this will work for Linux sed (although it appears to from the 
man
pages I popped up).

If this works on Linux, I'd be happy to push it.

THanks,

Carl