DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:from:to:cc:subject:date:message-id:references
	:in-reply-to:content-type:content-id:content-transfer-encoding
	:mime-version; q=dns; s=default; b=Y6g0FWp62S6GA2zH343Fxpo/dlpRY
	sgRlBJB+Tj4u3ge8e5ZB+EMoiU+0z0nl/Zkleye455Jj8vkh1G8QzC1I98QNFy4g
	/IDsf1mT5eKMuIN7VXfwAI2Naw7kyRkrwHgKlpzvjBJKfAPcDi9Rg1AiOyLWu2m5
	4Q0Znw5XJHc/6U=
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
From: Ken Brown <kbrown@cornell.edu>
To: "cygwin@cygwin.com" <cygwin@cygwin.com>
CC: Stephen Provine <stephpr@microsoft.com>
Subject: Re: Future setup regression caused by 'mkdir: always check-for-existence' commit
Date: Mon, 26 Aug 2019 18:24:08 +0000
Message-ID: <d8ae01c9-ddf5-560e-2542-25a8d6943184@cornell.edu>
References: <MWHPR21MB08452B51987E8A239851F7BEB9A10@MWHPR21MB0845.namprd21.prod.outlook.com> <860a3bd2-fc37-27c0-54f3-0fce8add7c24@cornell.edu> <MWHPR21MB0845636B4E90AAC4F4BF959EB9A10@MWHPR21MB0845.namprd21.prod.outlook.com>
In-Reply-To: <MWHPR21MB0845636B4E90AAC4F4BF959EB9A10@MWHPR21MB0845.namprd21.prod.outlook.com>
user-agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0
received-spf: None (protection.outlook.com: cornell.edu does not designate permitted sender hosts)
Content-Type: text/plain; charset="Windows-1252"
Content-ID: <11A18AA4924FD24FA36086B748176411@namprd04.prod.outlook.com>
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-userprincipalname: xSK0We3Rgq7JbnbVKF5Vbm8mF2h93utYlfBCNshCeUjQwBwaYI2SiHqRcspjuDWm3GNbrFap5J9z4wHrYbewsw==
X-IsSubscribed: yes
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from quoted-printable to 8bit by delorie.com id x7QIOndW021143

[Please don't top-post on this list.  Thanks.]

On 8/26/2019 1:33 PM, Stephen Provine via cygwin wrote:
> It's possible that this issue contributes to the problem, but the fact that you can manually create the "dev" directory outside of Cygwin (just mkdir from Windows command prompt) to fix the problem tells me otherwise. If I understand the logic change in commit b0c033bf3fae810b9e5a5c69f17bd4de63725691, it would indicate to me that "mkdir 0755 /dev" in a post-install bash script (regardless of which one) would no longer work when it previously worked.

I don't think you understand the logic change in that commit.  An attempt to 
create an existing directory would already fail before that commit; the failure 
would occur in the call to fh->mkdir().  The commit in question simply 
guarantees that the error code is EEXIST.  Take a look at the commit message.

Also, what postinstall script are you referring to that calls "mkdir /dev" 
without either checking for existence or using "mkdir -p"?  Cygwin's bash 
postinstall script doesn't do this.

(You must have missed Eric's reply to your original post, in which he asked the 
same question.)

> I'd be happy to try the test release for cygwin-3.1.0, but I don't know how to find it. Can you provide a link?

Just run's Cygwin's setup program, select the cygwin package, and select the 
test release (currently 3.1.0-0.2).

Ken

--
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