great idea! A few implementation quibbles, though.
http://codereview.appspot.com/5489134/diff/1/scripts/auxiliar/makelsr.py File scripts/auxiliar/makelsr.py (right): http://codereview.appspot.com/5489134/diff/1/scripts/auxiliar/makelsr.py#newcode76 scripts/auxiliar/makelsr.py:76: lilypondexe=conv_path+'lilypond' I have a slight preference for lilypond_bin but this isn't a serious complaint. http://codereview.appspot.com/5489134/diff/1/scripts/auxiliar/makelsr.py#newcode184 scripts/auxiliar/makelsr.py:184: e = os.system (lilypondexe + " -dno-print-pages -dsafe -o /tmp/lsrtest " + dest) 1. why did you remove the -V flag? 2. AFAIK if you simply do + dest, it will break if there are any snippets with a space in the filename. I think that as a policy we don't allow those, but the '%s' % dest solution avoids that potential problem. Why not keep it there? http://codereview.appspot.com/5489134/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel
