Sure, when there are no more comments from reviewers I will proceed to
modify the patch and add the comments into the makefiles themselves.

Any more files need dependencies besides .tely, .texi and .itexi?


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 = \
On 2011/10/04 11:57:19, Graham Percival wrote:
On 2011/10/02 00:05:41, Julien Rioux wrote:
> Find the file $(1) within the include dirs or assume $(outdir)/$(1)
if not
found
> (in case it is a generated file).

Cute!  I like it.  :)

Something more clever might be possible but that's what I came up with.

http://codereview.appspot.com/5131045/diff/5001/stepmake/stepmake/texinfo-vars.make#newcode15
stepmake/stepmake/texinfo-vars.make:15: )
An important question, as the order matters here: When writing @include
macros.itexi in a German manual, say de/contributor.texi, I assumed that
de/macros.itexi would take precedence over macros.itexi and so
de/macros.itexi is tracked as a dependency. Does that correspond to what
texi2html and texi2pdf would do?

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

_______________________________________________
lilypond-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to