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/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel
