RE: svn commit: r984007 - /subversion/trunk/subversion/libsvn_wc/update_editor.c

> -----Original Message-----
> From: Greg Stein [mailto:gstein_at_gmail.com]
> Sent: woensdag 11 augustus 2010 21:30
> To: dev_at_subversion.apache.org
> Subject: Re: svn commit: r984007 -
> /subversion/trunk/subversion/libsvn_wc/update_editor.c
>
> I think you *always* want wq operations to be cancelable. That simply
> means that wq items will be left behind and "svn cleanup" will fix it.
> I see no problem with that.

The problem here is that once you press ^C all future calls to the global
cancel handler (in svn and other code) return SVN_ERR_CANCELED..

So when you press ^C during update, none of the wq operations are handled,
while before we introduced the WQ all loggy operations were just handled.

So pressing ^C likely locks your working copy in 1.7 (requiring cleanup),
while older versions did not.
(I found this issue when I added some additional tracing to detect where in
ra_serf we leaked these cancel errors)

        Bert
Received on 2010-08-11 21:48:45 CEST