This is the mail archive of the cygwin mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [1.7.0-50] scp progress counter flies through first 175 MB or so


On Mon, Jun 29, 2009 at 04:38:40PM +0200, Corinna Vinschen wrote:
>On Jun 29 10:23, Christopher Faylor wrote:
>> On Mon, Jun 29, 2009 at 11:53:22AM +0200, Corinna Vinschen wrote:
>> >On Jun 29 11:28, Corinna Vinschen wrote:
>> >>On Jun 28 15:31, Christopher Faylor wrote:
>> >>>Btw, Corinna, were you proposing turning the "FIXME" code in peek_pipe
>> >>>back on?
>> >>[...]
>> >>I tested sftp with big files (Gigs) and it appears to work fine with
>> >>the FIXME code switched back to life.  I tested the unison command
>> >>using Cygwin's unison 2.31.4-2 under tcsh and it worked fine.  I tested
>> >>various combinations with pipes created by a native application and
>> >>with native applications on send and receive side of a Cygwin pipe and
>> >>all my testcases worked as expected.
>> >>
>> >>So, maybe we should really give it a try again.
>> >
>> >Btw., I also tried the rsync of 9000 files with the FIXME code active
>> >and it worked a couple of times in a row without fail.
>> 
>> I did make a change subsequent to your turning off this code.  Maybe
>> that was enough to get things working.  Either that or, if you are using
>> the trunk code, maybe you aren't hitting this at all anymore since there
>> are overlapped I/O accomodations.
>> 
>> Anyway, I wouldn't mind giving the code another try if you are up for
>> it.  Although I didn't write the code originally, it has always bothered
>> me that it didn't work because it seems like it should have.
>> 
>> Shall I turn it on, create a snapshot, and then ask our legion of
>> testers to try it?
>
>Sure, go ahead.  I would even go so far as to create a new -51 test
>release to get some more testing of this.

How about if we do that after seeing what happens with the snapshot
(especially since I already sent the email :-|)?

If we get good preliminary reports then a new version is certainly
called for.

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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]