Re: [PATCH 00/20] ref transactions part 2
I had a look at the changes in origin/pu and they look sane to me. make test passes all tests too. regards ronnie sahlberg On Wed, Jul 16, 2014 at 3:16 PM, Junio C Hamano wrote: > Ronnie Sahlberg writes: > >> On Tue, Jul 15, 2014 at 4:33 PM, Ronnie Sahlberg wrote: >>> This is the next 20 patches from my originally big patch series and follow >>> the previous 19 patches that is now in juns tree. >>> These patches were numbered 20-39 in the original 48-patch series. >>> >>> Changes since these patches were in the original series: >>> >>> - Addressing concerns from mhagger's review > > One patch in the series did not apply cleanly on top of the tip of > the previous series (now queued as rs/ref-transaction-0) and I had > to wiggle it. Please check the result (queued as three topics, this > one is rs/ref-transaction-1 which is built on the abovementioned > "-0", and the remainder from the previous round is rebased on "-1" > as rs/ref-transaction), all of which are queued on 'jch' (which is > part of 'pu'). > > Thanks. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 00/20] ref transactions part 2
Ronnie Sahlberg writes: > On Tue, Jul 15, 2014 at 4:33 PM, Ronnie Sahlberg wrote: >> This is the next 20 patches from my originally big patch series and follow >> the previous 19 patches that is now in juns tree. >> These patches were numbered 20-39 in the original 48-patch series. >> >> Changes since these patches were in the original series: >> >> - Addressing concerns from mhagger's review One patch in the series did not apply cleanly on top of the tip of the previous series (now queued as rs/ref-transaction-0) and I had to wiggle it. Please check the result (queued as three topics, this one is rs/ref-transaction-1 which is built on the abovementioned "-0", and the remainder from the previous round is rebased on "-1" as rs/ref-transaction), all of which are queued on 'jch' (which is part of 'pu'). Thanks. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 00/20] ref transactions part 2
Hi Michael, Here is the next set of 20 patches of those you already reviewed. I cut this patch off just before patch #40 in the previous series. These are the patches numbered 20-39 in the original series. I think I have addressed your concerns here so If you could take a quick look and hopefully bless them that would be awesome! Thanks ronnie sahlberg On Tue, Jul 15, 2014 at 4:33 PM, Ronnie Sahlberg wrote: > This is the next 20 patches from my originally big patch series and follow > the previous 19 patches that is now in juns tree. > These patches were numbered 20-39 in the original 48-patch series. > > Changes since these patches were in the original series: > > - Addressing concerns from mhagger's review > > > Ronnie Sahlberg (20): > refs.c: change ref_transaction_create to do error checking and return > status > refs.c: update ref_transaction_delete to check for error and return > status > refs.c: make ref_transaction_begin take an err argument > refs.c: add transaction.status and track OPEN/CLOSED/ERROR > tag.c: use ref transactions when doing updates > replace.c: use the ref transaction functions for updates > commit.c: use ref transactions for updates > sequencer.c: use ref transactions for all ref updates > fast-import.c: change update_branch to use ref transactions > branch.c: use ref transaction for all ref updates > refs.c: change update_ref to use a transaction > receive-pack.c: use a reference transaction for updating the refs > fast-import.c: use a ref transaction when dumping tags > walker.c: use ref transaction for ref updates > refs.c: make lock_ref_sha1 static > refs.c: remove the update_ref_lock function > refs.c: remove the update_ref_write function > refs.c: remove lock_ref_sha1 > refs.c: make prune_ref use a transaction to delete the ref > refs.c: make delete_ref use a transaction > > branch.c | 30 +++--- > builtin/commit.c | 24 +++-- > builtin/receive-pack.c | 96 +--- > builtin/replace.c | 15 +-- > builtin/tag.c | 15 +-- > builtin/update-ref.c | 11 ++- > fast-import.c | 53 +++ > refs.c | 242 > - > refs.h | 78 > sequencer.c| 27 -- > walker.c | 59 +++- > 11 files changed, 403 insertions(+), 247 deletions(-) > > -- > 2.0.1.442.g7fe6834.dirty > -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html