Skip to content

Commit 2e14c6d

Browse files
authored
Move EMCMAKE/EMCONFIGURE globals to test code. NFC (#23764)
This is the only place they are used.
1 parent 71e31f4 commit 2e14c6d

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

test/common.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@
3434
import clang_native
3535
import jsrun
3636
import line_endings
37-
from tools.shared import EMCC, EMXX, DEBUG, EMCONFIGURE, EMCMAKE
37+
from tools.shared import EMCC, EMXX, DEBUG
3838
from tools.shared import get_canonical_temp_dir, path_from_root
3939
from tools.utils import MACOS, WINDOWS, read_file, read_binary, write_binary, exit_with_error
4040
from tools.settings import COMPILE_TIME_SETTINGS
@@ -85,6 +85,8 @@
8585

8686
EMBUILDER = shared.bat_suffix(path_from_root('embuilder'))
8787
EMMAKE = shared.bat_suffix(path_from_root('emmake'))
88+
EMCMAKE = shared.bat_suffix(path_from_root('emcmake'))
89+
EMCONFIGURE = shared.bat_suffix(path_from_root('emconfigure'))
8890
EMRUN = shared.bat_suffix(shared.path_from_root('emrun'))
8991
WASM_DIS = Path(building.get_binaryen_bin(), 'wasm-dis')
9092
LLVM_OBJDUMP = os.path.expanduser(shared.build_llvm_tool_path(shared.exe_suffix('llvm-objdump')))

test/test_other.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
from tools.building import get_building_env
3232
from tools.shared import config
3333
from tools.shared import EMCC, EMXX, EMAR, EMRANLIB, FILE_PACKAGER, LLVM_NM
34-
from tools.shared import CLANG_CC, CLANG_CXX, LLVM_AR, LLVM_DWARFDUMP, LLVM_DWP, EMCMAKE, EMCONFIGURE, WASM_LD
34+
from tools.shared import CLANG_CC, CLANG_CXX, LLVM_AR, LLVM_DWARFDUMP, LLVM_DWP, WASM_LD
3535
from common import RunnerCore, path_from_root, is_slow_test, ensure_dir, disabled, make_executable
3636
from common import env_modify, no_mac, no_windows, only_windows, requires_native_clang, with_env_modify
3737
from common import create_file, parameterized, NON_ZERO, node_pthreads, TEST_ROOT, test_file
@@ -40,7 +40,7 @@
4040
from common import also_with_standalone_wasm, also_with_wasm2js, also_with_noderawfs
4141
from common import also_with_modularize, also_with_wasmfs, with_all_fs
4242
from common import also_with_minimal_runtime, also_with_wasm_bigint, also_with_wasm64, also_with_asan, flaky
43-
from common import EMTEST_BUILD_VERBOSE, PYTHON, WEBIDL_BINDER
43+
from common import EMTEST_BUILD_VERBOSE, PYTHON, WEBIDL_BINDER, EMCMAKE, EMCONFIGURE
4444
from common import requires_network, parameterize, copytree
4545
from tools import shared, building, utils, response_file, cache
4646
from tools.utils import read_file, write_file, delete_file, read_binary, MACOS, WINDOWS

tools/shared.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -773,8 +773,6 @@ class OFormat(Enum):
773773
EMXX = bat_suffix(path_from_root('em++'))
774774
EMAR = bat_suffix(path_from_root('emar'))
775775
EMRANLIB = bat_suffix(path_from_root('emranlib'))
776-
EMCMAKE = bat_suffix(path_from_root('emcmake'))
777-
EMCONFIGURE = bat_suffix(path_from_root('emconfigure'))
778776
EM_NM = bat_suffix(path_from_root('emnm'))
779777
FILE_PACKAGER = bat_suffix(path_from_root('tools/file_packager'))
780778
WASM_SOURCEMAP = bat_suffix(path_from_root('tools/wasm-sourcemap'))

0 commit comments

Comments
 (0)