Re: zsh and line breaks

["Peter A. Castro" <doctor@xxxxxxxxxxxx>]

On Thu, 15 Apr 2004, Clint Adams wrote:

> > The use of the shell wildcard (zsh*.html) makes the shell do a pattern
> > match against the list of files in the directory in which it is run.  If
> > I was building from a work directory (say zsh-4.2.0/.build/Doc) and
> > installing into a destination root other that "/" (say zsh-4.2.0/.inst)
> > make executes this bit of shell code and it fails to find any zsh*.html
> > files because make isn't currently in that directory (it's in
> > .build/Doc).  Prepending ${sdir} ensures the zsh*.html files are
>
> Yes, and the html files are generated in the build directory, so it
> won't find them in ${sdir}.

Ah!  Now I see what's going on.  I've been applying the -doc tar over the
source directory which gives me the .html files in the source dir and not
in the build dir, and since they are the same date or newer as the
zsh.texi source, no processing of it into html is done.  Hmmm... Ok, this
is will break a full build (sans -doc), I see that now.  So, you are
correct, the ${sdir} reference should be removed.  Sorry!  And, thanks
for pointing it out!

I'll have to decide what I want to do for the Cygwin port, but certainly
this should be corrected in cvs.  So, what's the procedure?  Do I need to
submit a patch to undo my original patch or can someone with cvs access
(Peter S?) simply back that part of the patch out?

> > So how, exactly, does this break VPATH?  Can you give me an example?
> > I'll try and correct it, but I don't yet see how it's broken.
>
> Is Cygwin texi2html outputting to your source directory?

Well, this has been part of the problem.  See texi2html is "back-level"
for cygwin, so I end up having to change the makefile to specify
different options for texi2html to actually work.  I'd gotten around that
by just overlaying the -doc tar (plus it was faster than regenerating
it), but I completely forgot about the fact that the man and html files
are generated and should rightly be created in the build dir, not the
source dir.  I'll update my build process for Cygwin to do the right
thing.  In the mean time, that change in cvs should be backed out.

-- 
Peter A. Castro <doctor@xxxxxxxxxxxx> or <Peter.Castro@xxxxxxxxxx>
	"Cats are just autistic Dogs" -- Dr. Tony Attwood