public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
* lwp-request stopped working with snapshots
@ 2014-10-14 18:22 Achim Gratz
  2014-10-14 19:34 ` Corinna Vinschen
  0 siblings, 1 reply; 8+ messages in thread
From: Achim Gratz @ 2014-10-14 18:22 UTC (permalink / raw)
  To: cygwin


I've ran into some strange problem with the snapshots.  This seems to be getting
complicated, so I'm throwing it out here in the hope someone has an
idea.  I've boiled it down to a minimal Cygwin (32bit, but it's
happening in 64bit too) installation, plus perl and perl_vendor.  As
installed

$ lwp-request http://server.local/

spits out the index page as it should.  Installing a snapshot, both the
earliest I still had locally available, which was 2014-08-19 and the
latest from 2014-10-11 produces the error:

"Transport endpoint not connected"

From the actual application that led me onto this hunt I know that the
request gets sent and the response is at least sometimes partially read
before the connection is destroyed.  The server seems to think the
client has closed the connection and the client thinks the same of the
server apparently.  This only happens when there is data sent from the
server, if I make the client wait for data it will dutifully sit there
until I send some and then things break down.

Now, I've been running exclusively on snapshots for quite some time and
this was working until about two weeks ago.  Looking at the updates
during that time I don't see anything obvious that would interfere with
LWP; I thought coreutils might and will still try to test this.  But I
don't really see how I'd need the combination of the snapshot
installation and one of the recently updated packages to cause this kind
of breakage, so if anyone has any ideas, please tell.


Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

Waldorf MIDI Implementation & additional documentation:
http://Synth.Stromeko.net/Downloads.html#WaldorfDocs

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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: lwp-request stopped working with snapshots
  2014-10-14 18:22 lwp-request stopped working with snapshots Achim Gratz
@ 2014-10-14 19:34 ` Corinna Vinschen
  2014-10-14 19:42   ` Achim Gratz
  0 siblings, 1 reply; 8+ messages in thread
From: Corinna Vinschen @ 2014-10-14 19:34 UTC (permalink / raw)
  To: cygwin

[-- Attachment #1: Type: text/plain, Size: 878 bytes --]

On Oct 14 20:21, Achim Gratz wrote:
> I've ran into some strange problem with the snapshots.
> [...]
> $ lwp-request http://server.local/
> 
> spits out the index page as it should.  Installing a snapshot, both the
> earliest I still had locally available, which was 2014-08-19 and the
> latest from 2014-10-11 produces the error:
> 
> "Transport endpoint not connected"

lwp-request is one of the tools using connect to try if a former
non-blocking connect worked.  This is ugly but, sigh, valid behaviour,

The latest changes to the socket code didn't take this scenario into
account.  I applied a fix and uploaded a new snashot to
https://cygwin.com/snapshots/

Please give it a try.


Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: lwp-request stopped working with snapshots
  2014-10-14 19:34 ` Corinna Vinschen
@ 2014-10-14 19:42   ` Achim Gratz
  2014-10-14 19:50     ` Corinna Vinschen
  0 siblings, 1 reply; 8+ messages in thread
From: Achim Gratz @ 2014-10-14 19:42 UTC (permalink / raw)
  To: cygwin

Corinna Vinschen writes:
> lwp-request is one of the tools using connect to try if a former
> non-blocking connect worked.  This is ugly but, sigh, valid behaviour,

What should it be doing instead?  LWP upstream has switched to a new
maintainer recently IIRC, so it might be a good time to suggest a few
cleanups.

> The latest changes to the socket code didn't take this scenario into
> account.  I applied a fix and uploaded a new snashot to
> https://cygwin.com/snapshots/
>
> Please give it a try.

I sure will do this first thing tomorrow, thanks!


Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

Factory and User Sound Singles for Waldorf Blofeld:
http://Synth.Stromeko.net/Downloads.html#WaldorfSounds

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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: lwp-request stopped working with snapshots
  2014-10-14 19:42   ` Achim Gratz
@ 2014-10-14 19:50     ` Corinna Vinschen
  2014-10-15  5:50       ` Achim Gratz
  2014-10-15 13:05       ` Andrey Repin
  0 siblings, 2 replies; 8+ messages in thread
From: Corinna Vinschen @ 2014-10-14 19:50 UTC (permalink / raw)
  To: cygwin

