Issue 3778: Use bounding box as skylines for markup in svg backend (issue 582010043 by barr...@gmail.com)

2020-04-28 Thread lemzwerg--- via Discussions on LilyPond development
LGTM https://codereview.appspot.com/582010043/

Re: Add `-dfont-ps-resdir` option to embed fonts to PDFs later (issue 577900043 by truer...@gmail.com)

2020-04-28 Thread trueroad
Thank you for your reviewing. https://codereview.appspot.com/577900043/diff/577910043/Documentation/usage/running.itely File Documentation/usage/running.itely (right): https://codereview.appspot.com/577900043/diff/577910043/Documentation/usage/running.itely#newcode652

Re: poll: switching our development platform

2020-04-28 Thread antlists
On 21/04/2020 18:54, Jonas Hahnfeld wrote: Okay, so let me try to briefly summarize the expressed opinions on the list so far (correct me if I got something wrong or missed a point): Han-Wen is in favor of Gerrit; David leans towards GitLab; Federico and me are in favor of GitLab; Werner is fine

Re: poll: switching our development platform

2020-04-28 Thread Dan Eble
On Apr 23, 2020, at 16:24, Jonas Hahnfeld wrote: >> I'd prefer file-names and thumbnails, unfolded by clicking. >> >> Is this behaviour adjustable? > > No, I'm merely linking the uploaded file (GitLab decides to display the > image inline) and AFAICS there are no options to influence this >

Re: [trial] getting started

2020-04-28 Thread Federico Bruni
Il giorno lun 27 apr 2020 alle 22:36, Carl Sorensen ha scritto: git checkout $RELEASE_BRANCH if [ 0 -eq $(git merge-base --is-ancestor $COMMIT_ID HEAD) ]; then echo "true"; else echo "false"; fi with $RELEASE_BRANCH being the name of the branch, and $COMMIT_ID replaced with the commit id

Re: [trial] getting started

2020-04-28 Thread Federico Bruni
Il giorno mar 28 apr 2020 alle 09:02, Jonas Hahnfeld ha scritto: Am Dienstag, den 28.04.2020, 00:27 +0200 schrieb Federico Bruni: Il giorno mer 22 apr 2020 alle 14:28, Jonas Hahnfeld ha scritto: > here we go: > https://gitlab.com/lilypond-issues/lilypond-trial > > To reiterate: This

Re: [PATCH v1] Issue 3778: Use bounding box as skylines for markup in svg backend

2020-04-28 Thread Kevin Barry
Hi Jonas, > Done (hopefully). Thank you very much; it looks like it worked! Kevin

Re: [PATCH v1] Issue 3778: Use bounding box as skylines for markup in svg backend

2020-04-28 Thread Jonas Hahnfeld
Am Dienstag, den 28.04.2020, 18:31 +0100 schrieb Kevin Barry: > Hi James, > > > We do have a process, see: > > http://lilypond.org/doc/v2.19/Documentation/contributor-big-page#summary-for-experienced-developers > > > > I have followed this as best I can, but it seems I need to have write >

Re: Add `-dfont-ps-resdir` option to embed fonts to PDFs later (issue 577900043 by truer...@gmail.com)

2020-04-28 Thread tdanielsmusic--- via Discussions on LilyPond development
Please read my suggested changes carefully and amend, as I've no knowledge of the subject matter. https://codereview.appspot.com/577900043/diff/577910043/Documentation/usage/running.itely File Documentation/usage/running.itely (right):

Re: [PATCH v1] Issue 3778: Use bounding box as skylines for markup in svg backend

2020-04-28 Thread Kevin Barry
Hi James, > We do have a process, see: > http://lilypond.org/doc/v2.19/Documentation/contributor-big-page#summary-for-experienced-developers I have followed this as best I can, but it seems I need to have write access to the issue tracker before I can send patches. If that is the case then can

Add `-dfont-ps-resdir` option to embed fonts to PDFs later (issue 577900043 by truer...@gmail.com)

2020-04-28 Thread lemzwerg--- via Discussions on LilyPond development
Very nice, thanks! There are some grammar glitches in the documentation – I hope that a native English speaker can improve that. https://codereview.appspot.com/577900043/diff/577910043/Documentation/usage/running.itely File Documentation/usage/running.itely (right):

Re: Make website fails

2020-04-28 Thread Jonas Hahnfeld
Am Dienstag, den 28.04.2020, 15:23 +0100 schrieb Phil Holmes: > On my machine make website returns: > > Traceback (most recent call last): > File "/media/IntelSSD/lilypond/lilypond-git/scripts/build/website_post.py", > line 179, in > lines = open(file).readlines() > File

