Re: [PATCH] Replace some adm_access batons in externals.c

2009-12-21 Thread Philip Martin
Daniel Näslund writes: > Use recursive locking when deleting an external. No need to make a > distinction between files and dirs as the previous code did. > > * subversion/libsvn_client/externals.c > (handle_external_item_change): Remove the adm_access functions and use > svn_wc_{acqurire,r

Re: [PATCH] Replace some adm_access batons in externals.c

2009-12-19 Thread Daniel Näslund
On Fri, Dec 18, 2009 at 10:04:37AM +, Philip Martin wrote: > Daniel Näslund writes: > > > On Thu, Dec 17, 2009 at 06:10:18PM +, Philip Martin wrote: > >> Daniel Näslund writes: > >> > > >> > + SVN_ERR(svn_wc__release_write_lock(ib->ctx->wc_ctx, > >> > +

Re: [PATCH] Replace some adm_access batons in externals.c

2009-12-18 Thread Philip Martin
Daniel Näslund writes: > On Thu, Dec 17, 2009 at 06:10:18PM +, Philip Martin wrote: >> Daniel Näslund writes: >> > >> > + SVN_ERR(svn_wc__release_write_lock(ib->ctx->wc_ctx, >> > + remove_target_abspath, >> > +

Re: [PATCH] Replace some adm_access batons in externals.c

2009-12-17 Thread Daniel Näslund
On Thu, Dec 17, 2009 at 06:10:18PM +, Philip Martin wrote: > Daniel Näslund writes: > > > Index: subversion/libsvn_client/externals.c > > === > > --- subversion/libsvn_client/externals.c(revision 891778) > > +++ subversion/li

Re: [PATCH] Replace some adm_access batons in externals.c

2009-12-17 Thread Philip Martin
Daniel Näslund writes: > Index: subversion/libsvn_client/externals.c > === > --- subversion/libsvn_client/externals.c (revision 891778) > +++ subversion/libsvn_client/externals.c (arbetskopia) > @@ -894,53 +894,17 @@ >