Re: [HACKERS] [PATCHES] SGML index build fix

Peter Eisentraut wrote:
> Am Mittwoch, 10. Januar 2007 01:41 schrieb Bruce Momjian:
> > Peter Eisentraut wrote:
> > > Bruce Momjian wrote:
> > > > > ? %-A4.tex-ps: %.sgml $(ALLSGML) stylesheet.dsl bookindex.sgml
> > > > > ? ????$(JADE.tex.call) -V texdvi-output -V '%paper-type%'=A4 -o $@ $<
> > > > > + ifndef DRAFT
> > > > > + ????(at)cmp -s HTML.index.start HTML.index || $(MAKE) $*
> > > > > + endif
> > >
> > > What is the point of that?
> >
> > If HTML.index changed during the build, we need to rerun it until it is
> > unchanged.
>
> But that rule doesn't change HTML.index.

Now that the rule is right:

%-A4.tex-ps: %.sgml $(ALLSGML) stylesheet.dsl bookindex.sgml
$(JADE.tex.call) -V texdvi-output -V '%paper-type%'=A4 -o $@ $<
ifndef DRAFT
@cmp -s HTML.index.start HTML.index || $(MAKE) $@
endif

The rule re-runs the makefile for the specific target, and the target
modifies HTML.index, or it is only the HTML rule that modifies that.
That was a question I had. If that is true, it has to be:

%-A4.tex-ps: %.sgml $(ALLSGML) stylesheet.dsl bookindex.sgml
$(JADE.tex.call) -V texdvi-output -V '%paper-type%'=A4 -o $@ $<
ifndef DRAFT
@cmp -s HTML.index.start HTML.index || $(MAKE) html $@
endif

--
Bruce Momjian bruce(at)momjian(dot)us
EnterpriseDB http://www.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +