Re: Doc: cautionary accidentals after a key change (3000) (issue 6872049)

2012-12-02 Thread k-ohara5a5a
Looks nice. Maybe make it shorter, if you want. https://codereview.appspot.com/6872049/diff/2001/Documentation/notation/changing-defaults.itely File Documentation/notation/changing-defaults.itely (right): https://codereview.appspot.com/6872049/diff/2001/Documentation/notation/changing-defaults.

Re: Changes to Kievan documentation (issue 6865049)

2012-12-02 Thread dak
https://codereview.appspot.com/6865049/diff/2001/Documentation/notation/ancient.itely File Documentation/notation/ancient.itely (right): https://codereview.appspot.com/6865049/diff/2001/Documentation/notation/ancient.itely#newcode2483 Documentation/notation/ancient.itely:2483: @node Kievan note

Re: Doc: cautionary accidentals after a key change (3000) (issue 6872049)

2012-12-02 Thread dak
https://codereview.appspot.com/6872049/diff/2001/Documentation/notation/changing-defaults.itely File Documentation/notation/changing-defaults.itely (right): https://codereview.appspot.com/6872049/diff/2001/Documentation/notation/changing-defaults.itely#newcode3324 Documentation/notation/changing

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle

2012-12-02 Thread Thomas Morley
2012/12/3 Aleksandr Andreev : > I went into my git-cl directory and did a git pull -r > > Works fine for me now. > > Aleksandr > > On Sun, Dec 2, 2012 at 6:08 PM, Thomas Morley > wrote: >> 2012/12/2 Phil Holmes : >>> - Original Message - From: "Graham Percival" >>> >>> To: "Phil Holmes"

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle

2012-12-02 Thread Aleksandr Andreev
I went into my git-cl directory and did a git pull -r Works fine for me now. Aleksandr On Sun, Dec 2, 2012 at 6:08 PM, Thomas Morley wrote: > 2012/12/2 Phil Holmes : >> - Original Message - From: "Graham Percival" >> >> To: "Phil Holmes" >> Cc: ; "David Kastrup" >> Sent: Sunday, Dece

Re: Changes to Kievan documentation (issue 6865049)

2012-12-02 Thread aleksandr . andreev
Please review. https://codereview.appspot.com/6865049/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle

2012-12-02 Thread Thomas Morley
2012/12/2 Phil Holmes : > - Original Message - From: "Graham Percival" > > To: "Phil Holmes" > Cc: ; "David Kastrup" > Sent: Sunday, December 02, 2012 8:55 PM > > Subject: Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py > fromgoogle > > >> On Sun, Dec 02, 2012 at 02:47:27PM -

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle

2012-12-02 Thread David Kastrup
Graham Percival writes: > On Sun, Dec 02, 2012 at 02:47:27PM -, Phil Holmes wrote: >> Graham - could you either give me access to git-cl or push this >> patch, please? > > I've given Phil push access to git-cl. David, if you have a > github account I'll add you too. Haven't. -- David Kast

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle

2012-12-02 Thread Phil Holmes
- Original Message - From: "Graham Percival" To: "Phil Holmes" Cc: ; "David Kastrup" Sent: Sunday, December 02, 2012 8:55 PM Subject: Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle On Sun, Dec 02, 2012 at 02:47:27PM -, Phil Holmes wrote: Graham - could

Re: Issue 2916: Document \hide and \omit (issue 6851102)

2012-12-02 Thread graham
LGTM https://codereview.appspot.com/6851102/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: How to do proper indentation?

2012-12-02 Thread Graham Percival
On Fri, Nov 30, 2012 at 02:16:21AM +0100, Thomas Morley wrote: > Trying to minimize my lack of knowledge on this topic I searched the > CG about It. > > CG 10.5.3 Indentation > mentions possibilities for emacs and vim, but I'm not familiar with them. > I'm using jEdit or gedit > > Following the l

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle

2012-12-02 Thread Graham Percival
On Sun, Dec 02, 2012 at 02:47:27PM -, Phil Holmes wrote: > Graham - could you either give me access to git-cl or push this > patch, please? I've given Phil push access to git-cl. David, if you have a github account I'll add you too. - Graham ___ l

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle

2012-12-02 Thread David Kastrup
"Phil Holmes" writes: > - Original Message - > From: "David Kastrup" > To: "Phil Holmes" > Cc: ; "Graham Percival" > Sent: Sunday, December 02, 2012 3:55 PM > Subject: Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py > fromgoogle > > [snip] > >> Then there should be some ot

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle

2012-12-02 Thread Phil Holmes
- Original Message - From: "David Kastrup" To: "Phil Holmes" Cc: ; "Graham Percival" Sent: Sunday, December 02, 2012 3:55 PM Subject: Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle [snip] Then there should be some other "save"-like operation available. Wha

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle

