[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [LSR] multi-measure-rest-length-control.ly snippet should not use \c

From: Federico Bruni
Subject: Re: [LSR] multi-measure-rest-length-control.ly snippet should not use \compressFullBarRests
Date: Mon, 17 Sep 2018 11:10:56 +0200

Il giorno mar 11 set 2018 alle 14:59, James Lowe <address@hidden> ha scritto:

On 30/08/18 10:54, Federico Bruni wrote:
Hi folks

Bug Squad, this does not require a new issue in the tracker, but just someone who can fix the snippet on the LSR website.

There's a recent snippet file imported from LSR which contains \compressFullBarRests.

$ git grep \compressFullBarRests Documentation/snippets
Documentation/snippets/multi-measure-rest-length-control.ly:25: \compressFullBarRests Documentation/snippets/new/multi-measure-rest-length-control.ly:17: \compressFullBarRests

Given this commit:

commit ba9a8dc730bb3a681455150bc0a3eef0976523af
Author: Trevor Daniels <address@hidden>
Date: Fri May 1 22:56:47 2015 +0100

   Issue 3687 (part 2): Amend docs to use \compressMMRests

     Amend the examples in the LM and NR to use the \compressMMRests
     music function rather than the predefs \compressFullBarRests and
     \expandFullBarRests to avoid advocating the use of techniques
     which may have undesirable side effects.

we should replace it with \compressMMRests.
Who can do this change in the LSR?
I cannot find this on the LSR.

It is in LilyPond's own Snippets (but that is not the same thing) so a change to this would require a change in git.

You are right. As it's a command introduced in a development version, it's only in Documentation/snippets/new.


grep -r "compressFullBarRests" * | wc -l

gives a count of 38 (of which 14 are *.po files) so we might as well change it for those files too right?

In which case a tracker would be best.

Let me know.

.po files should not be changed, as they are updated from a single pot file: Documentation/po/lilypond-doc.pot, which does not contain compressFullBarRests. So .po files are not up-to-date with the pot file. But this file has not been updated for years and I can't remember to which extent it's actually used by the doc build system. I would ignore it.

German and japanese translation are out-of-date, but translators are in charge of keeping it up-to-date.

IMO we should open a new issue only for the snippet.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]