Python 3.0rc1 (r30rc1:66507, Sep 18 2008, 14:47:08) [MSC v.1500 32 bit
(Intel)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import ftplib
>>> f = ftplib.FTP()
>>> f.connect('mirrors.kernel.org')
'220 Welcome to mirrors.kernel.org.'
>>> f.login()
'230 Login successful.'
>>> f.debugging = 1
>>> f.makeport()
*cmd* 'PORT 10,0,0,1,18.21875,56'
*resp* '500 Illegal PORT command.'
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\python30\lib\ftplib.py", line 300, in makeport
    resp = self.sendport(host, port)
  File "C:\python30\lib\ftplib.py", line 260, in sendport
    return self.voidcmd(cmd)
  File "C:\python30\lib\ftplib.py", line 249, in voidcmd
    return self.voidresp()
  File "C:\python30\lib\ftplib.py", line 224, in voidresp
    resp = self.getresp()
  File "C:\python30\lib\ftplib.py", line 219, in getresp
    raise error_perm(resp)
ftplib.error_perm: 500 Illegal PORT command.
>>>

Path in attachment.

Why not just use floor divide (//)?

At your discretion. I was thinking that it's not encouraging that such
an outstanding bug has passed silently until RC1. IMHO, a minimal test
suite for the ftplib module would be really necessary.

A dummy FTP server returning fixed response codes could be arranged and
used to test the basic FTP class methods.

On Fri, Sep 19, 2008 at 6:23 PM, Giampaolo Rodola'
<report@bugs.python.org> wrote:

Giampaolo Rodola' <billiejoex@users.sourceforge.net> added the comment:

At your discretion. I was thinking that it's not encouraging that such
an outstanding bug has passed silently until RC1. IMHO, a minimal test
suite for the ftplib module would be really necessary.

Yes, testing of some of these modules is quite sad.

A dummy FTP server returning fixed response codes could be arranged and
used to test the basic FTP class methods.

Would you like to contribute a patch?

---

Python tracker <report@bugs.python.org>
<http://bugs.python.org/issue3911\>

---

Would you like to contribute a patch?

Ok.
I started working on a patch which implements a dummy asyncore-based FTP
server including tests for all basic FTP() class methods.
I'll contribute a patch as soon as I'll wrote IPv6 tests.

Would you like to contribute a patch?

Done in issue bpo-3939.

Fixed in r66656.