diff options
| author | Hao Wu <hao.a.wu@intel.com> | 2016-11-16 16:35:56 +0800 |
|---|---|---|
| committer | Hao Wu <hao.a.wu@intel.com> | 2016-11-18 08:32:28 +0800 |
| commit | b4dc05e854736c8f353a0f6e18ca85faa4d3785e (patch) | |
| tree | 063e5f85e9c9c210b27571b3eefcd052a6e18395 /BaseTools | |
| parent | 4e7872d2f71f511df155a9048f06761afce751da (diff) | |
| download | edk2-b4dc05e854736c8f353a0f6e18ca85faa4d3785e.zip edk2-b4dc05e854736c8f353a0f6e18ca85faa4d3785e.tar.gz edk2-b4dc05e854736c8f353a0f6e18ca85faa4d3785e.tar.bz2 | |
BaseTools/BuildEnv: Do not modify the env 'PACKAGES_PATH' in BuildEnv
https://bugzilla.tianocore.org/show_bug.cgi?id=236
The script 'BuildEnv' modifies the value of the environment variable
'PACKAGES_PATH' (line 44). The script will substitute the ':' symbol
(separating multiple paths) with a space.
This is not supposed to happen since users might later use 'PACKAGES_PATH'
during the code-building process under a multiple-workspace scenario.
Cc: Liming Gao <liming.gao@intel.com>
Cc: Yonghong Zhu <yonghong.zhu@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Hao Wu <hao.a.wu@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
Diffstat (limited to 'BaseTools')
| -rwxr-xr-x | BaseTools/BuildEnv | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/BaseTools/BuildEnv b/BaseTools/BuildEnv index cb6403a..f748811 100755 --- a/BaseTools/BuildEnv +++ b/BaseTools/BuildEnv @@ -2,7 +2,7 @@ # Setup the environment for unix-like systems running a bash-like shell. # This file must be "sourced" not merely executed. For example: ". edksetup.sh" # -# Copyright (c) 2006 - 2015, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved.<BR> # Copyright (c) 2016, Linaro Ltd. All rights reserved.<BR> # This program and the accompanying materials # are licensed and made available under the terms and conditions of the BSD License @@ -41,8 +41,8 @@ RestorePreviousConfiguration() { export CONF_PATH=$WORKSPACE/Conf if [ ! -d $WORKSPACE/Conf ] && [ -n "$PACKAGES_PATH" ] then - PACKAGES_PATH=${PACKAGES_PATH//:/ } - for DIR in $PACKAGES_PATH + PATH_LIST=${PACKAGES_PATH//:/ } + for DIR in $PATH_LIST do if [ -d $DIR/Conf ] then @@ -138,8 +138,8 @@ SetEdkToolsPath() { # if [ -n "$PACKAGES_PATH"] then - PACKAGES_PATH=${PACKAGES_PATH//:/ } - for DIR in $PACKAGES_PATH + PATH_LIST=${PACKAGES_PATH//:/ } + for DIR in $PATH_LIST do if [ -d $DIR/BaseTools ] then |