2012-12-02 Thread David Kastrup
"Phil Holmes" writes: > - Original Message - >>> >>> The mail message would not apply - possibly because I had to go >>> Windows text->unicode->Ubuntu box. >> >> No. You should _not_ go "through" anything. You should just save the >> whole message as a file (using Ctrl-S or whatever is

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle

2012-12-02 Thread Phil Holmes
- Original Message - From: "David Kastrup" To: "Phil Holmes" Cc: ; "Graham Percival" Sent: Sunday, December 02, 2012 3:34 PM Subject: Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle "Phil Holmes" writes: If you save the whole message (rather than doing so

Re: markup-commands rest-by-number and rest (issue 6850073)

2012-12-02 Thread dak
thomasmorle...@googlemail.com writes: On 2012/11/27 20:03:06, benko.pal wrote: > Did you had a look on the compiled output of the new reg-tests? no. could you push to a dev/ branch? Sorry. I'm still a newbie in devel tasks. If you have a branch where you checked in your changes, you

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle

2012-12-02 Thread David Kastrup
"Phil Holmes" writes: >> If you save the whole message (rather than doing some copy&paste job on >> the mail client window) that should not be an issue in the first place. > > The mail message would not apply - possibly because I had to go > Windows text->unicode->Ubuntu box. No. You should _no

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle

2012-12-02 Thread Phil Holmes
- Original Message - From: "David Kastrup" To: Sent: Sunday, December 02, 2012 2:00 PM Subject: Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py fromgoogle "Phil Holmes" writes: - Original Message - From: "David Kastrup" To: "Phil Holmes" Cc: Sent: Sunday,

Re: Python hacker emergency on git-cl

2012-12-02 Thread Thomas Morley
2012/12/2 David Kastrup : > Thomas Morley writes: > >> 2012/12/2 David Kastrup : >>> Thomas Morley writes: >>> 2012/12/2 David Kastrup : > David Kastrup writes: > >> Do >> >> git pull -r >> >> first to get your patch up-to-date. > > To get your git-cl rep

Re: Python hacker emergency on git-cl

2012-12-02 Thread David Kastrup
Thomas Morley writes: > 2012/12/2 David Kastrup : >> Thomas Morley writes: >> >>> 2012/12/2 David Kastrup : David Kastrup writes: > Do > > git pull -r > > first to get your patch up-to-date. To get your git-cl repository up-to-date, sorry. >>> >>> Tried t

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py from google

2012-12-02 Thread David Kastrup
"Phil Holmes" writes: > - Original Message - > From: "David Kastrup" > To: "Phil Holmes" > Cc: > Sent: Sunday, December 02, 2012 1:43 PM > Subject: Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py > from google > > >> "Phil Holmes" writes: >> >>> I'd suggest pushing this d

Re: Python hacker emergency on git-cl

2012-12-02 Thread Thomas Morley
2012/12/2 David Kastrup : > Thomas Morley writes: > >> 2012/12/2 David Kastrup : >>> David Kastrup writes: >>> Do git pull -r first to get your patch up-to-date. >>> >>> To get your git-cl repository up-to-date, sorry. >>> >>> -- >>> David Kastrup >>> >>> >>>

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py from google

2012-12-02 Thread Phil Holmes
- Original Message - From: "David Kastrup" To: "Phil Holmes" Cc: Sent: Sunday, December 02, 2012 1:43 PM Subject: Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py from google "Phil Holmes" writes: I'd suggest pushing this direct to github: do you have the permission

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py from google

2012-12-02 Thread David Kastrup
"Phil Holmes" writes: > I'd suggest pushing this direct to github: do you have the permission > to do this? No. > If you don't, could you attach a patch as an attachment and I'll push > it? What is wrong with the mail called [PATCH]? You can just save it and apply it using git-am. -- David

Re: Python hacker emergency on git-cl

2012-12-02 Thread David Kastrup
Thomas Morley writes: > 2012/12/2 David Kastrup : >> David Kastrup writes: >> >>> Do >>> >>> git pull -r >>> >>> first to get your patch up-to-date. >> >> To get your git-cl repository up-to-date, sorry. >> >> -- >> David Kastrup >> >> >> ___ >> lilypo

Re: Python hacker emergency on git-cl

2012-12-02 Thread Thomas Morley
2012/12/2 David Kastrup : > David Kastrup writes: > >> Do >> >> git pull -r >> >> first to get your patch up-to-date. > > To get your git-cl repository up-to-date, sorry. > > -- > David Kastrup > > > ___ > lilypond-devel mailing list > lilypond-devel@gnu

Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py from google

2012-12-02 Thread Phil Holmes
I'd suggest pushing this direct to github: do you have the permission to do this? If you don't, could you attach a patch as an attachment and I'll push it? -- Phil Holmes ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.or

Re: Python hacker emergency on git-cl

2012-12-02 Thread David Kastrup
David Kastrup writes: > Do > > git pull -r > > first to get your patch up-to-date. To get your git-cl repository up-to-date, sorry. -- David Kastrup ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilyp

Re: Python hacker emergency on git-cl

2012-12-02 Thread David Kastrup
Thomas Morley writes: > 2012/12/2 Thomas Morley : >> 2012/12/2 Reinhold Kainhofer : >>> On 2012-12-02 12:50, Thomas Morley wrote: Trying to upload a new patch set for issue 2966, I ran into said git-cl-issue. Using LilyDev, I tried to apply Reinholds patch (after storing it in

Re: Python hacker emergency on git-cl

2012-12-02 Thread David Kastrup
Thomas Morley writes: > Trying to upload a new patch set for issue 2966, I ran into said git-cl-issue. > Using LilyDev, I tried to apply Reinholds patch (after storing it in > git-cl.diff), but the terminal returns: > > harm@harm-laptop ~/lilypond-git (dev/local_working)$ git apply git-cl.diff >

Re: Python hacker emergency on git-cl

2012-12-02 Thread Thomas Morley
2012/12/2 Thomas Morley : > 2012/12/2 Reinhold Kainhofer : >> On 2012-12-02 12:50, Thomas Morley wrote: >>> >>> Trying to upload a new patch set for issue 2966, I ran into said >>> git-cl-issue. >>> Using LilyDev, I tried to apply Reinholds patch (after storing it in >>> git-cl.diff), but the termi

Re: Python hacker emergency on git-cl

2012-12-02 Thread Thomas Morley
2012/12/2 Reinhold Kainhofer : > On 2012-12-02 12:50, Thomas Morley wrote: >> >> Trying to upload a new patch set for issue 2966, I ran into said >> git-cl-issue. >> Using LilyDev, I tried to apply Reinholds patch (after storing it in >> git-cl.diff), but the terminal returns: >> >> harm@harm-lapto

Re: Python hacker emergency on git-cl

2012-12-02 Thread Reinhold Kainhofer
On 2012-12-02 12:50, Thomas Morley wrote: Trying to upload a new patch set for issue 2966, I ran into said git-cl-issue. Using LilyDev, I tried to apply Reinholds patch (after storing it in git-cl.diff), but the terminal returns: harm@harm-laptop ~/lilypond-git (dev/local_working)$ git apply git

Re: Python hacker emergency on git-cl

2012-12-02 Thread Thomas Morley
2012/12/2 David Kastrup : > James writes: > >> David, >> >> On 2 December 2012 08:55, David Kastrup wrote: >>> James writes: >>> >> ... >>> Well, I have been using it already, and it is clearly an improvement >>> over the previous situation, so it should be committed to git-cl, and >>> people sh

Re: IR: Improve performer documentation (issue 6868047)

2012-12-02 Thread src
Reviewers: dak, Message: On 2012/12/02 11:11:54, dak wrote: https://codereview.appspot.com/6868047/diff/1/scm/document-translation.scm#newcode21 scm/document-translation.scm:21: ;; type predicates ly:engraver? and ly:performer?. I think that this approach is not really helping as it is inflexib

IR: Improve performer documentation (issue 6868047)

2012-12-02 Thread dak
Improving the docs certainly is quite worthwhile. https://codereview.appspot.com/6868047/diff/1/scm/document-translation.scm File scm/document-translation.scm (right): https://codereview.appspot.com/6868047/diff/1/scm/document-translation.scm#newcode21 scm/document-translation.scm:21: ;; type p

Re: Python hacker emergency on git-cl

2012-12-02 Thread David Kastrup
James writes: > David, > > On 2 December 2012 08:55, David Kastrup wrote: >> James writes: >> > ... >> Well, I have been using it already, and it is clearly an improvement >> over the previous situation, so it should be committed to git-cl, and >> people should be encouraged to update their ver

Re: Python hacker emergency on git-cl

2012-12-02 Thread James
David, On 2 December 2012 08:55, David Kastrup wrote: > James writes: > ... > Well, I have been using it already, and it is clearly an improvement > over the previous situation, so it should be committed to git-cl, and > people should be encouraged to update their version of git-cl afterwards >

Re: Python hacker emergency on git-cl

2012-12-02 Thread David Kastrup
James writes: > Hello, > > On 1 December 2012 17:24, Reinhold Kainhofer > wrote: >> On 2012-12-01 17:12, David Kastrup wrote:> My interpretation is that Google >> has set codereview.appspot.com so that >>> any http request will automatically get redirected to https, and git-cl >>> is unable to

Re: Python hacker emergency on git-cl

2012-12-02 Thread Trevor Daniels
James wrote Sunday, December 02, 2012 8:15 AM > > On 1 December 2012 17:24, Reinhold Kainhofer > wrote: >> On 2012-12-01 17:12, David Kastrup wrote:> My interpretation is that Google >> has set codereview.appspot.com so that >>> any http request will automatically get redirected to https, and g

Re: Python hacker emergency on git-cl

2012-12-02 Thread James
Hello, On 1 December 2012 17:24, Reinhold Kainhofer wrote: > On 2012-12-01 17:12, David Kastrup wrote:> My interpretation is that Google > has set codereview.appspot.com so that >> any http request will automatically get redirected to https, and git-cl >> is unable to deal with that. > > Well, gi