This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author neologix
Recipients neologix, phlogistonjohn, vinay.sajip
Date 2012-04-21.11:15:19
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1335006920.45.0.840355189671.issue14632@psf.upfronthosting.co.za>
In-reply-to
Content
Thanks for the report and patch.

I'm making Vinay nosy, as he's the logging expert.

A couple remarks:
- it would be nice to add your testcase to the logging tests (Lib/test/test_logging.py)
- I think you could probably save the whole fstat() result (instead of st_dev and st_inode): this would allow you to use os.pathsamestat() to check whether we're still refering to the same file, and it could maybe be useful someday (maybe use st_mtime)
- typo: "existance" -> "existence"
- I see you're catching OSError when calling fstat(): however, I don't see what error could be raised apart from EBADF (if the FD is closed, then it's likely a bug that shouldn't be masked, and it will blow up on the next write()), or I/O errors, that shouldn't be silenced
History
Date User Action Args
2012-04-21 11:15:20neologixsetrecipients: + neologix, vinay.sajip, phlogistonjohn
2012-04-21 11:15:20neologixsetmessageid: <1335006920.45.0.840355189671.issue14632@psf.upfronthosting.co.za>
2012-04-21 11:15:19neologixlinkissue14632 messages
2012-04-21 11:15:19neologixcreate