From: Jonathan Nieder <jrnieder@gmail.com>
Date: Tue, 13 May 2014 20:25:10 GMT
Subject: Re: [PATCH v6 00/42] Use ref transactions for all ref updates
Message-ID: <20140513202510.GW9218@google.com>
In-Reply-To: <1398976662-6962-1-git-send-email-sahlberg@google.com>

Hi,

Ronnie Sahlberg wrote:

> This patch series is based on next and expands on the transaction API.

Sorry to take so long to get to this.

For the future, it's easier to review patches based on some particular
branch that got merged into next, since next is a moving target
(series come and go from there depending on what seems to need testing
at a given moment).  Is mh/ref-transaction the relevant branch to
build on?

Trying to apply the series to mh/ref-transaction, I get conflicts in
patch 13 due to absence of rs/ref-update-check-errors-early.

Trying to apply the series to a merge of mh/ref-transaction and
rs/ref-update-check-errors-early, I get a minor conflict in patch 15
but it is easy to resolve and the rest goes smoothly.

Looking forward to reading the rest.  Thanks.
Jonathan