mirror of
https://github.com/IfcOpenShell/IfcOpenShell.git
synced 2026-09-09 21:53:40 +00:00
build-all - use shutil.which instead of custom function
To make it more idiomatic and custom function also was misinterpreting folders with the same name as existing executables.
This commit is contained in:
+1
-10
@@ -203,15 +203,6 @@ def cecho(message, color=NO_COLOR):
|
|||||||
logger.info(f"{color}{message}\033[0m")
|
logger.info(f"{color}{message}\033[0m")
|
||||||
|
|
||||||
|
|
||||||
def which(cmd: str) -> Union[str, None]:
|
|
||||||
PATH = os.getenv("PATH")
|
|
||||||
assert PATH
|
|
||||||
for path in PATH.split(":"):
|
|
||||||
if os.path.exists(path) and cmd in os.listdir(path):
|
|
||||||
return cmd
|
|
||||||
return None
|
|
||||||
|
|
||||||
|
|
||||||
# Flags.
|
# Flags.
|
||||||
APPLE = platform.system() == "Darwin"
|
APPLE = platform.system() == "Darwin"
|
||||||
MAC_CROSS_COMPILE_INTEL = "mac-cross-compile-intel" in flags
|
MAC_CROSS_COMPILE_INTEL = "mac-cross-compile-intel" in flags
|
||||||
@@ -441,7 +432,7 @@ if "wasm" in flags:
|
|||||||
required_commands.remove(yacc)
|
required_commands.remove(yacc)
|
||||||
|
|
||||||
for cmd in required_commands:
|
for cmd in required_commands:
|
||||||
if which(cmd) is None:
|
if shutil.which(cmd) is None:
|
||||||
missing_commands.append(cmd)
|
missing_commands.append(cmd)
|
||||||
|
|
||||||
if missing_commands:
|
if missing_commands:
|
||||||
|
|||||||
Reference in New Issue
Block a user