Open GoogleCodeExporter opened 9 years ago
[deleted comment]
Updated patch absent junk.
Original comment by g.rodola
on 17 Apr 2013 at 2:31
Attachments:
I guess as pyftpdlib 1.0.0 now supports Python 3, the "not PY3" ImportError
could be removed too? (Or be modified to say "needs pyftpdlib >= 1.0.0")
And then __test__ would additionally need to depend on which version of
pyftpdlib was imported.
P.S. Nice to see you here Giampaolo :)
Original comment by gc...@loowis.durge.org
on 17 Apr 2013 at 10:08
Yes, I suppose that can be safely removed.
Please note that I have not extensively tested this.
1.0.0 version of pyftpdlib has a filesystem class which now expects Unicode
instead of bytes and that is explicitly assert-ed in the code.
FWICT I ran fs/tests/test_ftpfs.py, then connected with an FTP client and
successfully listed a directory.
It looks like it works, but you know...
> P.S. Nice to see you here Giampaolo :)
Thanks. =)
Original comment by g.rodola
on 17 Apr 2013 at 10:33
Feel free to apply that, Andrew! Agree re Python 3 support.
Original comment by willmcgugan
on 17 Apr 2013 at 10:53
IIRC test_ftpfs.py only checks the fs/ftpfs.py code (i.e. 'FTP client' mode),
AFAIK there's not yet any tests for fs/expose/ftp.py (i.e. 'FTP server' mode).
Original comment by gc...@loowis.durge.org
on 17 Apr 2013 at 10:56
Original issue reported on code.google.com by
g.rodola
on 17 Apr 2013 at 2:20Attachments: