- **Comment**:
Greetings James, a happy new year to you as well!
Indeed, legacy.itely not being included is not a bug since this file is
supposed to be created from legacy-intro.itely. As per your request, `git
status show` only returns:
> nothing to commit, working tree clean
However, you appear to have something left from my previous (buggy) patch set;
could you check what `git ls-files -o | grep -v -E "out|\.log|\.pyc"` tells
you? The only three files created by my latest patch in Documentation/snippets
should be:
> Documentation/snippets/chord-names-alternative.ly
> Documentation/snippets/legacy-intro.itely
> Documentation/snippets/legacy.snippet-list
You may want to remove any other file (for example legacy-code.itely), which
should fix the problem.
Here’s to hoping we can get this sorted out! :-)
---
** [issues:#5449] Chord names clean-up; no more Banter, exceptionsPartial or
\powerChords.**
**Status:** Started
**Created:** Mon Dec 17, 2018 05:23 PM UTC by Valentin Villenave
**Last Updated:** Wed Jan 02, 2019 03:33 PM UTC
**Owner:** Valentin Villenave
**Attachments:**
- [Screenshot of
patchset4.png](https://sourceforge.net/p/testlilyissues/issues/5449/attachment/Screenshot%20of%20patchset4.png)
(68.5 kB; image/png)
Chord names clean-up; no more Banter, exceptionsPartial or \powerChords.
This rather large patch
- removes the legacy code for unsupported non-Ignatzek chord names;
that code hasn’t been working properly for nearly 15 years anyway, see
https://lists.gnu.org/archive/html/lilypond-user/2008-08/msg00288.html
- stores a version of this code, with minor corrections, as a
self-contained snippet (for historical purposes mainly), both
on the LSR (as an upgrade to snippet #102) and in a dedicated
subsection of the Snippets doc page.
- as a result, chord-generic-names.scm is no longer found in scm/
and the chord-names-jazz.ly doc chart no longer includes any
alternative chord naming systems (that were already broken anyway).
- chordNamesExceptionsFull and chordNamesExceptionsPartial
properties have been removed as well (both having no longer had
any effect whatsoever for quite some time).
- the main (and now only) chordNamesExceptions property def
has been slightly updated for slightly neater glyphs and spacing,
and so as to now include power chords by default; as a result,
\powerChords is no longer needed, thereby solving any possible
confusion with \germanChords and the like. (I can’t think of any
case where users would explicitely type \chords{c:5} in order to
get the same output as \chords{c} -- can you?)
- the documentation has been updated as well; convert rules
have been added (although these may be merged with previous
2.21.0 rules once the release cycle gets back on track).
http://codereview.appspot.com/363880043
---
Sent from sourceforge.net because [email protected] is
subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at
https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is
a mailing list, you can unsubscribe from the mailing list._______________________________________________
Testlilyissues-auto mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
