Re: Issue 5381: Change intermediate PDF filename (issue 357760043 by address@hidden)

[trueroad]

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/