Michael Haggerty <mhagger@xxxxxxxxxxxx> writes: > On 09/13/2014 01:57 AM, Jonathan Nieder wrote: >> Michael Haggerty wrote: >>>> Jonathan Nieder <jrnieder@xxxxxxxxx> writes: >> >>>>> so I'll send a reroll of the series as-is in an hour or so. >>> >>> Jonathan: Is a current version of this patch series set up for review in >>> Gerrit? >> >> Yes. >> (https://code-review.googlesource.com/#/q/project:git+topic:ref-transaction) > > I just worked through the patch series, leaving lots of comments in > Gerrit. Overall it looks pretty good and makes a lot of very worthwhile > progress. The only patch that gives me a bit of heartburn is > > [PATCH 15/19] refs.c: fix handling of badly named refs > > not because it is necessarily wrong, but because it has a lot of > non-local effects that are hard to evaluate. I made a bunch of comments > in Gerrit about that patch, too, and will wait for a response before > having another go at it. > > Thanks for all your hard and detailed work, Ronnie and Jonathan! > > Michael Jonathan: Is a current version of this patch series set up to be fetched so that it can be reviewed outside Gerrit? Running ls-remote against https://code.googlesource.com/git shows many refs under refs/changes/* but it is unclear to me if there is a coherent single "here is the latest and greatest, dependents rebased on dependeds in the right order" thing that I can fetch and look at with "log -p master..". -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html