launchpad-reviewers team mailing list archive

[İlker Emre Koç <mp+494749@xxxxxxxxxxxxxxxxxx>]

İlker Emre Koç has proposed merging ~ilkeremrekoc/launchpad:cve-lib-del-print into launchpad:master.

Commit message:
Comment out print statements in updated cve-lib

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~ilkeremrekoc/launchpad/+git/launchpad/+merge/494749

These print statements get called out in a lot tests and as a result
break the tests that check for output.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~ilkeremrekoc/launchpad:cve-lib-del-print into launchpad:master.

diff --git a/lib/contrib/cve_lib.py b/lib/contrib/cve_lib.py
index a39af97..afaf036 100644
--- a/lib/contrib/cve_lib.py
+++ b/lib/contrib/cve_lib.py
@@ -36,8 +36,8 @@ def set_cve_dir(path):
             p = tmp
             #print("INFO: using '%s'" % p, file=sys.stderr)
 
-    if not found and not path.endswith("ignored"):
-        print("WARN: could not find CVEs in '%s' (or relative to UCT)" % path, file=sys.stderr)
+    # if not found and not path.endswith("ignored"):
+        # print("WARN: could not find CVEs in '%s' (or relative to UCT)" % path, file=sys.stderr)
     return p
 
 if 'UCT' in os.environ: