LDP
/
python-tldp
mirror of https://github.com/tLDP/python-tldp
0
1
Fork 0
Code Releases Activity

adjusting some logging and exceptions for verbosity/clarity

This commit is contained in:
Martin A. Brown 2016-02-15 21:15:29 -08:00
parent 9b59d99422
commit ae189e0d83
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
tldp/sources.py
View File

@ -25,8 +25,8 @@ class SourceDirs(object):
results = [os.path.exists(x) for x in self.sourcedirs]
if not all(results):
for result, sdir in zip(results, self.sourcedirs):
logger.critical("[Errno 2] No such file or directory: " + sdir)
raise OSError("[Errno 2] No such file or directory: " + sdir)
logger.critical("Directory does not exist: " + sdir)
raise OSError(errno.ENOENT, os.strerror(errno.ENOENT), sdir)
def enumerateDocuments(self):
for sdir in self.sourcedirs:
@ -56,9 +56,11 @@ class SourceDocument(object):
# -- canonicalize the pathname we are given.
self.filename = os.path.abspath(filename)
if not os.path.exists(self.filename):
raise OSError("Missing source document: " + self.filename)
logger.critical("Missing source document: %s", self.filename)
raise OSError(errno.ENOENT, os.strerror(errno.ENOENT), self.filename)
if not os.path.isfile(self.filename):
raise OSError("Wrong type, not a plain file: " + self.filename)
logger.critical("Source document is not a plain file: %s", self.filename)
raise TypeError("Wrong type, not a plain file: " + self.filename)
logger.debug("Found existing %s", self.filename)
self.dirname, self.basename = os.path.split(self.filename)