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. :-)

Shane



_______________________________________________
Zope-Dev maillist  -  [EMAIL PROTECTED]
http://lists.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists - 
 http://lists.zope.org/mailman/listinfo/zope-announce
 http://lists.zope.org/mailman/listinfo/zope )

Reply via email to