#849 kojira: sanity check in pruneLocalRepos
Merged 6 years ago by mikem. Opened 6 years ago by julian8628.
julian8628/koji kojira-sanity-check  into  master

file modified
+6
@@ -377,6 +377,12 @@ 

          if self.delete_pids:

              #skip

              return

+         if not os.path.exists(topdir):

+             self.logger.warn("%s doesn't exist, skipping", topdir)

+             return

+         if os.path.isfile(topdir):

+             self.logger.warn("%s is not directory, skipping", topdir)

+             return

          self.logger.debug("Scanning %s for repos", topdir)

          self.logger.debug('max age allowed: %s seconds (from %s)',

                  getattr(self.options, timername), timername)

a minor fix for not throwing Errors in pruneLocalRepos if topdir either doesn't exists or isn't a directory.

It's an odd place to catch this particular condition, but harmless I guess...

Commit f869eb4 fixes this pull-request

Pull-Request has been merged by mikem

6 years ago
Metadata