From: Thanh Ha <thanh.ha@linuxfoundation.org>
Date: Mon, 27 Aug 2018 01:25:24 +0000 (-0400)
Subject: Use logger instead of print in version module
X-Git-Tag: v0.17.0~3
X-Git-Url: https://gerrit.linuxfoundation.org/infra/gitweb?a=commitdiff_plain;h=d10d79b4b8b0a95da454aa05436990c608247182;p=releng%2Flftools.git

Use logger instead of print in version module

Change-Id: I31329886088d8d7a469e19bc0937912546400109
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
---

diff --git a/lftools/cli/version.py b/lftools/cli/version.py
index c2a893ea..f94c1305 100644
--- a/lftools/cli/version.py
+++ b/lftools/cli/version.py
@@ -12,12 +12,15 @@
 __author__ = 'Thanh Ha'
 
 
+import logging
 import os
 import subprocess
 import sys
 
 import click
 
+log = logging.getLogger(__name__)
+
 
 @click.group()
 @click.pass_context
@@ -97,7 +100,7 @@ def patch(ctx, release_tag, patch_dir, project):
     using RELEASE_TAG in order to version bump by x.y.(z+1)-SNAPSHOT.
     """
     if not os.path.isdir(patch_dir):
-        print("{} is not a valid directory.".format(patch_dir))
+        log.error("{} is not a valid directory.".format(patch_dir))
         sys.exit(404)
     status = subprocess.call(['version', 'patch', release_tag, patch_dir, project])
     sys.exit(status)