Steve Alexander wrote:
> Shane Hathaway wrote:
>> os.stat() raises OSError if the file is not found, in which case mtime 
>> should be set to 0.
> Surely if the file is not found, that's an error because the 
> PageTemplateFile is pointing at a source file that doesn't exist.
> I cannot think of any reason I'd want not to be informed that the source 
> file for a PageTemplateFile isn't there. That's clearly a programming 
> error.
> So, I suggest removing the try: except: clause entirely, and letting 
> os.stat() raise its error.
> Or, am I missing something here?

I vaguely recall having a similar discussion with someone regarding 
DTMLFile, and we decided it had to ignore missing files, but I don't 
remember why.  Also, the open() call just below that line will raise an 
equivalent exception.  Use your judgment--mine is clouded. :-)


Zope-Dev maillist  -  [EMAIL PROTECTED]
**  No cross posts or HTML encoding!  **
(Related lists - )

Reply via email to