<p>Hi Bob,</p>
<p>Thanks for the review.<br />Here is the original description of the issue.<br />There has been a build failure from SCT, related to <a href="https://edk2.groups.io/g/devel/message/60407?p=,,,20,0,0,0::relevance,,gEfiFormBrowserExProtocolGuid,20,2,0,74529350">https://edk2.groups.io/g/devel/message/60407?p=,,,20,0,0,0::relevance,,gEfiFormBrowserExProtocolGuid,20,2,0,74529350.</a><br />The build log printed the error code obviously but returned with sys.exit(0).<br />Unfortunately this behavior confused the build system and made me notice the failure so late.<br />If there is a concern of the conflict among multiple autogenworkers, I wonder how you think this way.<br /><br />1. No change in BaseTools/Source/Python/AutoGen/AutoGenWorker.py<br />2. BaseTools/Source/Python/build/build.py</p>
<p>--- a/BaseTools/Source/Python/build/build.py</p>
<p>+++ b/BaseTools/Source/Python/build/build.py</p>
<p>@@ -880,7 +880,10 @@ class Build():</p>
<p> </p>
<p>             self.AutoGenMgr.join()</p>
<p>             rt = self.AutoGenMgr.Status</p>
<p>-            return rt, 0</p>
<p>+            err = 0</p>
<p>+            if not rt:</p>
<p>+                err = UNKNOWN_ERROR << Use a fixed error type of FatalError instead of getting the exact error type from the autogenworkers.</p>
<p>+            return rt, err</p>
<p>         except FatalError as e:</p>
<p>             return False, e.args[0]</p>
<p>         except:</p>
<p><br />In this way, no change in the multi-thread operation and build.py is able to exit with a standard default error. It's not needed to catch the exact error type.<br />Please share your opinion.<br /><br />Thanks,<br />Irene<br /><br /><br /></p>
<p class="MsoPlainText"> </p>

<div width="1" style="color:white;clear:both">_._,_._,_</div>
<hr>
Groups.io Links:<p>


You receive all messages sent to this group.



<p>

<a target="_blank" href="https://edk2.groups.io/g/devel/message/60440">View/Reply Online (#60440)</a> |


  


|


  
    <a target="_blank" href="https://groups.io/mt/74507933/1813853">Mute This Topic</a>
  

| <a href="https://edk2.groups.io/g/devel/post">New Topic</a><br>



<br>

<a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> |
<a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> |

<a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a>

 [edk2-devel-archive@redhat.com]<br>
<div width="1" style="color:white;clear:both">_._,_._,_</div>