Make website fails

2020-04-28 Thread Phil Holmes
On my machine make website returns: Traceback (most recent call last): File "/media/IntelSSD/lilypond/lilypond-git/scripts/build/website_post.py", line 179, in lines = open(file).readlines() File "/usr/lib/python3.5/encodings/ascii.py", line 26, in decode return codecs.ascii_decode(input,

Re: [PATCH v1] Issue 3778: Use bounding box as skylines for markup in svg backend

2020-04-28 Thread James
Hello Kevin, On 27/04/2020 18:07, Kevin Barry wrote: Hi James, Thank you for responding. On Mon, Apr 27, 2020 at 11:09:50AM +0100, James wrote: Is this patch read for full testing or is it work-in-progress you just want someone to comment on it? It's not work in progress no - unless someone

Re: Issue 5934: remove Repeated_music::folded_music_length (issue 561670043 by nine.fierce.ball...@gmail.com)

2020-04-28 Thread David Kastrup
d...@gnu.org writes: > Interesting. In 2.18 it appears to be used as length-callback in > TremoloRepeatedMusic. Was that wrong or did something change in > unfolded_music_length ? > > https://codereview.appspot.com/561670043/ I need to focus on repo search techniques. Issue 3745, apparently.

Re: Use a hash table for the lexer keywords (issue 549920043 by hanw...@gmail.com)

2020-04-28 Thread dak
Regarding the "Get rid of unused function ly:lexer-keywords": that one was added with commit 1ae6f5710e714f13da1bfbfbd840835316618d1f Author: Han-Wen Nienhuys Date: Thu Dec 14 14:10:56 2006 +0100 add ly:lexer-keywords to make keyword extraction easier and exact. but there appears to be no

Issue 5934: remove Repeated_music::folded_music_length (issue 561670043 by nine.fierce.ball...@gmail.com)

2020-04-28 Thread dak
Interesting. In 2.18 it appears to be used as length-callback in TremoloRepeatedMusic. Was that wrong or did something change in unfolded_music_length ? https://codereview.appspot.com/561670043/

PATCHES - Countdown for April 28th

2020-04-28 Thread James
Hello, Here is the current patch countdown list. The next countdown will be on April 30th A quick synopsis of all patches currently in the review process can be found here: http://philholmes.net/lilypond/allura/ *** Push: 5930 Make dblatex an optional dependency - Jonas Hahnfeld

Re: Rewrite Skyline code (issue 547980044 by hanw...@gmail.com)

2020-04-28 Thread jonas . hahnfeld
On 2020/04/28 07:20:26, hanwenn wrote: > On 2020/04/28 06:49:58, hahnjo wrote: > > https://codereview.appspot.com/547980044/diff/572060043/lily/skyline.cc > > File lily/skyline.cc (right): > > > > > https://codereview.appspot.com/547980044/diff/572060043/lily/skyline.cc#newcode175 > >

Re: Rewrite Skyline code (issue 547980044 by hanw...@gmail.com)

2020-04-28 Thread hanwenn
On 2020/04/28 06:49:58, hahnjo wrote: > https://codereview.appspot.com/547980044/diff/572060043/lily/skyline.cc > File lily/skyline.cc (right): > > https://codereview.appspot.com/547980044/diff/572060043/lily/skyline.cc#newcode175 > lily/skyline.cc:175: Building front_; > On 2020/04/27 21:30:18,

Re: Rewrite Skyline code (issue 547980044 by hanw...@gmail.com)

2020-04-28 Thread jonas . hahnfeld
https://codereview.appspot.com/547980044/diff/572060043/lily/skyline.cc File lily/skyline.cc (right): https://codereview.appspot.com/547980044/diff/572060043/lily/skyline.cc#newcode175 lily/skyline.cc:175: Building front_; On 2020/04/27 21:30:18, hanwenn wrote: > On 2020/04/27 07:01:28, hahnjo

Re: [trial] getting started

2020-04-28 Thread Jonas Hahnfeld
Am Dienstag, den 28.04.2020, 00:27 +0200 schrieb Federico Bruni: > Il giorno mer 22 apr 2020 alle 14:28, Jonas Hahnfeld ha > scritto: > > here we go: > > https://gitlab.com/lilypond-issues/lilypond-trial > > > > To reiterate: This is _NOT_ meant for "production" work, just for > > evaluation.