diff options
author | Alon Zakai <alonzakai@gmail.com> | 2012-09-22 12:12:25 -0700 |
---|---|---|
committer | Alon Zakai <alonzakai@gmail.com> | 2012-09-22 12:12:25 -0700 |
commit | 5c1d87ef5e3220b529733d2f983f0c64b3809de6 (patch) | |
tree | 2125da793f380e032ee54025835084ef9d8e80e5 | |
parent | a3e03954ae5a57cfad6cae7a1c364260c75cd2ea (diff) |
move settings.py to a less conspicuous place and rename it, to avoid confusion
-rwxr-xr-x | tests/runner.py | 8 | ||||
-rw-r--r-- | tools/settings_template_readonly.py (renamed from settings.py) | 0 | ||||
-rw-r--r-- | tools/shared.py | 4 |
3 files changed, 5 insertions, 7 deletions
diff --git a/tests/runner.py b/tests/runner.py index 49787383..cab5157b 100755 --- a/tests/runner.py +++ b/tests/runner.py @@ -4,8 +4,6 @@ ''' Simple test runner -See settings.py file for options¶ms. Edit as needed. - These tests can be run in parallel using nose, for example nosetests --processes=4 -v -s tests/runner.py @@ -62,7 +60,7 @@ from tools.shared import * try: assert COMPILER_OPTS != None except: - raise Exception('Cannot find "COMPILER_OPTS" definition. Is %s set up properly? You may need to copy the template from settings.py into it.' % EM_CONFIG) + raise Exception('Cannot find "COMPILER_OPTS" definition. Is %s set up properly? You may need to copy the template settings file into it.' % EM_CONFIG) # Core test runner class, shared between normal tests and benchmarks @@ -356,7 +354,7 @@ if 'benchmark' not in str(sys.argv) and 'sanity' not in str(sys.argv) and 'brows if Settings.USE_TYPED_ARRAYS: js_engines = filter(lambda engine: engine != V8_ENGINE, js_engines) # V8 issue 1822 js_engines = filter(lambda engine: engine not in self.banned_js_engines, js_engines) - if len(js_engines) == 0: return self.skip('No JS engine present to run this test with. Check %s and settings.py and the paths therein.' % EM_CONFIG) + if len(js_engines) == 0: return self.skip('No JS engine present to run this test with. Check %s and the paths therein.' % EM_CONFIG) for engine in js_engines: js_output = self.run_generated_code(engine, filename + '.o.js', args) if output_nicerizer is not None: @@ -8828,7 +8826,7 @@ elif 'sanity' in str(sys.argv): self.assertContained('make sure LLVM_ROOT and NODE_JS are correct', output) self.assertContained('This command will now exit. When you are done editing those paths, re-run it.', output) assert output.split()[-1].endswith('===='), 'We should have stopped: ' + output - assert (open(CONFIG_FILE).read() == open(path_from_root('settings.py')).read()), 'Settings should be copied from settings.py' + assert (open(CONFIG_FILE).read() == open(path_from_root('tools', 'settings_template_readonly.py')).read()), 'Settings should be copied from tools/settings_template_readonly.py' # Second run, with bad EM_CONFIG for settings in ['blah', 'LLVM_ROOT="blarg"; JS_ENGINES=[]; COMPILER_ENGINE=NODE_JS=SPIDERMONKEY_ENGINE=[]']: diff --git a/settings.py b/tools/settings_template_readonly.py index 45a30569..45a30569 100644 --- a/settings.py +++ b/tools/settings_template_readonly.py diff --git a/tools/shared.py b/tools/shared.py index 24e6b707..8adff34a 100644 --- a/tools/shared.py +++ b/tools/shared.py @@ -20,7 +20,7 @@ if '\n' in EM_CONFIG: else: CONFIG_FILE = os.path.expanduser(EM_CONFIG) if not os.path.exists(CONFIG_FILE): - shutil.copy(path_from_root('settings.py'), CONFIG_FILE) + shutil.copy(path_from_root('tools', 'settings_template_readonly.py'), CONFIG_FILE) print >> sys.stderr, ''' ============================================================================== Welcome to Emscripten! @@ -103,7 +103,7 @@ def check_sanity(force=False): try: subprocess.call([JAVA, '-version'], stdout=PIPE, stderr=PIPE) except: - print >> sys.stderr, 'WARNING: java does not seem to exist, required for closure compiler. -O2 and above will fail. You need to define JAVA in ~/.emscripten (see settings.py)' + print >> sys.stderr, 'WARNING: java does not seem to exist, required for closure compiler. -O2 and above will fail. You need to define JAVA in ~/.emscripten' if not os.path.exists(CLOSURE_COMPILER): print >> sys.stderr, 'WARNING: Closure compiler (%s) does not exist, check the paths in %s. -O2 and above will fail' % (CLOSURE_COMPILER, EM_CONFIG) |