回复: [PATCH] BaseTools: Fix BaseTools nmake cleanall hang issue

["gaoliming" <gaoliming@byosoft.com.cn>]

Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>

> -----邮件原件-----
> 发件人: Yunhua Feng <fengyunhua@byosoft.com.cn>
> 发送时间: 2020年10月30日 11:11
> 收件人: devel@edk2.groups.io
> 抄送: Bob Feng <bob.c.feng@intel.com>; Liming Gao
> <gaoliming@byosoft.com.cn>; Yuwei Chen <yuwei.chen@intel.com>
> 主题: [PATCH] BaseTools: Fix BaseTools nmake cleanall hang issue
> 
> On windows system, when use command chcp displays the number of the
> active console code page, if the active console code is 936, run
> make cleanall in the BaseTools will hang.
> Issue reproduce step:
> chcp 936
> edksetup.bat VS2015
> cd BaseTools
> nmake cleanall
> 
> Cc: Bob Feng <bob.c.feng@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Yuwei Chen <yuwei.chen@intel.com>
> Signed-off-by: Yunhua Feng <fengyunhua@byosoft.com.cn>
> ---
>  BaseTools/Source/C/Makefiles/NmakeSubdirs.py | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/BaseTools/Source/C/Makefiles/NmakeSubdirs.py
> b/BaseTools/Source/C/Makefiles/NmakeSubdirs.py
> index 9b699ea086..1f4a45004f 100644
> --- a/BaseTools/Source/C/Makefiles/NmakeSubdirs.py
> +++ b/BaseTools/Source/C/Makefiles/NmakeSubdirs.py
> @@ -45,7 +45,10 @@ def RunCommand(WorkDir=None, *Args, **kwargs):
> 
>      output_lock.acquire(True)
>      print("execute command \"{0}\" in directory {1}".format("
".join(Args),
> WorkDir))
> -    print(message)
> +    try:
> +        print(message)
> +    except:
> +        pass
>      output_lock.release()
> 
>      return p.returncode, stdout
> --
> 2.27.0.windows.1