[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5449 Chord names clean-up; no more Banter, exceptionsPartial or \powerChords.

[Auto mailings of changes to Lily Issues via Testlilyissues-auto]

• 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 (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 address@hidden 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
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto