Message50210
Logged In: YES
user_id=642936
On (1): agreed.
On (2): There is still a debug message emitted for a bad
checksum: In TarInfo.frombuf() at the bottom a ValueError is
raised if they don't match and is passed on to
TarFile.next() where it is put out as a debug message using
the _dbg() method in the except clause. The debug message
where it is now (r46040) is senseless because the try-block
will be left when TarInfo.frombuf() fails .
On (3): You're right, I attached a patch that adds another
Exception HeaderError which is raised in TarInfo.frombuf()
instead of ValueError in case of a bad header. I hope that
is acceptable. |
|
Date |
User |
Action |
Args |
2007-08-23 15:48:46 | admin | link | issue1484695 messages |
2007-08-23 15:48:46 | admin | create | |
|