[Ohrrpgce] recommended workflow for merging a git feature branch before dcommit

Ralph Versteegen teeemcee at gmail.com
Tue Sep 5 05:04:15 PDT 2017


Suppose you're on branch A and have some extra commits there. The natural
option would be rebase master on top of A, delete A, and dcommit from
master to svn:
git rebase A master  # any commits to master will follow after those in A
git branch -d A
git svn dcommit  # pushing any commits on A or on master to svn

But it's not necessary to be on the master branch when you use git svn
dcommit; unlike git push and git pull, for which each of your branches may
have a remote git branch which it tracks (meaning, which push and pull act
on), git svn dcommit/rebase appears to just default to the 'wip' svn
trunk/branch if you're on some git branch that doesn't have an svn branch
of the same name (eg "web" or "dwimmercrafty" git branches).
So "git svn dcommit" will 1) fetch any new svn wip commits you don't have
yet, 2) rebase the current branch on top of svn/wip, 3) make commits to svn
(and update svn/wip). Well, actually I think the real order is 3, 1, 2?
Since IIRC svn lets you make commits without updating first, as long as
none of the same files have been modified? Making an svn commit without
even having the latest svn revision so that you can verify it compiles
seems fairly harmless as long as you at least test it immediately
afterwards. But it happens to me by accident all the time when you've made
a commit I wasn't aware of, and I've not once had a problem result from it.

So anyway, if you're on branch A, you can just do:
git svn dcommit
git checkout master
git svn rebase  # If there were no extra commits to master, master and A
should now be identical
git branch -d A



On 5 September 2017 at 11:06, James Paige <Bob at hamsterrepublic.com> wrote:

> Hey, TMC, what workflow do you use when you want to merge a feature branch
> before you "git svn dcommit" it?
>
> I remember being warned that ordinary merges are a source of problems.
>
> ---
> James
>
> _______________________________________________
> Ohrrpgce mailing list
> ohrrpgce at lists.motherhamster.org
> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.motherhamster.org/pipermail/ohrrpgce-motherhamster.org/attachments/20170906/67c68716/attachment.html>


More information about the Ohrrpgce mailing list