Re: Build dependencies for .texi files (issue 1852). (issue 5131045)

[julien . rioux]

This should address the comments posted at
http://codereview.appspot.com/5169041/


http://codereview.appspot.com/5131045/diff/5001/Documentation/GNUmakefile
File Documentation/GNUmakefile (left):

http://codereview.appspot.com/5131045/diff/5001/Documentation/GNUmakefile#oldcode250
Documentation/GNUmakefile:250: $(eval $(outdir)/(manual).texi: $(call
src-wildcard,$(manual)/*.ite??)))
This appears to be a previous unsuccessful attempt at tracking
dependencies: Remove.

http://codereview.appspot.com/5131045/diff/5001/Documentation/GNUmakefile
File Documentation/GNUmakefile (right):

http://codereview.appspot.com/5131045/diff/5001/Documentation/GNUmakefile#newcode234
Documentation/GNUmakefile:234: $(outdir)/web.texi:
$(outdir)/we-wrote.itexi $(outdir)/others-did.itexi
$(outdir)/weblinks.itexi
The explicit rule is necessary since weblinks.itexi is a generated file.

http://codereview.appspot.com/5131045/diff/5001/stepmake/stepmake/texinfo-rules.make
File stepmake/stepmake/texinfo-rules.make (right):

http://codereview.appspot.com/5131045/diff/5001/stepmake/stepmake/texinfo-rules.make#newcode32
stepmake/stepmake/texinfo-rules.make:32: $(DO_TEXI_DEP) cp -f $< $@
Done similarly like in stepmake/stepmake/c-rules.make

http://codereview.appspot.com/5131045/diff/5001/stepmake/stepmake/texinfo-vars.make
File stepmake/stepmake/texinfo-vars.make (right):

http://codereview.appspot.com/5131045/diff/5001/stepmake/stepmake/texinfo-vars.make#newcode11
stepmake/stepmake/texinfo-vars.make:11: find-texi = \
Find the file $(1) within the include dirs or assume $(outdir)/$(1) if
not found (in case it is a generated file).

http://codereview.appspot.com/5131045/diff/5001/stepmake/stepmake/texinfo-vars.make#newcode17
stepmake/stepmake/texinfo-vars.make:17: scan-texi = \
Recursively scan the file $(1) for @include, search for included files
in the include dirs and return all dependencies.

http://codereview.appspot.com/5131045/diff/5001/stepmake/stepmake/texinfo-vars.make#newcode23
stepmake/stepmake/texinfo-vars.make:23: DO_TEXI_DEP = ( echo ./$@:
$(call scan-texi,$<) > $(basename $@).dep ) &&
Find dependencies for the target $@, based on the source file $<, and
write them to a .dep file.

http://codereview.appspot.com/5131045/