no git patchset from .news files? (was Re: gnucash-htdocs branch master updated. svn_last-12-g3580119)
Derek Atkins
warlord at MIT.EDU
Mon Jul 15 17:07:59 EDT 2013
John Ralls <jralls at ceridwen.us> writes:
>> Good question. I haven't looked at that yet, but I think so.
>
> Confirmed: It doesn't matter. A single commit, even a git tag -a or a
> git merge, has a subject line. Here's what I've written:
> Single commit:
>
> Subject: gnucash trunk: Update Danish TRanslation, copied from the
> Translation Project.
>
> Multiple commits:
> Subject: Multiple changes pushed to gnucash master
Any chance we could standardize on one way to output the subject, e.g.:
Subject: <project> <branch>: <Message>
So that would result in:
Subject: gnucash trunk: Update Danish TRanslation, copied from the
Translation Project.
or
Subject: gnucash master: Multiple changes pushed
[snip]
> The other thing that I notice is that the summary of changes is for
> the whole push. That's fine if all of the commits are about more or
> less the same thing, but it might be puzzling if they're a bunch of
> unrelated bug-fixes. I'm inclined to say that we should remove the
> summary at the end of the message and incorporate it into the
> per-change log instead by changing custom_showrev to either "git log
> -n 1 --stat %s" or "git log -n 1 --patch-with-stat %s" depending upon
> which list it's going to.
I'm not sure I know enough yet to give an informed answer. Could you
show some example output for what each would look like? (I guess I
could run the commands myself to look).
> Comments, anyone?
>
> Regards,
> John Ralls
-derek
--
Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory
Member, MIT Student Information Processing Board (SIPB)
URL: http://web.mit.edu/warlord/ PP-ASEL-IA N1NWH
warlord at MIT.EDU PGP key available
More information about the gnucash-devel
mailing list