Simplify logic for checking if report folder can be removed
This commit is contained in:
parent
e62f9bbbf1
commit
3d3d552579
@ -48,7 +48,8 @@ class AbiChecker(object):
|
||||
self.setup_logger()
|
||||
self.report_dir = os.path.abspath(report_dir)
|
||||
self.keep_all_reports = keep_all_reports
|
||||
self.should_keep_report_dir = os.path.isdir(self.report_dir)
|
||||
self.can_remove_report_dir = not (os.path.isdir(self.report_dir) or
|
||||
keep_all_reports)
|
||||
self.old_repo = old_repo
|
||||
self.old_rev = old_rev
|
||||
self.new_repo = new_repo
|
||||
@ -280,7 +281,7 @@ class AbiChecker(object):
|
||||
)
|
||||
os.remove(self.old_dumps[mbed_module])
|
||||
os.remove(self.new_dumps[mbed_module])
|
||||
if not self.should_keep_report_dir and not self.keep_all_reports:
|
||||
if self.can_remove_report_dir:
|
||||
os.rmdir(self.report_dir)
|
||||
self.log.info(compatibility_report)
|
||||
return compliance_return_code
|
||||
|
Loading…
Reference in New Issue
Block a user