David, On Tue, Jul 24, 2012 at 11:50 AM, David Kastrup <d...@gnu.org> wrote:
> David Nalesnik <david.nales...@gmail.com> writes: > > > On Tue, Jul 24, 2012 at 11:34 AM, Graham Percival > > <gra...@percival-music.ca> wrote: > > > > On Tue, Jul 24, 2012 at 05:26:01PM +0100, Phil Holmes wrote: > > > I'd be happy to push this, but would like a little advice. If I > > > go to Rietveld and download the raw patch set, it comes without > > > proper email addresses and formatting. This means to push stuff > > > from Rietveld, I have to git apply the diff, then manually > > > update the author (assuming I remember). Is there a way of > > > getting a formatted patch? > > > > > > Don't apply things directly from Rietveld. Due to the problems > > that you discovered, we can only use it as a reviewing tool, not a > > patch management tool. > > > > David Nalesnik should send you the file he gets from > > git format-patch, after he fixed all merge conflicts etc. > > > > > > Arggh...sorry for making a hash of this :( Hopefully, this patch will > > be properly formatted... > > Yup. A matter of personal taste (you can do that just with git commit > --amend): the commit message is > > [PATCH] Function for overriding broken spanners > > We don't commit anything other than patches. That's redundant. But > > Issue 2679: Function for overriding broken spanners > > would make it easier to find the corresponding issue in case one needs > to revisit the commit at one point of time. > I just tried to change the commit message as you said. When I check the amended patch, however, the subject line at the top of the file is: Subject: [PATCH] Issue 2679: Function for overriding broken spanners Apparently, [PATCH] is added automatically when I make a new patch with the LilyDev interface. (It doesn't appear in nano.) I can, of course, just remove "[PATCH]" manually and save. -David
_______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel