Reviewers: ,
Message:
Please review.
Description:
Issue 5806: Tweak mf files to avoid FontForge internal overlap error
When compiling LilyPond, there were 352 error messages caused by
FontForge trying to resolve overlapping path problems:
>>> Internal Error (overlap) in [...]
Fortunately, the
Am Samstag, den 29.02.2020, 02:36 +0100 schrieb David Kastrup:
> Hi, I've run scripts/auciliar/fixcc.py on the stable branch now. So
> that it is reasonably easy to cherry-pick new commits from master into
> the stable branch, it would make sense doing that in staging/master as
> well. This is a
On 19/02/2020 16:32, Federico Bruni wrote:
Can anybody fix this typo in the website?
http://lilypond.org/productions.html
/opera series/opera seria/
See https://en.wikipedia.org/wiki/Opera_seria
It was reported by Luca Rossetto.
He suggests the following wording:
Luca Rossetto Casel, for his
On 2020/02/29 10:37:28, hahnjo wrote:
> Rename DATADIR to CONFIG_DATADIR
I need this additional fix to make it work with Guile 2.2
https://codereview.appspot.com/579340043/
On 23/02/20 11:50, torsten.haemme...@web.de wrote:
> modified: scm/define-note-names.scm
> - Missing quarter-tone definitions added, now all languages are
> complete.
> - Slightly changed sort order of norsk so that the commonly used forms
> (the ones with doulbe-s) are in front of the less com
On 2020/02/29 10:44:40, antlists_youngman.org.uk wrote:
> Smelling pistake, sorry ... - just spotted it.
Hi Wol,
Too late - this has already been pushed.
But as it's only the commit message description, it won't have
any effect anyway.
But taking a closer look, I've found "propoer" with an extra
Reviewers: hahnjo, Dan Eble,
Message:
On 2020/02/28 23:33:32, Dan Eble wrote:
> On 2020/02/28 23:31:17, Dan Eble wrote:
> > This does not look good. I expect this change to break the feature
I added in
> > 0d72930e579a5784ecb26da2f9880d8c9da05e71 (Issue 5635). Well, a call
to strip
> > (None) is
https://codereview.appspot.com/583590043/diff/571790043/scripts/build/output-distance.py
File scripts/build/output-distance.py (right):
https://codereview.appspot.com/583590043/diff/571790043/scripts/build/output-distance.py#newcode531
scripts/build/output-distance.py:531: if self.contents[0] is
Very nice, thanks! LGTM.
https://codereview.appspot.com/571780043/diff/549620043/mf/feta-arrowheads.mf
File mf/feta-arrowheads.mf (right):
https://codereview.appspot.com/571780043/diff/549620043/mf/feta-arrowheads.mf#newcode58
mf/feta-arrowheads.mf:58: y5 = 0;
Please use an indentation of 8 spa
LGTM; some minor nits
https://codereview.appspot.com/553580043/diff/571780045/mf/GNUmakefile
File mf/GNUmakefile (right):
https://codereview.appspot.com/553580043/diff/571780045/mf/GNUmakefile#newcode130
mf/GNUmakefile:130: $(outdir)/emmentaler-brace.otf:
emmentaler-brace.fontforge.py $(outdir)/
LGTM
https://codereview.appspot.com/547690053/
https://codereview.appspot.com/547690053/diff/567300043/config.make.in
File config.make.in (right):
https://codereview.appspot.com/547690053/diff/567300043/config.make.in#newcode48
config.make.in:48: GROFF = @GROFF@
These sort of drive-by changes without any mention in commit message or
issue ma
added
* Remove unused GROFF and LD autoconf vars
to the commit msg.
https://codereview.appspot.com/547690053/diff/567300043/config.make.in
File config.make.in (right):
https://codereview.appspot.com/547690053/diff/567300043/config.make.in#newcode48
config.make.in:48: GROFF = @GROFF@
On 20
On 2020/02/29 22:13:42, hanwenn wrote:
> added
>
> * Remove unused GROFF and LD autoconf vars
>
> to the commit msg.
>
> https://codereview.appspot.com/547690053/diff/567300043/config.make.in
> File config.make.in (right):
>
>
https://codereview.appspot.com/547690053/diff/567300043/config.
https://codereview.appspot.com/553580043/diff/571780045/mf/GNUmakefile
File mf/GNUmakefile (right):
https://codereview.appspot.com/553580043/diff/571780045/mf/GNUmakefile#newcode130
mf/GNUmakefile:130: $(outdir)/emmentaler-brace.otf:
emmentaler-brace.fontforge.py $(outdir)/emmentaler-brace.otf-t
On 2020/02/29 22:28:14, dak wrote:
> On 2020/02/29 22:13:42, hanwenn wrote:
> > added
> >
> > * Remove unused GROFF and LD autoconf vars
> >
> > to the commit msg.
> >
> >
https://codereview.appspot.com/547690053/diff/567300043/config.make.in
> > File config.make.in (right):
> >
> >
>
http
Reviewers: ,
Message:
tested by running 'make dist'
Description:
Remove unused .1 => .txt rule
Please review this at https://codereview.appspot.com/557560044/
Affected files (+0, -5 lines):
M stepmake/stepmake/documentation-rules.make
Index: stepmake/stepmake/documentation-rules.make
diff -
https://codereview.appspot.com/557560044/
https://codereview.appspot.com/547690053/
> > Can we assume that FontForge's python support and is
> > always enabled? Shall we check this?
>
> the FF page doesn't say that python is optional.
It's a build option in both the old (configure) and new (cmake)
builds...
https://codereview.appspot.com/553580043/
On 2020/02/29 22:37:03, hanwenn wrote:
> tested by running 'make dist'
Maybe move the removal of the GROFF test/flag here? If there should be
some unexpected breakage at some point of time, that makes it more
likely that it can be handled with a single revert.
https://codereview.appspot.com/5575
On 2020/02/29 23:57:03, dak wrote:
> On 2020/02/29 22:37:03, hanwenn wrote:
> > tested by running 'make dist'
>
> Maybe move the removal of the GROFF test/flag here? If there should
be some
> unexpected breakage at some point of time, that makes it more likely
that it can
> be handled with a sing
Re-formatted: spaces inserted and strictly 8 char indentations.
https://codereview.appspot.com/571780043/diff/549620043/mf/feta-arrowheads.mf
File mf/feta-arrowheads.mf (right):
https://codereview.appspot.com/571780043/diff/549620043/mf/feta-arrowheads.mf#newcode58
mf/feta-arrowheads.mf:58: y5 =
22 matches
Mail list logo