[Libreoffice-commits] dev-tools.git: esc-reporting/esc-analyze.py

[Shinnok]

 esc-reporting/esc-analyze.py |    7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

New commits:
commit c596332399de89fd561d39837b4fc0d1beac2d6f
Author: Shinnok <admin at shinnok.com>
Date:   Wed Dec 20 12:39:47 2017 +0100

    Cleanup missing license reports in the daily mails.
    
    By cross-referencing against licenseCompany.csv domain too.
    
    Change-Id: I4060e63d0d121825e1ce0b2e3342bb4d3f7e0ccc
    Reviewed-on: https://gerrit.libreoffice.org/46843
    Reviewed-by: Michael Meeks <michael.meeks at collabora.com>
    Tested-by: Michael Meeks <michael.meeks at collabora.com>

diff --git a/esc-reporting/esc-analyze.py b/esc-reporting/esc-analyze.py
index f3de599..e0458d3 100755
--- a/esc-reporting/esc-analyze.py
+++ b/esc-reporting/esc-analyze.py
@@ -608,6 +608,9 @@ def util_is_company_license(email):
         domain = line[:line.index(' ')]
       if email.endswith(domain):
         return True
+    for domain in cfg['companies']:
+      if email.endswith(domain):
+        return True
     return False
 
 def analyze_reports():
@@ -687,7 +690,9 @@ def analyze_reports():
           entry['email'] = ownerEmail
           entry['license'] = 'GERRIT NO LICENSE'
           statList['reportList']['missing_license'].append(entry)
-        elif not statList['people'][ownerEmail]['licenseOK']:
+        elif not statList['people'][ownerEmail]['licenseOK']\
+          and not util_is_company_license(ownerEmail):
+          print(ownerEmail)
           entry['license'] = 'GERRIT: ' + statList['people'][ownerEmail]['licenseText']
           statList['reportList']['missing_license'].append(entry)