Re: Issue 2961: \fret-diagram-verbose with capo > 1 (issue 6854106)

2012-12-29 Thread pkx166h
This has been approved, if you don't have push access let me know and I can push it for you. James https://codereview.appspot.com/6854106/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Fix KeyError in website_post.py from Issue 847 (issue 7021043)

2012-12-29 Thread julien . rioux
LGTM https://codereview.appspot.com/7021043/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

make doc: cannot import name TexModule (but dblatex is now a required dependency)

2012-12-29 Thread Federico Bruni
This is a followup to this thread: http://lists.gnu.org/archive/html/lilypond-devel/2012-08/msg00716.html I used to work around the problem by removing dblatex. But dblatex is now a required dependency (see issue 2529) and configure won't create the Makefile if dblatex is not installed. Can yo

Re: make doc: cannot import name TexModule (but dblatex is now a requireddependency)

2012-12-29 Thread Phil Holmes
- Original Message - From: "Federico Bruni" To: "Lily-Devel List" Sent: Saturday, December 29, 2012 2:25 PM Subject: make doc: cannot import name TexModule (but dblatex is now a requireddependency) Please check the logfile suffix-lyxml.dblatex.log for errors What does the logfi

Re: make doc: cannot import name TexModule (but dblatex is now a requireddependency)

2012-12-29 Thread David Kastrup
"Phil Holmes" writes: > - Original Message - > From: "Federico Bruni" > To: "Lily-Devel List" > Sent: Saturday, December 29, 2012 2:25 PM > Subject: make doc: cannot import name TexModule (but dblatex is now a > requireddependency) > > > >> Please check the logfile suffix-lyxml.dblatex

Re: Issue 2961: \fret-diagram-verbose with capo > 1 (issue 6854106)

2012-12-29 Thread torsten . haemmerle
Hi James, I don't have push access, so it'd be nice if you pushed it for me. Torsten https://codereview.appspot.com/6854106/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: make doc: cannot import name TexModule (but dblatex is now arequireddependency)

2012-12-29 Thread Phil Holmes
- Original Message - From: "David Kastrup" To: Sent: Saturday, December 29, 2012 2:32 PM Subject: Re: make doc: cannot import name TexModule (but dblatex is now arequireddependency) "Phil Holmes" writes: - Original Message - From: "Federico Bruni" To: "Lily-Devel List"

Re: make doc: cannot import name TexModule (but dblatex is now arequireddependency)

2012-12-29 Thread David Kastrup
"Phil Holmes" writes: >> Error: cannot import name TexModule >> >> -- >> David Kastrup > > > Yes - sorry - didn't see the attachment. So - what might cause this > error? grep TexModule `dpkg -L dblatex` 2>/dev/null /usr/share/pyshared/dbtexmf/dblatex/grubber/xetex.py:from plugins import TexMo

Re: Doc: NR 5.3 Add \single command (issue 6742057)

2012-12-29 Thread pkx166h
Thanks for help on this so far. I really don't understand this deeply enough, so again bear with me if I am showing my ignorance. https://codereview.appspot.com/6742057/diff/1/Documentation/notation/changing-defaults.itely File Documentation/notation/changing-defaults.itely (left): https://cod

Re: make doc: cannot import name TexModule (but dblatex is now arequireddependency)

2012-12-29 Thread Federico Bruni
Il 29/12/2012 16:51, David Kastrup ha scritto: I have no idea, but an incomplete installation of dblatex or one that is unable to find its own parts seems plausible. The file which contains TexModule is plugins.py I have that file both under python2.6 and python2.7 This is what I get: # grep

Re: Issue 2961: \fret-diagram-verbose with capo > 1 (issue 6854106)

2012-12-29 Thread pkx166h
author Torsten Hämmerle Sat, 29 Dec 2012 16:10:43 + (16:10 +) committer James Lowe Sat, 29 Dec 2012 16:12:22 + (16:12 +) commitf29be3cf2d0571eb281a3ba7761bcc993fd024e6 Please close Rietveld issue and thank you for your time helping to improve LilyPond. Ein glückliches neues J

PO: remove duplicates entries for hh and cc from ALL_PO_SOURCES (issue 7029043)

2012-12-29 Thread lilyfan
Reviewers: dak, phileholmes_googlemail.com, Graham Percival, Message: Please review Description: PO: remove duplicates entries for hh and cc from ALL_PO_SOURCES Introduced with version 1.0.1 : in stepmake/stepmake/C.make ALL_C_SOURCES = h, c, y and l files in stepmake/stepmake/Targets.m