Changeset d2647feb in sasview

Timestamp:

May 1, 2018 6:42:33 AM (6 years ago)

Author:

wojciech

Branches:

master, magnetic_scatt, release-4.2.2, ticket-1009, ticket-1094-headless, ticket-1242-2d-resolution, ticket-1243, ticket-1249, unittest-saveload

Children:

c9bf97b

Parents:

c82cf89

Message:

Logging support added

File:

• src/sas/sasview/sasview.py (modified) (3 diffs)

src/sas/sasview/sasview.py

@@ -198 +198 @@
 def check_sasmodels_compiler():
     """
-    Checking c compiler for sasmodels and trying to trigger xcode command line
-    for installation
-    """
+    Checking c compiler for sasmodels and raises xcode command line
+    tools for installation
+    """
+    #TODO: This doesn't work if someone uses different compiler e.g. gcc and has no cc installed. Will fix it
     import subprocess
     logger = logging.getLogger(__name__)
@@ -210 +211 @@
     except subprocess.CalledProcessError:
         raise RuntimeError("No compiler installed. Please follow installation instructions\n")
+        logger.error("No compiler installed. Please follow installation instructions\n")
+        logger.error(traceback.format_exc())
 
 def setup_sasmodels():
@@ -233 +236 @@
     setup_sasmodels()
     setup_wx()
-    check_sasmodels_compiler()
+    if sys.platform == "darwin":
+        check_sasmodels_compiler()
     SasView()