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
> > > > > + ????@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@momjian.us
  EnterpriseDB    http://www.enterprisedb.com

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