Re: Making website build almost completely quiet (issue4515042)

2011-05-26 Thread Francisco Vila
2011/5/25 Graham Percival : > On Wed, May 25, 2011 at 05:02:42PM +0100, Phil Holmes wrote: >> Could you push, please? > > Sure, please do >  git format-origin patch I'd better say 'git format-patch origin'. > (I think -- it's been a while!  actual command is in the CG) > and send me the file. --

Re: Making website build almost completely quiet (issue4515042)

2011-05-25 Thread Graham Percival
On Wed, May 25, 2011 at 05:02:42PM +0100, Phil Holmes wrote: > Could you push, please? Sure, please do git format-origin patch (I think -- it's been a while! actual command is in the CG) and send me the file. Cheers, - Graham ___ lilypond-devel mail

Re: Making website build almost completely quiet (issue4515042)

2011-05-25 Thread Phil Holmes
- Original Message - From: To: ; Cc: ; Sent: Wednesday, May 25, 2011 4:50 PM Subject: Re: Making website build almost completely quiet (issue4515042) LGTM http://codereview.appspot.com/4515042/ :-) Could you push, please? -- Phil Holmes

Re: Making website build almost completely quiet (issue4515042)

2011-05-25 Thread percival . music . ca
LGTM http://codereview.appspot.com/4515042/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Making website build almost completely quiet (issue4515042)

2011-05-25 Thread PhilEHolmes
Hopefully final patch. http://codereview.appspot.com/4515042/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Making website build almost completely quiet (issue4515042)

2011-05-25 Thread Phil Holmes
- Original Message - From: To: Cc: ; Sent: Sunday, May 15, 2011 9:19 PM Subject: Re: Making website build almost completely quiet (issue4515042) could we get a subject like "add a -q quiet flag to make websit scripts" ? I've given the patch set that name. Is

Re: Making website build almost completely quiet (issue4515042)

2011-05-15 Thread percival . music . ca
could we get a subject like "add a -q quiet flag to make websit scripts" ? Also, don't mention that it includes previous patches -- that's implicitly done everywhere in git (or any source versioning system) http://codereview.appspot.com/4515042/diff/10001/make/website.make File make/website.mak

Re: Making website build almost completely quiet (issue4515042)

2011-05-15 Thread PhilEHolmes
2nd draft of quiet website build. http://codereview.appspot.com/4515042/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Making website build almost completely quiet (issue4515042)

2011-05-15 Thread percival . music . ca
On 2011/05/15 07:08:51, Graham Percival wrote: ... wait a moment, this is your previous patch. Why are we still discussing it? Please close this issue, then comment on your newer google code issue. ok, never mind, this *is* the newer google code issue. In your git, do git pull -r git

Re: Making website build almost completely quiet (issue4515042)

2011-05-15 Thread percival . music . ca
On 2011/05/14 15:56:33, PhilEHolmes wrote: Replies to Graham's comments. Not sure I've worked out how the review site does quoting yet, so apologies if this looks strange. This looks fine. Yeah - that's what I said in the comment on the review. All the changes are with respect to the ol

Re: Making website build almost completely quiet (issue4515042)

2011-05-14 Thread PhilEHolmes
Replies to Graham's comments. Not sure I've worked out how the review site does quoting yet, so apologies if this looks strange. http://codereview.appspot.com/4515042/diff/1/make/website.make File make/website.make (right): http://codereview.appspot.com/4515042/diff/1/make/website.make#newcode

Re: Making website build almost completely quiet (issue4515042)

2011-05-11 Thread percival . music . ca
looks basically good. http://codereview.appspot.com/4515042/diff/1/make/website.make File make/website.make (right): http://codereview.appspot.com/4515042/diff/1/make/website.make#newcode8 make/website.make:8: quiet-run = $(findstring s, $(MAKEFLAGS)) I think I'd still rather have this be: quie