lilypond-devel
[Top][All Lists]
Advanced

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

makelsr updates (issue 5991068)


From: PhilEHolmes
Subject: makelsr updates (issue 5991068)
Date: Sun, 08 Apr 2012 13:59:14 +0000

Reviewers: Graham Percival,

Message:
This is how I've changed makelsr.  If you're happy I'll push.

Description:
Adds minor progress messages (easier to track what's going on)
and uses directory list rather than explicit list of tags.

Please review this at http://codereview.appspot.com/5991068/

Affected files:
  M scripts/auxiliar/makelsr.py


Index: scripts/auxiliar/makelsr.py
diff --git a/scripts/auxiliar/makelsr.py b/scripts/auxiliar/makelsr.py
index a03466310b36cf47d7e91f12c7e6f79422a680d3..3d088b526981748ddd7d7f2d8bdc9ee348fec97b 100755
--- a/scripts/auxiliar/makelsr.py
+++ b/scripts/auxiliar/makelsr.py
@@ -37,27 +37,15 @@ LY_HEADER_NEW = '''%% DO NOT EDIT this file manually; it is automatically
 %% This file is in the public domain.
 ''' % NEW_LYS

-TAGS = []
-# NR 1
-TAGS.extend (['pitches', 'rhythms', 'expressive-marks',
-'repeats', 'simultaneous-notes', 'staff-notation',
-'editorial-annotations', 'text'])
-# NR 2
-TAGS.extend (['vocal-music', 'chords', 'keyboards',
-'percussion', 'fretted-strings', 'unfretted-strings',
-'ancient-notation', 'winds', 'world-music'
-])
-
-# other
-TAGS.extend (['contexts-and-engravers', 'tweaks-and-overrides',
-'paper-and-layout', 'breaks', 'spacing', 'midi', 'titles', 'template'])
-
 def exit_with_usage (n=0):
     sys.stderr.write (USAGE)
     sys.exit (n)

 if len (sys.argv) >= 2:
     in_dir = sys.argv[1]
+    if not (os.path.isdir (in_dir)):
+        sys.stderr.write (in_dir + ' is not a directory.\n')
+        exit (2)
     if len (sys.argv) >= 3:
         exit_with_usage (2)
     if not (os.path.isdir (DEST) and os.path.isdir (NEW_LYS)):
@@ -65,6 +53,8 @@ if len (sys.argv) >= 2:
 else:
     in_dir = ''

+TAGS = os.listdir (in_dir)
+
 # which convert-ly to use
 if os.path.isfile("out/bin/convert-ly"):
     conv_path='out/bin/'
@@ -149,6 +139,7 @@ def copy_ly (srcdir, name, tags):
     dest = os.path.join (DEST, name)
     tags = ', '.join (tags)
     s = open (os.path.join (srcdir, name)).read ()
+ sys.stderr.write ('\nmakelsr.py: reading ' + os.path.join (srcdir, name) + '\n')

     for path in TEXIDOCS:
         texidoc_translation_path = \
@@ -172,6 +163,7 @@ def copy_ly (srcdir, name, tags):
     s = strip_white_spaces_re.sub ('', s)
     s = escape_backslashes_in_header (s)
     open (dest, 'w').write (s)
+    sys.stderr.write ('makelsr.py: writing ' + dest + '\n')

     e = os.system (convert_ly+(" -d -e '%s'" % dest))
     if e:





reply via email to

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