Fix detection of missing LLVM

README.md

@@ -141,6 +141,9 @@ The background pattern was downloaded from www.subtlepatterns.com
 Changelog
 =========
 
+1.3.8 (01/2016)
+    - Fix logging when LLVM is not installed
+
 1.3.7 (12/2015)
     - Added icon to use Continuous Signals from the GUI
 

opengeode/opengeode.py

@@ -117,7 +117,7 @@ except ImportError:
 
 
 __all__ = ['opengeode', 'SDL_Scene', 'SDL_View', 'parse']
-__version__ = '1.3.7'
+__version__ = '1.3.8'
 
 if hasattr(sys, 'frozen'):
     # Detect if we are running on Windows (py2exe-generated)
@@ -2171,34 +2171,34 @@ def init_logging(options):
 
     # Set log level for all libraries
     LOG.setLevel(level)
-    try:
-        modules = (
-            sdlSymbols,
-            genericSymbols,
-            ogAST,
-            ogParser,
-            Lander,
-            AdaGenerator,
-            undoCommands,
-            Renderer,
-            Clipboard,
-            Statechart,
-            Helper,
-            Asn1scc,
-            Connectors,
-            Pr,
-            TextInteraction,
-            Connectors,
-            LlvmGenerator,
-            CGenerator,
-            StgBackend
-        )
-        for module in modules:
-            module.LOG.addHandler(handler_console)
-            module.LOG.setLevel(level)
-    except (NameError, AttributeError) as err:
-        # Some modules may not be loaded (like llvm on Windows)
-        LOG.info(str(err))
+    modules = (
+        sdlSymbols,
+        genericSymbols,
+        ogAST,
+        ogParser,
+        Lander,
+        AdaGenerator,
+        undoCommands,
+        Renderer,
+        Clipboard,
+        Statechart,
+        Helper,
+        Asn1scc,
+        Connectors,
+        Pr,
+        TextInteraction,
+        Connectors,
+        LlvmGenerator,
+        CGenerator,
+        StgBackend
+    )
+    for each in modules:
+        try:
+            each.LOG.addHandler(handler_console)
+            each.LOG.setLevel(level)
+        except AttributeError as err:
+            # Discard unloaded modules (e.g. if LLVM is missing on target)
+            LOG.info(str(err))
 
 
 def parse(files):