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