YourWishes/jerryscript
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix several pylint warnings. (#1644)

Browse Source 
Fixed
 * 'unused-wildcard-import'
 * 'bad-whitespace'
 * 'bad-continuation'
 * 'wrong-import-order'
in 'build.py', 'run-tests.py' and 'check-license.py'.

Related issue: #1600

JerryScript-DCO-1.0-Signed-off-by: László Langó llango.u-szeged@partner.samsung.com
This commit is contained in:
László Langó
2017-03-13 11:18:49 +01:00
committed by GitHub
parent b140158104
commit 57ea06782b
3 changed files with 87 additions and 77 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tools/build.py
+16 -16
View File
@@ -16,22 +16,22 @@
import argparse import argparse
import multiprocessing import multiprocessing
import os
import shutil import shutil
import subprocess import subprocess
import sys import sys
from os import makedirs, uname import settings
from settings import *
BUILD_DIR = path.join(PROJECT_DIR, 'build') BUILD_DIR = os.path.join(settings.PROJECT_DIR, 'build')
DEFAULT_PORT_DIR = path.join(PROJECT_DIR, 'targets/default') DEFAULT_PORT_DIR = os.path.join(settings.PROJECT_DIR, 'targets/default')
PROFILE_DIR = path.join(PROJECT_DIR, 'jerry-core/profiles') PROFILE_DIR = os.path.join(settings.PROJECT_DIR, 'jerry-core/profiles')
DEFAULT_PROFILE = 'es5.1' DEFAULT_PROFILE = 'es5.1'
def default_toolchain(): def default_toolchain():
(sysname, _, _, _, machine) = uname() (sysname, _, _, _, machine) = os.uname()
toolchain = path.join(PROJECT_DIR, 'cmake', 'toolchain_%s_%s.cmake' % (sysname.lower(), machine.lower())) toolchain = os.path.join(settings.PROJECT_DIR, 'cmake', 'toolchain_%s_%s.cmake' % (sysname.lower(), machine.lower()))
return toolchain if path.isfile(toolchain) else None return toolchain if os.path.isfile(toolchain) else None
def get_arguments(): def get_arguments():
devhelp_preparser = argparse.ArgumentParser(add_help=False) devhelp_preparser = argparse.ArgumentParser(add_help=False)
@@ -112,10 +112,10 @@ def generate_build_options(arguments):
build_options.append('-DMEM_HEAP_SIZE_KB=%d' % arguments.mem_heap) build_options.append('-DMEM_HEAP_SIZE_KB=%d' % arguments.mem_heap)
build_options.append('-DPORT_DIR=%s' % arguments.port_dir) build_options.append('-DPORT_DIR=%s' % arguments.port_dir)
if path.isabs(arguments.profile): if os.path.isabs(arguments.profile):
PROFILE = arguments.profile PROFILE = arguments.profile
else: else:
PROFILE = path.join(PROFILE_DIR, arguments.profile + '.profile') PROFILE = os.path.join(PROFILE_DIR, arguments.profile + '.profile')
build_options.append('-DFEATURE_PROFILE=%s' % PROFILE) build_options.append('-DFEATURE_PROFILE=%s' % PROFILE)
@@ -145,21 +145,21 @@ def generate_build_options(arguments):
return build_options return build_options
def configure_output_dir(arguments): def configure_output_dir(arguments):
if not path.isabs(arguments.builddir): if not os.path.isabs(arguments.builddir):
arguments.builddir = path.join(PROJECT_DIR, arguments.builddir) arguments.builddir = os.path.join(settings.PROJECT_DIR, arguments.builddir)
if arguments.clean and path.exists(arguments.builddir): if arguments.clean and os.path.exists(arguments.builddir):
shutil.rmtree(arguments.builddir) shutil.rmtree(arguments.builddir)
if not path.exists(arguments.builddir): if not os.path.exists(arguments.builddir):
makedirs(arguments.builddir) os.makedirs(arguments.builddir)
def configure_build(arguments): def configure_build(arguments):
configure_output_dir(arguments) configure_output_dir(arguments)
build_options = generate_build_options(arguments) build_options = generate_build_options(arguments)
cmake_cmd = ['cmake', '-B' + arguments.builddir, '-H' + PROJECT_DIR] cmake_cmd = ['cmake', '-B' + arguments.builddir, '-H' + settings.PROJECT_DIR]
cmake_cmd.extend(build_options) cmake_cmd.extend(build_options)
return subprocess.call(cmake_cmd) return subprocess.call(cmake_cmd)
tools/check-license.py
+2 -2
View File
@@ -17,7 +17,7 @@
import os import os
import re import re
import sys import sys
from settings import * import settings
license = re.compile( license = re.compile(
@@ -48,7 +48,7 @@ dirs = [
exclude_dirs = [ exclude_dirs = [
'targets/esp8266', 'targets/esp8266',
os.path.relpath (TEST262_TEST_SUITE_DIR, PROJECT_DIR), os.path.relpath (settings.TEST262_TEST_SUITE_DIR, settings.PROJECT_DIR),
] ]
exts = [ exts = [
tools/run-tests.py
+42 -32
View File
@@ -18,9 +18,9 @@ import argparse
import os import os
import subprocess import subprocess
import sys import sys
from settings import * import settings
OUTPUT_DIR = path.join(PROJECT_DIR, 'build', 'tests') OUTPUT_DIR = os.path.join(settings.PROJECT_DIR, 'build', 'tests')
parser = argparse.ArgumentParser() parser = argparse.ArgumentParser()
parser.add_argument('--toolchain', action='store', default='', help='Add toolchain file') parser.add_argument('--toolchain', action='store', default='', help='Add toolchain file')
@@ -49,10 +49,10 @@ if len(sys.argv) == 1:
script_args = parser.parse_args() script_args = parser.parse_args()
if path.isabs(script_args.outdir): if os.path.isabs(script_args.outdir):
OUTPUT_DIR = script_args.outdir OUTPUT_DIR = script_args.outdir
else: else:
OUTPUT_DIR = path.join(PROJECT_DIR, script_args.outdir) OUTPUT_DIR = os.path.join(settings.PROJECT_DIR, script_args.outdir)
class Options: class Options:
def __init__(self, name='', build_args=None, test_args=None): def __init__(self, name='', build_args=None, test_args=None):
@@ -62,7 +62,7 @@ class Options:
if test_args is None: if test_args is None:
test_args = [] test_args = []
self.out_dir = path.join(OUTPUT_DIR, name) self.out_dir = os.path.join(OUTPUT_DIR, name)
self.build_args = build_args self.build_args = build_args
self.build_args.append('--builddir=%s' % self.out_dir) self.build_args.append('--builddir=%s' % self.out_dir)
self.test_args = test_args self.test_args = test_args
@@ -95,13 +95,11 @@ jerry_test_suite_options.append(Options('jerry_test_suite-es2015-subset-snapshot
jerry_test_suite_options.append(Options('jerry_test_suite-es2015-subset-debug-snapshot', ['--debug', '--profile=es2015-subset', '--snapshot-save=on', '--snapshot-exec=on'], ['--snapshot'])) jerry_test_suite_options.append(Options('jerry_test_suite-es2015-subset-debug-snapshot', ['--debug', '--profile=es2015-subset', '--snapshot-save=on', '--snapshot-exec=on'], ['--snapshot']))
# Test options for test262 # Test options for test262
test262_test_suite_options = [ test262_test_suite_options = [Options('test262_tests')]
Options('test262_tests'),
]
# Test options for jerry-debugger # Test options for jerry-debugger
debugger_test_options = [ debugger_test_options = [
Options('jerry_debugger_tests', ['--debug', '--jerry-debugger=on', '--jerry-libc=off']), Options('jerry_debugger_tests', ['--debug', '--jerry-debugger=on', '--jerry-libc=off'])
] ]
# Test options for buildoption-test # Test options for buildoption-test
@@ -119,13 +117,13 @@ jerry_buildoptions = [
] ]
def get_bin_dir_path(out_dir): def get_bin_dir_path(out_dir):
return path.join(out_dir, 'bin') return os.path.join(out_dir, 'bin')
def get_binary_path(out_dir): def get_binary_path(out_dir):
return path.join(get_bin_dir_path(out_dir), 'jerry') return os.path.join(get_bin_dir_path(out_dir), 'jerry')
def create_binary(buildoptions): def create_binary(buildoptions):
build_cmd = [BUILD_SCRIPT] build_cmd = [settings.BUILD_SCRIPT]
build_cmd.extend(buildoptions) build_cmd.extend(buildoptions)
if script_args.toolchain: if script_args.toolchain:
@@ -160,16 +158,17 @@ def run_jerry_debugger_tests():
if ret_build: if ret_build:
break break
for file in os.listdir(DEBUGGER_TESTS_DIR): for file in os.listdir(settings.DEBUGGER_TESTS_DIR):
if file.endswith(".js"): if file.endswith(".js"):
test_case, _ = os.path.splitext(file) test_case, _ = os.path.splitext(file)
test_case_path = os.path.join (DEBUGGER_TESTS_DIR, test_case) test_case_path = os.path.join(settings.DEBUGGER_TESTS_DIR, test_case)
test_cmd = [ test_cmd = [
DEBUGGER_TEST_RUNNER_SCRIPT, settings.DEBUGGER_TEST_RUNNER_SCRIPT,
get_binary_path(job.out_dir), get_binary_path(job.out_dir),
DEBUGGER_CLIENT_SCRIPT, settings.DEBUGGER_CLIENT_SCRIPT,
os.path.relpath (test_case_path, PROJECT_DIR), os.path.relpath(test_case_path, settings.PROJECT_DIR),
] ]
if job.test_args: if job.test_args:
test_cmd.extend(job.test_args) test_cmd.extend(job.test_args)
@@ -184,7 +183,11 @@ def run_jerry_tests():
if ret_build: if ret_build:
break break
test_cmd = [TEST_RUNNER_SCRIPT, get_binary_path(job.out_dir), JERRY_TESTS_DIR] test_cmd = [
settings.TEST_RUNNER_SCRIPT,
get_binary_path(job.out_dir),
settings.JERRY_TESTS_DIR
]
skip_list = [] skip_list = []
if '--profile=es2015-subset' not in job.build_args: if '--profile=es2015-subset' not in job.build_args:
@@ -210,14 +213,14 @@ def run_jerry_test_suite():
if ret_build: if ret_build:
break break
test_cmd = [TEST_RUNNER_SCRIPT, get_binary_path(job.out_dir)] test_cmd = [settings.TEST_RUNNER_SCRIPT, get_binary_path(job.out_dir)]
if '--profile=minimal' in job.build_args: if '--profile=minimal' in job.build_args:
test_cmd.append(JERRY_TEST_SUITE_MINIMAL_LIST) test_cmd.append(settings.JERRY_TEST_SUITE_MINIMAL_LIST)
elif '--profile=es2015-subset' in job.build_args: elif '--profile=es2015-subset' in job.build_args:
test_cmd.append(JERRY_TEST_SUITE_DIR) test_cmd.append(settings.JERRY_TEST_SUITE_DIR)
else: else:
test_cmd.append(JERRY_TEST_SUITE_ES51_LIST) test_cmd.append(settings.JERRY_TEST_SUITE_ES51_LIST)
if script_args.skip_list: if script_args.skip_list:
test_cmd.append("--skip-list=" + script_args.skip_list) test_cmd.append("--skip-list=" + script_args.skip_list)
@@ -236,7 +239,11 @@ def run_test262_test_suite():
if ret_build: if ret_build:
break break
test_cmd = [TEST262_RUNNER_SCRIPT, get_binary_path(job.out_dir), TEST262_TEST_SUITE_DIR] test_cmd = [
settings.TEST262_RUNNER_SCRIPT,
get_binary_path(job.out_dir),
settings.TEST262_TEST_SUITE_DIR
]
if job.test_args: if job.test_args:
test_cmd.extend(job.test_args) test_cmd.extend(job.test_args)
@@ -252,7 +259,10 @@ def run_unittests():
if ret_build: if ret_build:
break break
ret_test |= run_check([UNITTEST_RUNNER_SCRIPT, get_bin_dir_path(job.out_dir)]) ret_test |= run_check([
settings.UNITTEST_RUNNER_SCRIPT,
get_bin_dir_path(job.out_dir)
])
return ret_build | ret_test return ret_build | ret_test
@@ -268,28 +278,28 @@ def main():
ret = 0 ret = 0
if script_args.check_signed_off_tolerant: if script_args.check_signed_off_tolerant:
ret = run_check([SIGNED_OFF_SCRIPT, '--tolerant']) ret = run_check([settings.SIGNED_OFF_SCRIPT, '--tolerant'])
if not ret and script_args.check_signed_off_travis: if not ret and script_args.check_signed_off_travis:
ret = run_check([SIGNED_OFF_SCRIPT, '--travis']) ret = run_check([settings.SIGNED_OFF_SCRIPT, '--travis'])
if not ret and (script_args.all or script_args.check_signed_off): if not ret and (script_args.all or script_args.check_signed_off):
ret = run_check([SIGNED_OFF_SCRIPT]) ret = run_check([settings.SIGNED_OFF_SCRIPT])
if not ret and (script_args.all or script_args.check_cppcheck): if not ret and (script_args.all or script_args.check_cppcheck):
ret = run_check([CPPCHECK_SCRIPT]) ret = run_check([settings.CPPCHECK_SCRIPT])
if not ret and (script_args.all or script_args.check_doxygen): if not ret and (script_args.all or script_args.check_doxygen):
ret = run_check([DOXYGEN_SCRIPT]) ret = run_check([settings.DOXYGEN_SCRIPT])
if not ret and (script_args.all or script_args.check_pylint): if not ret and (script_args.all or script_args.check_pylint):
ret = run_check([PYLINT_SCRIPT]) ret = run_check([settings.PYLINT_SCRIPT])
if not ret and (script_args.all or script_args.check_vera): if not ret and (script_args.all or script_args.check_vera):
ret = run_check([VERA_SCRIPT]) ret = run_check([settings.VERA_SCRIPT])
if not ret and (script_args.all or script_args.check_license): if not ret and (script_args.all or script_args.check_license):
ret = run_check([LICENSE_SCRIPT]) ret = run_check([settings.LICENSE_SCRIPT])
if not ret and (script_args.all or script_args.jerry_debugger): if not ret and (script_args.all or script_args.jerry_debugger):
ret = run_jerry_debugger_tests() ret = run_jerry_debugger_tests()