Comment 1 for bug 450629

Revision history for this message
Sidnei da Silva (sidnei) wrote :

[1] parse_lsb_release: Please put the fd.close() in a 'finally:' statement.

[2] fetch_to_files: If the url ends with a '/', the filename will be an empty string:

>>> url = "X/"
>>> url.split("/")[-1]
''

[3] test_fetch_many_async: s/retrives/retrieves

Looks good! +1.