From 8d3dc7551dee75ec50a300c50be3dd278acea43e Mon Sep 17 00:00:00 2001 From: Feiyun Wang Date: Fri, 31 Mar 2023 23:58:54 +0800 Subject: [PATCH] Fix #1059: Linking for mingw/x86_64 on MSYS2 fails, w/ ar execution speed up --- tools/my_spawn.py | 45 +++++++++++++++++++++++++-------------------- 1 file changed, 25 insertions(+), 20 deletions(-) diff --git a/tools/my_spawn.py b/tools/my_spawn.py index 542d5135..5f5ffa69 100644 --- a/tools/my_spawn.py +++ b/tools/my_spawn.py @@ -11,34 +11,39 @@ def configure(env): import subprocess def mySubProcess(cmdline, env): - # print("SPAWNED: " + cmdline) - proc = subprocess.run( - args=cmdline, - shell=True, - env=env, - ) + # print(cmdline) + proc = subprocess.run(args=cmdline, shell=True, env=env) rv = proc.returncode if rv: print("=====") - print(rv, "(", hex(rv), ")") + print("subprocess.run().returncode=", rv, "(", hex(rv), ")") + print("len(cmdline)=", len(cmdline)) print("=====") return rv def mySpawn(sh, escape, cmd, args, env): - - newargs = " ".join(args[1:]) - cmdline = cmd + " " + newargs - - rv = 0 - if len(cmdline) > 32000 and cmd.endswith("ar"): - cmdline = cmd + " " + args[1] + " " + args[2] + " " - for i in range(3, len(args)): - rv = mySubProcess(cmdline + args[i], env) - if rv: - break + if len(args) > 3 and cmd.endswith("ar"): + # print("Long ar command is split.\nargc=", len(args)) + lead = len(" ".join(args[0:3])) + begin = 3 + length = lead + 1 + len(args[begin]) + rv = 0 + for i in range(4, len(args)): + length += 1 + len(args[i]) + if length >= 8 * 1024 - 32: + cmdline = " ".join(args[0:3] + args[begin:i]) + # print("objs=", i - begin, ", length=", len(cmdline)) + rv = mySubProcess(cmdline, env) + if rv: + break + begin = i + length = lead + 1 + len(args[i]) + if not rv: + cmdline = " ".join(args[0:3] + args[begin:]) + # print("objs=", len(args) - begin, ", length=", len(cmdline)) + rv = mySubProcess(cmdline, env) else: - rv = mySubProcess(cmdline, env) - + rv = mySubProcess(args, env) return rv env["SPAWN"] = mySpawn