wllvm and gllvm have the same logging API. I need it to see why gllvm, clang, and wllvm get different results when configuring musllvm.

Author: ianamason

README.md

@@ -241,17 +241,25 @@ Debugging
 ---------
 
 The WLLVM tools can show various levels of output to aid with debugging.
-To show this output set WLLVM_OUTPUT to one of the following levels:
+To show this output set the `WLLVM_OUTPUT_LEVEL` environment
+variable to one of the following levels:
 
- * `CRITICAL`
  * `ERROR`
  * `WARNING`
  * `INFO`
  * `DEBUG`
 
-For example
+For example:
+```
+    export WLLVM_OUTPUT_LEVEL=DEBUG
+```
+Output will be directed to the standard error stream, unless you specify the
+path of a logfile via the `WLLVM_OUTPUT_FILE` environment variable.
 
-    export WLLVM_OUTPUT=DEBUG
+For example:
+```
+    export WLLVM_OUTPUT_FILE=/tmp/wllvm.log
+```
 
 
 Sanity Checking

wllvm/compilers.py

@@ -35,6 +35,7 @@ def wcompile(mode):
     except Exception as e:
         _logger.debug('%s: exception case: %s', mode, str(e))
 
+    _logger.info('Calling %s returned %d',  list(sys.argv), rc)  
     return rc
 
 
@@ -204,9 +205,9 @@ def getBuilder(cmd, mode):
     compilerEnv = 'LLVM_COMPILER'
     cstring = os.getenv(compilerEnv)
     pathPrefix = os.getenv(llvmCompilerPathEnv) # Optional
-    _logger.info('WLLVM compiler using %s', cstring)
+    _logger.debug('WLLVM compiler using %s', cstring)
     if pathPrefix:
-        _logger.info('WLLVM compiler path prefix "%s"', pathPrefix)
+        _logger.debug('WLLVM compiler path prefix "%s"', pathPrefix)
 
     if cstring == 'clang':
         return ClangBuilder(cmd, mode, pathPrefix)

wllvm/logconfig.py

@@ -7,23 +7,32 @@
 import os
 import sys
 
-_loggingEnv = 'WLLVM_OUTPUT'
+# iam: 6/30/2017 decided to move to a gllvm style where we can set the level and the output file
+_loggingEnvLevel_old = 'WLLVM_OUTPUT'
+_loggingEnvLevel_new = 'WLLVM_OUTPUT_LEVEL'
 
-_validLogLevels = ['CRITICAL', 'ERROR', 'WARNING', 'INFO', 'DEBUG']
+_loggingDestination = 'WLLVM_OUTPUT_FILE'
+
+_validLogLevels = ['ERROR', 'WARNING', 'INFO', 'DEBUG']
 
 def logConfig(name):
 
-    logging.basicConfig(level=logging.WARNING, format='%(levelname)s:%(message)s')
+    destination = os.getenv(_loggingDestination)
+
+    if destination:
+        logging.basicConfig(filename=destination, level=logging.WARNING, format='%(levelname)s:%(message)s')
+    else:
+        logging.basicConfig(level=logging.WARNING, format='%(levelname)s:%(message)s')
 
     retval = logging.getLogger(name)
 
-    level = os.getenv(_loggingEnv)
+    level = os.getenv(_loggingEnvLevel_old) or os.getenv(_loggingEnvLevel_new) 
 
     if level:
         level = level.upper()
         if not level in _validLogLevels:
-            logging.error('"%s" is not a valid value for %s. Valid values are %s',
-                          level, _loggingEnv, _validLogLevels)
+            logging.error('"%s" is not a valid value for %s or %s. Valid values are %s',
+                          level, _loggingEnvLevel_old, _loggingEnvLevel_new, _validLogLevels)
             sys.exit(1)
         else:
             retval.setLevel(getattr(logging, level))