World: r3wp

:)

btw graham, do you happen to have heard of git?

repository thingy?

yep

used it, by any chance?

yes :)

nope ...

wow, fixed it :)

Umm... does Ubuntu use git ??

i use git

I know I installed something which failed to work for me ... in downloading 
from the ubuntu repository

and if you'd be using it too, developing protos together would be 
much more fun :)

so what's the fix?

dropped the SYST write from 230

I think that might help with directory parsing ...

made the last "read client" in the read event handler block conditional:
unless client/spec/ready [read client]

in case it's an Amiga ftp server ...

:)

returned "client/spec/ready" instead of "false" from the awake

one problem remains: the read after RETR does not return

hmm... my 0.0.5 added a return true

on a successful download

so just comment out line 261 ?

write-cmdport/only client [SYST]

so where do I set the state then?

here's a diff of my changes: http://gist.github.com/274796

that's a diff against the 0.0.5 from wik.is

wow .. you managed to nuke my 'read !

hehe :)

now what's expected sequence on the cmd connection for RETR?

ah!!

it completes :)

i'll have to print timestamps to even get an idea how long that stalls 
:)

but now i know that it's 150 for when the transfer starts and 226 
when it finishes

150 is just a mark ... comment

ok

all those can be ignored

can you email the new version?  mine is corrupted now ...with the 
manually applied diff's!

Or upload it ... using the credentials I sent you.

sec, still debugging

closed 12-Jan-2010/2:48:18+1:00
fin 12-Jan-2010/2:51:18+1:00

wow, stalls for precisely 3 minutes

ah, found it

it takes two to takedown a TCP connection

i.e. even if the server closes it's side, until the client also closes, 
the connection is still open

or even more plainly: in the dataport's close handler, you need to 
close the port as well

I close cmd ...

oh ... I read that I don't need to close the port ...

oh .. misunderstood ... what I read

if the dataport is closed on you, you need to close it as well