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 Jun 29 10:42, Christopher Faylor wrote:
> 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.

Fine with me.


Corinna

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

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