build(log): add cmake option ALSO_LOG_TO_STDERR

allow turning on/off glog's alsologtostderr flag in custom build.
by default OFF in release build, ON in debug build.

Fixes #799

CMakeLists.txt

@@ -20,6 +20,7 @@ option(BUILD_SAMPLE "Build sample Rime plugin" OFF)
 option(BUILD_TEST "Build and run tests" ON)
 option(BUILD_SEPARATE_LIBS "Build separate rime-* libraries" OFF)
 option(ENABLE_LOGGING "Enable logging with google-glog library" ON)
+option(ALSO_LOG_TO_STDERR "Log to stderr as well as log file" OFF)
 option(ENABLE_ASAN "Enable Address Sanitizer (Unix Only)" OFF)
 option(INSTALL_PRIVATE_HEADERS "Install private headers (usually needed for externally built Rime plugins)" OFF)
 option(ENABLE_EXTERNAL_PLUGINS "Enable loading of externally built Rime plugins (from directory set by RIME_PLUGINS_DIR variable)" OFF)
@@ -86,6 +87,9 @@ if(ENABLE_LOGGING)
 
   set(RIME_ENABLE_LOGGING 1)
 
+  if (ALSO_LOG_TO_STDERR)
+     set(RIME_ALSO_LOG_TO_STDERR 1)
+  endif()
 endif()
 
 find_package(Threads)

Makefile

@@ -85,6 +85,7 @@ debug:
 	-DCMAKE_INSTALL_PREFIX=$(prefix) \
 	-DCMAKE_BUILD_TYPE=Debug \
 	-DBUILD_MERGED_PLUGINS=OFF \
+	-DALSO_LOG_TO_STDERR=ON \
 	-DENABLE_EXTERNAL_PLUGINS=ON
 	cmake --build $(build)
 

src/rime/build_config.h.in

@@ -6,6 +6,7 @@
 #define RIME_BUILD_CONFIG_H_
 
 #cmakedefine RIME_ENABLE_LOGGING
+#cmakedefine RIME_ALSO_LOG_TO_STDERR
 
 #cmakedefine RIME_DATA_DIR "@RIME_DATA_DIR@"
 #cmakedefine RIME_PLUGINS_DIR "@RIME_PLUGINS_DIR@"

src/rime/setup.cc

@@ -73,7 +73,9 @@ RIME_API void SetupLogging(const char* app_name,
                            const char* log_dir) {
 #ifdef RIME_ENABLE_LOGGING
   FLAGS_minloglevel = min_log_level;
+#ifdef RIME_ALSO_LOG_TO_STDERR
   FLAGS_alsologtostderr = true;
+#endif  // RIME_ALSO_LOG_TO_STDERR
   if (log_dir) {
     if (log_dir[0] == '\0') {
       google::LogToStderr();