BaseTools: Fix GenFds error doesn't break build.

Fix a bug because of b3497bad1221704a5dbc5da0b10f42625f1ad2ed.
Before the patch, when GenFds fail, the build continue and return success.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Derek Lin <derek.lin2@hpe.com>
Reviewed-by: Bob Feng <bob.c.feng@intel.com>

1 files changed, 5 insertions, 2 deletions

diff --git a/BaseTools/Source/Python/build/build.py b/BaseTools/Source/Python/build/build.py
index cbbb291..97271e6 100644
--- a/BaseTools/Source/Python/build/build.py
+++ b/BaseTools/Source/Python/build/build.py
@@ -3,6 +3,7 @@
 #

 #  Copyright (c) 2014, Hewlett-Packard Development Company, L.P.<BR>

 #  Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.<BR>

+#  Copyright (c) 2018, Hewlett Packard Enterprise Development, L.P.<BR>

 #

 #  This program and the accompanying materials

 #  are licensed and made available under the terms and conditions of the BSD License

@@ -1384,7 +1385,8 @@ class Build():
 

         # genfds

         if Target == 'fds':

-            GenFdsApi(AutoGenObject.GenFdsCommandDict, self.Db)

+            if GenFdsApi(AutoGenObject.GenFdsCommandDict, self.Db):

+                EdkLogger.error("build", COMMAND_FAILURE)

             return True

 

         # run

@@ -2122,7 +2124,8 @@ class Build():
                         # Generate FD image if there's a FDF file found

                         #

                         GenFdsStart = time.time()

-                        GenFdsApi(Wa.GenFdsCommandDict, self.Db)

+                        if GenFdsApi(Wa.GenFdsCommandDict, self.Db):

+                            EdkLogger.error("build", COMMAND_FAILURE)

 

                         #

                         # Create MAP file for all platform FVs after GenFds.