From 4e4a80330eb657a3c074a53b7b739df0469b5591 Mon Sep 17 00:00:00 2001
From: Yuanle Song <sylecn@gmail.com>
Date: Wed, 26 Apr 2017 19:12:46 +0800
Subject: [PATCH] v0.19.1 do not try to run sudo in configlogger.py

This may require user interactive and is very bad behavior on import/app
runs.

also update default log level for non-prod env to DEBUG.
---
 project.clj                              |  2 +-
 src/leiningen/new/python/configlogger.py | 19 +++----------------
 2 files changed, 4 insertions(+), 17 deletions(-)

diff --git a/project.clj b/project.clj
index 7347f1a..7dfcf67 100644
--- a/project.clj
+++ b/project.clj
@@ -1,4 +1,4 @@
-(defproject python/lein-template "0.19.0"
+(defproject python/lein-template "0.19.1"
   :description "lein template for a python project"
   :repositories [["snapshots"
                   {:url "http://devserv.game.yy.com/nexus/content/repositories/snapshots"
diff --git a/src/leiningen/new/python/configlogger.py b/src/leiningen/new/python/configlogger.py
index 8587bbc..4cdb907 100644
--- a/src/leiningen/new/python/configlogger.py
+++ b/src/leiningen/new/python/configlogger.py
@@ -28,26 +28,13 @@ def load_logger_config():
         fileConfig(resource_filename("{{name}}", "logger.conf"))
         return
 
-    try:
-        os.makedirs(logdir)
-        fileConfig(resource_filename("{{name}}", "logger.conf"))
-        return
-    except OSError as _:
-        pass
-
-    import subprocess
-    exit_code = subprocess.call(["sudo", "mkdir", "-p", logdir])
-    if exit_code == 0:
-        fileConfig(resource_filename("{{name}}", "logger.conf"))
-        return
-
-    TESTING = os.getenv("PYTEST") == "1"
-    level = logging.DEBUG if TESTING else logging.INFO
+    ENV = os.getenv("ENV", "test")
+    level = logging.INFO if ENV == "prod" else logging.DEBUG
     logging.basicConfig(
         format='%(asctime)s [%(module)s] %(levelname)-8s %(message)s',
         level=level)
     logging.info("Create log dir %s failed. Using basic config, level=%s",
-                 logdir, "DEBUG" if TESTING else "INFO")
+                 logdir, level)
 
 
 logging.captureWarnings(True)
-- 
GitLab