Thank you for your reviewing.


https://codereview.appspot.com/357760043/diff/20001/make/lilypond-book-rules.make
File make/lilypond-book-rules.make (right):

https://codereview.appspot.com/357760043/diff/20001/make/lilypond-book-rules.make#newcode33
make/lilypond-book-rules.make:33: mv $@ $(outdir)/$*.tmp.pdf
On 2018/07/14 10:21:51, Dan Eble wrote:
On 2018/07/14 08:22:49, trueroad wrote:
> Unfortunately, we cannnot specify an output filename for LaTeX.

How about creating the intermediate file in a different directory?
Would that
solve the problem?


Done.

https://codereview.appspot.com/357760043/diff/20001/stepmake/stepmake/texinfo-rules.make
File stepmake/stepmake/texinfo-rules.make (right):

https://codereview.appspot.com/357760043/diff/20001/stepmake/stepmake/texinfo-rules.make#newcode70
stepmake/stepmake/texinfo-rules.make:70: ifeq ($(WEB_VERSION),yes)
On 2018/07/14 10:31:33, Dan Eble wrote:
Is the only difference between the two long commands in these
alternatives "-D
web_version"?  How about something like the following?

TEXI2PDF_WEB_VERSION_FLAGS :=
ifeq ($(WEB_VERSION),yes)
TEXI2PDF_WEB_VERSION_FLAGS += -D web_version
endif

$(outdir)/%.pdf: $(outdir)/%.texi [...]
         [...] $(TEXI2PDF_FLAGS) $(TEXI2PDF_WEB_VERSION_FLAGS) [...]

Done.

https://codereview.appspot.com/357760043/

_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to