Return-Path: X-Original-To: apmail-subversion-users-archive@minotaur.apache.org Delivered-To: apmail-subversion-users-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 18BC7981D for ; Sat, 2 Jun 2012 20:46:09 +0000 (UTC) Received: (qmail 35594 invoked by uid 500); 2 Jun 2012 20:46:08 -0000 Delivered-To: apmail-subversion-users-archive@subversion.apache.org Received: (qmail 35578 invoked by uid 500); 2 Jun 2012 20:46:08 -0000 Mailing-List: contact users-help@subversion.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Delivered-To: mailing list users@subversion.apache.org Received: (qmail 35569 invoked by uid 99); 2 Jun 2012 20:46:08 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 02 Jun 2012 20:46:08 +0000 X-ASF-Spam-Status: No, hits=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: local policy) Received: from [192.109.42.8] (HELO einhorn.in-berlin.de) (192.109.42.8) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 02 Jun 2012 20:45:57 +0000 X-Envelope-From: stsp@stsp.name Received: from ted.stsp.name (ted.stsp.name [217.197.84.34]) by einhorn.in-berlin.de (8.13.6/8.13.6/Debian-1) with ESMTP id q52KjYWi006394 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT); Sat, 2 Jun 2012 22:45:34 +0200 Received: from ted.stsp.name (stsp@localhost [127.0.0.1]) by ted.stsp.name (8.14.5/8.14.3) with ESMTP id q52KjYPD025851; Sat, 2 Jun 2012 22:45:34 +0200 (CEST) Received: (from stsp@localhost) by ted.stsp.name (8.14.5/8.14.3/Submit) id q52KjXu9002791; Sat, 2 Jun 2012 22:45:33 +0200 (CEST) Date: Sat, 2 Jun 2012 22:45:33 +0200 From: Stefan Sperling To: Andreas Krey Cc: "Michael P. Reilly" , users@subversion.apache.org Subject: Re: Can't kill svn Message-ID: <20120602204533.GF6277@ted.stsp.name> Mail-Followup-To: Andreas Krey , "Michael P. Reilly" , users@subversion.apache.org References: <20120601191347.GF10680@inner.h.iocl.org> <20120602091416.GA6277@ted.stsp.name> <20120602200142.GK10680@inner.h.iocl.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20120602200142.GK10680@inner.h.iocl.org> User-Agent: Mutt/1.5.21 (2010-09-15) X-Scanned-By: MIMEDefang_at_IN-Berlin_e.V. on 192.109.42.8 On Sat, Jun 02, 2012 at 10:01:42PM +0200, Andreas Krey wrote: > On Sat, 02 Jun 2012 11:14:16 +0000, Stefan Sperling wrote: > ... > > Some signals never have an immediate effect in Subversion. > > Subversion handles signals gracefully at defined points to ensure state > > is cleaned up properly. > > Hanging more than a minute isn't exactly what I consider 'graceful'. > > > When the signal is received, a flag is set that > > is checked next time Subversion invokes the cancellation handler which > > then cleans up state and causes an exit. > > No, it doesn't. With 1.7.4: > > | a:cc-svn-7 andreaskrey$ svn17 up -r85321 > | Updating '.': > | D tools > | U ivy.xml > | U src/.... > | U . > | > | Fetching external item into 'ioc': > | D moda/CHANGES > | U moda/h/... > (redacted output lists) > | U moda/src/... > | ^Csvn: E200015: Caught signal > > Ok, it did terminate, and fast. > > But there is no 'graceful' in here: > > | a:cc-svn-7 andreaskrey$ svn17 up -r85321 > | Updating '.': > | > | Fetching external item into 'ioc': > | svn: warning: W155004: Working copy '/Users/andreaskrey/cc-svn-7/ioc' locked. > > ...and that situation persists. "graceful" != "requiring manual > intervention, as with 'svn cleanup'". > > I only see the point of an arbitrary wait when svn then at least leaves > the sandbox in a state that doesn't require cleanup. > > Incidentally, I only found that because I wanted to see if there is > still the cascade of > > | svn: warning: Error handling externals definition for 'moda': > | svn: warning: Caught signal > | svn: warning: Error handling externals definition for 'modb': > | svn: warning: Caught signal > | svn: warning: Error handling externals definition for 'etc': > | svn: warning: Caught signal > | svn: warning: Error handling externals definition for 'ant-scripts': > | svn: warning: Caught signal > > for a single ^C. Instead the wc is borked with: > > | svn: Failed to add directory 'tools': an unversioned directory of the same name already exists > > and a spurious conflict on svn:externals on the root (this with 1.6.6). > (svn is 1.6.6, svn17 is 1.7.4.) Yes, that looks like bad behaviour (externals again... *sigh*) Can you please file detailed reports for these problems in the issue tracker so they don't get lost? Thanks.