Re: [Patch] BaseTools: Fix the RaiseError variable issue caused by 855698fb69f

["Chen, Hesheng" <hesheng.chen@intel.com>]

Reviewed-by: Hess Chen <hesheng.chen@intel.com>

Chen, Hess
Intel China Software Center
Tel: +86-21-6116-6740
Email: hesheng.chen@intel.com

-----Original Message-----
From: Zhu, Yonghong 
Sent: Wednesday, September 5, 2018 11:27 AM
To: edk2-devel@lists.01.org
Cc: Chen, Hesheng <hesheng.chen@intel.com>; Gao, Liming <liming.gao@intel.com>
Subject: [Patch] BaseTools: Fix the RaiseError variable issue caused by 855698fb69f

The bug is that it cause the RaiseError always be set to TRUE even we call the function with FALSE parameter.

Cc: Hess Chen <hesheng.chen@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com>
---
 BaseTools/Source/Python/Common/EdkLogger.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/BaseTools/Source/Python/Common/EdkLogger.py b/BaseTools/Source/Python/Common/EdkLogger.py
index 80697bf..af77074 100644
--- a/BaseTools/Source/Python/Common/EdkLogger.py
+++ b/BaseTools/Source/Python/Common/EdkLogger.py
@@ -196,12 +196,12 @@ def error(ToolName, ErrorCode, Message=None, File=None, Line=None, ExtraData=Non
         LogText =  _ErrorMessageTemplate % TemplateDict
     else:
         LogText = _ErrorMessageTemplateWithoutFile % TemplateDict
 
     _ErrorLogger.log(ERROR, LogText)
-    RaiseError = IsRaiseError
-    if RaiseError:
+
+    if RaiseError and IsRaiseError:
         raise FatalError(ErrorCode)
 
 # Log information which should be always put out
 quiet   = _ErrorLogger.error
 
--
2.6.1.windows.1