X-Recipient: archive-cygwin@delorie.com
DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:subject:to:references:from:message-id:date
	:mime-version:in-reply-to:content-type:content-transfer-encoding
	:reply-to; q=dns; s=default; b=t7qKW3mZ3hb6DKp2yW4uymR31km1pJ04X
	+o2jrVAIlb+FnSpZ4OXGQ1xNDeVbFNAVK5jm+cETbUHvZzQ5w6lups27nMHeqYns
	KpL8RNe5mRGoGZQak3DoBwEyV8YzyTe7Isg1U6AqSRKsc+g7zibc0YO5Jru5QBLH
	Vessb8tr8o=
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:subject:to:references:from:message-id:date
	:mime-version:in-reply-to:content-type:content-transfer-encoding
	:reply-to; s=default; bh=TJEJRdsfNCbPQLhrS7L2RiYpFkU=; b=upQMMY2
	RlO7/i/CNxrEEd9XCa9tvPGSjO+WTo651UM+0v06PVZdu/vmqym8VVHGWQM29jgH
	U31pansTdtNjjqzp7pbW90KelvRgJtTtnfRHGLvXBK69RjO1GbaNu/yQm/3lU96B
	Rlpq9v6WrZ5BGLb7VLuIY3n+ysDB+wAPiNxo=
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe@cygwin.com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin@cygwin.com>
List-Help: <mailto:cygwin-help@cygwin.com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
Delivered-To: mailing list cygwin@cygwin.com
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=-6.7 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_2,RCVD_IN_DNSWL_LOW,SPF_SOFTFAIL autolearn=ham version=3.3.2 spammy=Hx-spam-relays-external:ESMTPA
X-HELO: resqmta-po-06v.sys.comcast.net
Subject: Re: subversion problem
To: cygwin@cygwin.com
References: <9c2c7358137b4513a7b8f5fc08386354@vsrv060ex03.ssd.fsi.com> <77a982f8-d617-7a1d-2ac1-c0843ba30c94@acm.org> <xolin8jpcbe.wl-atara-y@mx.scn.tv> <xolh8nzph27.wl-atara-y@mx.scn.tv>
From: David Rothenberger <daveroth@acm.org>
Message-ID: <e35d02bd-b86f-f045-c48e-beb582a2a44c@acm.org>
Date: Wed, 25 Apr 2018 09:59:43 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0
MIME-Version: 1.0
In-Reply-To: <xolh8nzph27.wl-atara-y@mx.scn.tv>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 7bit
X-CMAE-Envelope: MS4wfEtLq1PjnnvPV6MYhp8Jb2kP2eBKt49Ej3roK7QRWMwQCfg2tyv+0ubympdzmKxk8CUMMltoInsmhDhsa3dAOLj6nWw1ASBJf2vPVhJUHTLq0TvCusBy Y8uSfJZLLsf0aNiMciCJSfdr3HxJlMum/Zs3TfTX9v4KpMpiZuT15LFc
X-IsSubscribed: yes
Reply-To: cygwin@cygwin.com

Yasutaka Atarashi wrote:
> I had some doubt that the modification suggested in my mail:
> Cygwin subversion and UNC path https://cygwin.com/ml/cygwin/2018-04/msg00271.html
> would cause the failure.
> However, this would not be the case.

Yes, I agree your change is not the source of the problem.

> 
> I'm afraid to say that it seems to be an oversight in 05-retry-loop.patch.
> 
> The first call of the loop target function must be made BEFORE the call of WIN32_RETRY_LOOP() while Cygwin branch of svn_io_file_rename2() calls WIN32_RETRY_LOOP() directly.

Thanks, I agree this is the likely source of the problem. I won't have time to try it until this weekend. Thanks for investigating.


-- 
David Rothenberger  ----  daveroth@acm.org


--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

