[prev in list] [next in list] [prev in thread] [next in thread] 

List:       lilypond-user
Subject:    Re: PDF file gets "-1" appended
From:       Knute Snortum <ksnortum () gmail ! com>
Date:       2023-06-30 20:25:10
Message-ID: CALmeJxRGm8dmxXJw7Dn3heREDbmEYcJE6NYSOzaYp-8UkF-9UA () mail ! gmail ! com
[Download RAW message or body]

On Fri, Jun 30, 2023 at 1:16 PM Jean Abou Samra <jean@abou-samra.fr> wrote:

> Le mardi 27 juin 2023 à 21:17 +0200, Valentin Petzel a écrit :
>
> Hello Knute,
>
> so you are using books to allow specification of the midi filename. This
> is
> probably a fine usecase, but it still seems like a bit of an abuse of the
> book
> mechanic to me. Rather I'd adapt the midi output name logic itself.
>
> This code adapts the internal function responsible for writing out the
> midi
> performances (scm/midi.scm:write-performances-midis) in such a way that
>
>
>
> #(set-current-module (resolve-module '(lily))) is true lock picking :-) I
> mean, sometimes you really can't avoid monkeypatching internals, but I'd
> reserve it for desperate cases.
>

Thanks for the insight.  Is there a better way to do this?

--
Knute Snortum

[Attachment #3 (text/html)]

<div dir="ltr"><div dir="ltr">On Fri, Jun 30, 2023 at 1:16 PM Jean Abou Samra \
&lt;<a href="mailto:jean@abou-samra.fr">jean@abou-samra.fr</a>&gt; \
wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" \
style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex"><div class="msg-6573307245330249331"><div><div>Le \
mardi 27 juin 2023 à 21:17 +0200, Valentin Petzel a écrit  :</div><blockquote \
type="cite" style="margin:0px 0px 0px 0.8ex;border-left:2px solid \
rgb(114,159,207);padding-left:1ex"><div>Hello Knute,<br></div><div><br></div><div>so \
you are using books to allow specification of the midi filename. This is \
<br></div><div>probably a fine usecase, but it still seems like a bit of an abuse of \
the book <br></div><div>mechanic to me. Rather I'd adapt the midi output name logic \
itself.<br></div><div><br></div><div>This code adapts the internal function \
responsible for writing out the midi <br></div><div>performances \
(scm/midi.scm:write-performances-midis) in such a way \
that</div></blockquote><div><br></div><div><br></div><div>#(set-current-module \
(resolve-module &#39;(lily))) is true lock picking :-) I mean, sometimes you really \
can&#39;t avoid monkeypatching internals, but I&#39;d reserve it for desperate \
cases.</div></div></div></blockquote><div><br></div><div>Thanks for the insight.   Is \
there a better way to do this?</div><div><br></div><div>--</div><div>Knute \
Snortum</div><div>  <br></div></div></div>



[prev in list] [next in list] [prev in thread] [next in thread]