On 01/20/2017 08:51 AM, Eric Blake wrote:
> On 01/18/2017 11:22 PM, Steven Penny wrote:
>> On Tue, 17 Jan 2017 21:45:07, Steven Penny wrote:
>>> I would like to keep this productive; my next step is to try to unpack that
>>> large commit, while also helping Eric to reproduce.
>>
>> I have root caused this. With the new readline, running ./configure creates
>> config.h, as expected. However new version adds this line:
>>
>> #define HAVE_PSELECT 1
>>
>> If you remove this from config.h, then make, the resultant DLL behaves as it did
>> with previous readline. Not sure what is causing this change, I will try to dig
>> into it more.
> 
> I've uploaded a test release of readline-7.0.1-2 which disables pselect
> support. I would appreciate if you could test that it fixes things for
> you without using the official release cygwin1.dll (although you have

s/without//

> already tested that the recent test build of cygwin also fixes things).

(that is, please test that bash-4.4.5-1, cygwin-2.6.1-1, and
libreadline7-7.0.1-2 should work; you've already tested that
bash-4.4.5-1, libreadline7-7.0.1-1, and cygwin1-20170119 snapshot
worked; and likewise mixing libreadline7-7.0.1-2 and the snapshot should
work although it is less important)

> 
> Once we have another cygwin release, I'll build readline-7.0.1-3 which
> re-enables pselect.
> 

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org