On Sat, 07 Jan 2012 07:53:54 -0400, David Bremner <david@tethera.net> wrote: > On Fri, 6 Jan 2012 10:03:19 +0000, David Edmondson <dme@dme.org> wrote: > > --- > > > > - Prefix the branch name with 'review/' > > - Avoid `shell-command', which also results in better error reporting > > when 'git-am' fails. > > > > One thing I noticed is that reviewing a patch for the second time fails > because the branch already exists. I'm not sure if this is covered under > "upkeep of the repo" but it is a little inconvenient. Oh. It's supposed to delete the existing branch. It did in my test. What happens for you?