Clarify a few comments. NFC.
This commit is contained in:
parent
06c45c9ff0
commit
b44870e779
2 changed files with 3 additions and 6 deletions
|
|
@ -88,11 +88,8 @@ class BuildPlan:
|
|||
if run_script:
|
||||
if sys.platform.startswith("win32"):
|
||||
# Without "call", "cmd /c {}.bat" will return 0.
|
||||
# See https://stackoverflow.com/a/30736987 for a detailed
|
||||
# explanation of why, including disassembly/decompilation
|
||||
# of relevant code in cmd.exe.
|
||||
# Running the script manually from a command prompt is
|
||||
# unaffected- i.e. "call" is not required.
|
||||
# See https://stackoverflow.com/a/30736987 for a detailed explanation of why.
|
||||
# Running the script manually from a command prompt is unaffected.
|
||||
subprocess.check_call(["cmd", "/c", "call {}.bat".format(self.script)])
|
||||
else:
|
||||
subprocess.check_call(["sh", "{}.sh".format(self.script)])
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue