r19711 - gnucash-docs/trunk/guide - Merge branch 'bug633066' into HEAD

Yawar Amin yawar.amin at gmail.com
Fri Oct 29 07:53:54 EDT 2010

Hi Geert,

On 2010-10-29, at 05:21, Geert Janssens wrote:

> […]
> The commit log message is a bit confusing in the svn context though. The merge 
> didn't happen in svn. Instead, svn only sees the summary of several commits in 
> the git branch you used to work on this. In this case it's luckily not really 
> an issue. The branch was called "bug633066" which gives a good hint to what 
> the commit intended to resolve. But branch names may not always be so helpful.

You are absolutely right, basically I never even wanted a Subversion user to be aware that a merge had been done. As far as SVN is concerned the history is linear. I should have checked carefully before committing. Mea culpa!

> I'm thinking of ways we can improve this: the first obvious improvement would 
> be to have a more elaborate log message when merging a development branch back 
> to head, one that makes sense in the svn context as well.
> For example in this case the log message could have been:
> Bug #633066 - Add a few structuring and translator comments to gnucash-
> guide.xml
> (git - Merge branch 'bug633066' into HEAD)
> Or something like that. Especially the "Bug #633066" inclusion is very useful. 
> This construct is recognized by trac and will result in links to the relevant 
> bug report in the trac browser. The git part helps git users to see how this 
> commit was realized.
> Anyway, that is my first idea. I'm open for improvements or simplifications.

I think what I should have done is a git merge –no-commit and then edited the commit message to give a good description. I’ll keep this in mind for the future. Thanks for pointing it out!



-------------- next part --------------
A non-text attachment was scrubbed...
Name: PGP.sig
Type: application/pgp-signature
Size: 194 bytes
Desc: This is a digitally signed message part
URL: <http://lists.gnucash.org/pipermail/gnucash-devel/attachments/20101029/86313949/attachment.bin>

More information about the gnucash-devel mailing list