From 62f00dad22bc657290780dc65c6f1b8ac6e88f9b Mon Sep 17 00:00:00 2001 From: Sean Brogan Date: Thu, 22 Sep 2022 04:44:59 +0800 Subject: BaseTools: Edk2ToolsBuild: Fixing pipeline build due to path too long Current implementation of looking up toolchain will _insert_ the findings from vsvarsall.bat to existing path and potentially stuff the variable to exceed the length of maximal path length accepted by Windows. This change updated the logic to use the discovered shell varialbes to replace the existing path, which is desirable in the specific use case. Cc: Bob Feng Cc: Liming Gao Cc: Yuwei Chen Co-authored-by: Sean Brogan Signed-off-by: Kun Qin Reviewed-by: Sean Brogan Reviewed-by: Bob Feng --- BaseTools/Edk2ToolsBuild.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'BaseTools') diff --git a/BaseTools/Edk2ToolsBuild.py b/BaseTools/Edk2ToolsBuild.py index 1ea8187..f862468 100644 --- a/BaseTools/Edk2ToolsBuild.py +++ b/BaseTools/Edk2ToolsBuild.py @@ -122,7 +122,7 @@ class Edk2ToolsBuild(BaseAbstractInvocable): for key in vc_vars.keys(): logging.debug(f"Var - {key} = {vc_vars[key]}") if key.lower() == 'path': - shell_env.insert_path(vc_vars[key]) + shell_env.set_path(vc_vars[key]) else: shell_env.set_shell_var(key, vc_vars[key]) -- cgit v1.1