From 787c578ca247cb2bc200e78d69889710a1382bc0 Mon Sep 17 00:00:00 2001
From: Alec Leamas <leamas.alec@gmail.com>
Date: Nov 26 2015 16:24:19 +0000
Subject: Handle licensecheck crash gracefully (bz #1241481).


---

diff --git a/plugins/generic.py b/plugins/generic.py
index 3ee63ca..5a7dbda 100644
--- a/plugins/generic.py
+++ b/plugins/generic.py
@@ -706,7 +706,12 @@ class CheckLicenseField(GenericCheckBase):
             self.log.debug("Scanning sources in " + source_dir)
             if os.path.exists(source_dir):
                 cmd = 'licensecheck -r ' + source_dir
-                out = check_output(cmd, shell=True)
+                try:
+                    out = check_output(cmd, shell=True)
+                except (OSError, CalledProcessError) as err:
+                    self.set_passed(self.PENDING,
+                                    "Cannot run licensecheck: " + str(err))
+                    return
                 self.log.debug("Got license reply, length: %d" % len(out))
                 files_by_license = self._parse_licenses(out)
                 filename = os.path.join(ReviewDirs.root,
@@ -726,7 +731,7 @@ class CheckLicenseField(GenericCheckBase):
                                 len(files_by_license[self.unknown_license])
                 msg += ' Detailed output of licensecheck in ' + filename
             self.set_passed(self.PENDING, msg)
-        except OSError, e:
+        except OSError as e:
             self.log.error('OSError: %s' % str(e))
             msg = ' Programmer error: ' + e.strerror
             self.set_passed(self.PENDING, msg)