got rid of NODE and NODE_MODULES in settings.py; the cleanest way to deal with this is to setup PATH and NODE_PATH prior to running mathics/jupyter in a script

Author: poke1024

mathics/layout/client.py

@@ -1,9 +1,6 @@
 #!/usr/bin/env python
 # -*- coding: utf-8 -*-
 
-# you may use "NODE" in settings.py to specify a custom node binary, and you may use NODE_PATH in settings.py
-# to specify a custom node_modules path (that has the necessary node modules mathjax-node, ...).
-#
 # Your installation of nodejs with the following packages: mathjax-node svg2png (install them using
 # npm).
 
@@ -15,8 +12,6 @@
 from subprocess import Popen
 import os
 
-from mathics import settings
-
 import socket
 import json
 import struct
@@ -116,8 +111,6 @@ def __init__(self):
     def _create_client(self):
         try:
             popen_env = os.environ.copy()
-            if settings.NODE_MODULES:
-                popen_env["NODE_PATH"] = os.path.expandvars(settings.NODE_MODULES)
 
             server_path = os.path.join(
                 os.path.dirname(os.path.realpath(__file__)), 'server.js')
@@ -127,7 +120,7 @@ def abort(message):
                 raise WebEngineUnavailable(error_text + message)
 
             process = Popen(
-                [os.path.expandvars(settings.NODE), server_path],
+                ['node', server_path],
                 stdout=subprocess.PIPE,
                 env=popen_env)
 

mathics/settings.py

@@ -13,10 +13,6 @@
 DEBUG = True
 TEMPLATE_DEBUG = DEBUG
 
-# node.js based layout engine
-NODE = 'node'  # path to node binary; default 'node' assumes it is in PATH
-NODE_MODULES = None  # overrides NODE_PATH environment variable if not None
-
 # set only to True in DEBUG mode
 DEBUG_MAIL = True
 PROPAGATE_EXCEPTIONS = True