[tor-commits] [sbws/master] Clarify: no, there doesn't seem to be a nice way to do this

[pastly at torproject.org]

commit 59e4d88add83fc8014b419cde07b61809a0f2f39
Author: Matt Traudt <sirmatt at ksu.edu>
Date:   Wed Jun 6 21:40:05 2018 -0400

    Clarify: no, there doesn't seem to be a nice way to do this
---
 sbws/config.log.default.ini | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/sbws/config.log.default.ini b/sbws/config.log.default.ini
index 6c51326..3d8d9bd 100644
--- a/sbws/config.log.default.ini
+++ b/sbws/config.log.default.ini
@@ -58,6 +58,8 @@ class = handlers.RotatingFileHandler
 formatter = sbwssys
 # when not using level, it seems to be using the one passed as cli arg,
 # which is the desired behaviour
-# dunno if there is a way to have log file as a variable
-# setting one as default
+# There doesn't seem to be a way to put ${paths:log_filepath} here since the
+# logging library eval()s this, which doesn't give the ini parsing library a
+# chance to replace the text. So we do it in configure_logging() in
+# sbws/util/config.py instead.
 args = ('/nonexist',)