[-- Attachment #1: Type: text/plain, Size: 1145 bytes --]

On Oct 14 21:41, Achim Gratz wrote:
> Corinna Vinschen writes:
> > lwp-request is one of the tools using connect to try if a former
> > non-blocking connect worked.  This is ugly but, sigh, valid behaviour,
> 
> What should it be doing instead?  LWP upstream has switched to a new
> maintainer recently IIRC, so it might be a good time to suggest a few
> cleanups.

Calling select or poll instead.  But, anyway, uglyness is in the eye of
the beholder and ultimately it *is* a valid approach and it worked
before.  So it was clearly a newly introduced bug in Cygwin.

> > The latest changes to the socket code didn't take this scenario into
> > account.  I applied a fix and uploaded a new snashot to
> > https://cygwin.com/snapshots/
> >
> > Please give it a try.
> 
> I sure will do this first thing tomorrow, thanks!

Oh, good!  I just realized that I missed to handle EALREADY, so I
applied YA patch and just replaced the snapshot with a new one.


Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: lwp-request stopped working with snapshots
  2014-10-14 19:50     ` Corinna Vinschen
@ 2014-10-15  5:50       ` Achim Gratz
  2014-10-15  9:51         ` Achim Gratz
  2014-10-15 13:05       ` Andrey Repin
  1 sibling, 1 reply; 8+ messages in thread
From: Achim Gratz @ 2014-10-15  5:50 UTC (permalink / raw)
  To: cygwin

Corinna Vinschen <corinna-cygwin <at> cygwin.com> writes:
> Oh, good!  I just realized that I missed to handle EALREADY, so I
> applied YA patch and just replaced the snapshot with a new one.

The snapshot fixes things in the minial test installation.  I'll update my
other systems later today.


Regards,
Achim.


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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: lwp-request stopped working with snapshots
  2014-10-15  5:50       ` Achim Gratz
@ 2014-10-15  9:51         ` Achim Gratz
  0 siblings, 0 replies; 8+ messages in thread
From: Achim Gratz @ 2014-10-15  9:51 UTC (permalink / raw)
  To: cygwin

Achim Gratz <Stromeko <at> NexGo.DE> writes:
> The snapshot fixes things in the minial test installation.  I'll update my
> other systems later today.

The new snapshot works fine on all systems I've installed it on so far.


Regards,
Achim.



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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: lwp-request stopped working with snapshots
  2014-10-14 19:50     ` Corinna Vinschen
  2014-10-15  5:50       ` Achim Gratz
@ 2014-10-15 13:05       ` Andrey Repin
  2014-10-15 14:26         ` Corinna Vinschen
  1 sibling, 1 reply; 8+ messages in thread
From: Andrey Repin @ 2014-10-15 13:05 UTC (permalink / raw)
  To: Corinna Vinschen, cygwin

Greetings, Corinna Vinschen!

>> > The latest changes to the socket code didn't take this scenario into
>> > account.  I applied a fix and uploaded a new snashot to
>> > https://cygwin.com/snapshots/
>> >
>> > Please give it a try.
>> 
>> I sure will do this first thing tomorrow, thanks!

> Oh, good!  I just realized that I missed to handle EALREADY, so I
> applied YA patch and just replaced the snapshot with a new one.

For what my limited testing worth, the 64-bit build works without notable
issues.


--
WBR,
Andrey Repin (anrdaemon@yandex.ru) 15.10.2014, <16:47>

Sorry for my terrible english...


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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: lwp-request stopped working with snapshots
  2014-10-15 13:05       ` Andrey Repin
@ 2014-10-15 14:26         ` Corinna Vinschen
  0 siblings, 0 replies; 8+ messages in thread
From: Corinna Vinschen @ 2014-10-15 14:26 UTC (permalink / raw)
  To: cygwin

[-- Attachment #1: Type: text/plain, Size: 1189 bytes --]


Hi Achim,
Hi Andrey,

On Oct 15 09:51, Achim Gratz wrote:
> Achim Gratz <Stromeko <at> NexGo.DE> writes:
> > The snapshot fixes things in the minial test installation.  I'll update my
> > other systems later today.
> 
> The new snapshot works fine on all systems I've installed it on so far.
> 
> 
> Regards,
> Achim.

On Oct 15 17:02, Andrey Repin wrote:
> Greetings, Corinna Vinschen!
> 
> >> > The latest changes to the socket code didn't take this scenario into
> >> > account.  I applied a fix and uploaded a new snashot to
> >> > https://cygwin.com/snapshots/
> >> >
> >> > Please give it a try.
> >> 
> >> I sure will do this first thing tomorrow, thanks!
> 
> > Oh, good!  I just realized that I missed to handle EALREADY, so I
> > applied YA patch and just replaced the snapshot with a new one.
> 
> For what my limited testing worth, the 64-bit build works without notable
> issues.

thanks for testing!  If you see any other fishy connection problem,
please let me know.


Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2014-10-15 14:26 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-10-14 18:22 lwp-request stopped working with snapshots Achim Gratz
2014-10-14 19:34 ` Corinna Vinschen
2014-10-14 19:42   ` Achim Gratz
2014-10-14 19:50     ` Corinna Vinschen
2014-10-15  5:50       ` Achim Gratz
2014-10-15  9:51         ` Achim Gratz
2014-10-15 13:05       ` Andrey Repin
2014-10-15 14:26         ` Corinna Vinschen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).