Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(646)

Issue 571250043: Drop undocumented lilymidi and lilysong (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 4 months ago by hahnjo
Modified:
4 years, 3 months ago
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Drop undocumented lilymidi and lilysong Both have been introduced by commit d0c106f039 ("Singing support") and haven't received many updates since then. They are mentioned nowhere in the documentation and there are no regression tests to make sure they work correctly. Also remove elisp/lilypond-song.el which called the lilysong script. This still leaves ly/festival.ly, not documented either. At least there are some regression tests including it, so it may be working as intended.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1003 lines) Patch
D elisp/lilypond-song.el View 1 chunk +0 lines, -556 lines 0 comments Download
M scripts/GNUmakefile View 1 chunk +1 line, -1 line 0 comments Download
D scripts/lilymidi.py View 1 chunk +0 lines, -230 lines 0 comments Download
D scripts/lilysong.py View 1 chunk +0 lines, -216 lines 0 comments Download

Messages

Total messages: 2
pkx166h-lilypond
See: https://sourceforge.net/p/testlilyissues/issues/1245/ This is the state of festival.
4 years, 4 months ago (2019-12-14 15:32:10 UTC) #1
lemzwerg
4 years, 4 months ago (2019-12-15 06:37:39 UTC) #2
LGTM, thanks.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b