diff options
| author | Sean Brogan <sean.brogan@microsoft.com> | 2022-09-22 04:44:59 +0800 |
|---|---|---|
| committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2022-09-22 12:23:20 +0000 |
| commit | 62f00dad22bc657290780dc65c6f1b8ac6e88f9b (patch) | |
| tree | 472dd85637de3dd062e5a2ce86a6ba69332b10e2 /BaseTools | |
| parent | 2c17d676e402d75a3a674499342f7ddaccf387bd (diff) | |
| download | edk2-62f00dad22bc657290780dc65c6f1b8ac6e88f9b.tar.gz edk2-62f00dad22bc657290780dc65c6f1b8ac6e88f9b.tar.bz2 edk2-62f00dad22bc657290780dc65c6f1b8ac6e88f9b.zip | |
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 <bob.c.feng@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Yuwei Chen <yuwei.chen@intel.com>
Co-authored-by: Sean Brogan <sean.brogan@microsoft.com>
Signed-off-by: Kun Qin <kuqin12@gmail.com>
Reviewed-by: Sean Brogan <sean.brogan@microsoft.com>
Reviewed-by: Bob Feng <bob.c.feng@intel.com>
Diffstat (limited to 'BaseTools')
| -rw-r--r-- | BaseTools/Edk2ToolsBuild.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/BaseTools/Edk2ToolsBuild.py b/BaseTools/Edk2ToolsBuild.py index 1ea8187de6..f862468ce2 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])
|