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:subject:date:message-id:references
	:in-reply-to:content-type:content-transfer-encoding
	:mime-version; q=dns; s=default; b=BgA8eMTMbyu86avh0gYjuUOJe5160
	P/31LSipSSHffEg8qGEoMHrnGQIKIgqp1BPLaQrhfK/e/j5KEK0+tEeH65MSLeYe
	9p1/ZkFeTub/W6q6txrqNqDYRhoVgBnhrrMpjC/5WAe8XkZcdP+b9zWpPBNFKBAr
	P9MvCwVNEWdBvs=
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
From: Bryan Phelps <bryphe AT outlook DOT com>
To: "cygwin AT cygwin DOT com" <cygwin AT cygwin DOT com>
Subject: Re: Bug Report: Regression in Cygwin 2.11.0-1
Date: Sat, 1 Sep 2018 20:08:01 +0000
Message-ID: <BL2PR16MB096188578C751002BF54F8E5C80E0@BL2PR16MB0961.namprd16.prod.outlook.com>
References: <20180901155435 DOT GL6350 AT calimero DOT vinschen DOT de>
In-Reply-To: <20180901155435.GL6350@calimero.vinschen.de>
received-spf: None (protection.outlook.com: outlook.com does not designate permitted sender hosts)
Content-Type: text/plain; charset="iso-8859-1"
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit

> As for the bug in question:  I pushed a patch which should fix this 
> issue.  I created new developer snapshots and uploaded them to 
> https://cygwin.com/snapshots/.  Please give them a try. 

Thank you Corinna for the quick fix and investigation!

I set up an environment to test it out: https://github.com/bryphe/test-ocaml-build and ran a build against the patched dll.
It looks like the patch fixes our scenario - the build is green with the patched dll: https://ci.appveyor.com/project/bryphe/test-ocaml-build, so the patch fixes our problem.

> On the other hand, those that will find problems, the people building
packages, will only do that on "official" releases.

I'm pretty new to the Cygwin ecosystem - we could certainly create nightly builds that test against cygwin releases, if that helps (and adjust our CLI script to grab the nightly / latest test release of Cygwin).

I wonder if, for a change like this, we could add a test case that exercises it - looking through the source, it looks like cygload.cc has a few test cases around pathing - perhaps it would be helpful to add a regression test for the minimal repro?

Thanks again for the help and investigation!
Bryan
--
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