Revision: 15847
          http://sourceforge.net/p/edk2/code/15847
Author:   hchen30
Date:     2014-08-20 07:41:38 +0000 (Wed, 20 Aug 2014)
Log Message:
-----------
This patch is going to:
1. Fix a bug in GenFds for wrongly handling '"' of command line parameter.
2. Provide different input parameter mode for different OS. For Linux, using a 
list. For Windows, using a string.
3. Remove 3 un-existing function calling.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Hess Chen <[email protected]>
Reviewed-by: Yingke Liu <[email protected]>

Modified Paths:
--------------
    trunk/edk2/BaseTools/Source/Python/GenFds/GenFds.py
    trunk/edk2/BaseTools/Source/Python/build/build.py

Modified: trunk/edk2/BaseTools/Source/Python/GenFds/GenFds.py
===================================================================
--- trunk/edk2/BaseTools/Source/Python/GenFds/GenFds.py 2014-08-20 07:05:55 UTC 
(rev 15846)
+++ trunk/edk2/BaseTools/Source/Python/GenFds/GenFds.py 2014-08-20 07:41:38 UTC 
(rev 15847)
@@ -157,7 +157,7 @@
 
         if Options.Macros:
             for Pair in Options.Macros:
-                Pair.strip('"')
+                Pair = Pair.strip('"')
                 List = Pair.split('=')
                 if len(List) == 2:
                     if List[0].strip() == "EFI_SOURCE":

Modified: trunk/edk2/BaseTools/Source/Python/build/build.py
===================================================================
--- trunk/edk2/BaseTools/Source/Python/build/build.py   2014-08-20 07:05:55 UTC 
(rev 15846)
+++ trunk/edk2/BaseTools/Source/Python/build/build.py   2014-08-20 07:41:38 UTC 
(rev 15847)
@@ -251,8 +251,9 @@
     # ubuntu may fail with an error message that the command is not found.
     # So here we may need convert command from string to list instance.
     if not isinstance(Command, list):
-        Command = Command.split()
-    
+        if platform.system() != 'Windows':
+            Command = Command.split()
+
     Proc = None
     EndOfProcedure = None
     try:
@@ -928,7 +929,6 @@
         if BuildModule:
             BuildCommand = BuildCommand + [Target]
             LaunchCommand(BuildCommand, AutoGenObject.MakeFileDir)
-            self.CreateAsBuiltInf()
             return True
 
         # build library
@@ -946,7 +946,6 @@
             for Mod in AutoGenObject.ModuleBuildDirectoryList:
                 NewBuildCommand = BuildCommand + ['-f', 
os.path.normpath(os.path.join(Mod, makefile)), 'pbuild']
                 LaunchCommand(NewBuildCommand, AutoGenObject.MakeFileDir)
-            self.CreateAsBuiltInf()
             return True
 
         # cleanlib
@@ -1056,7 +1055,6 @@
         BuildCommand = BuildCommand + [Target]
         if BuildModule:
             LaunchCommand(BuildCommand, AutoGenObject.MakeFileDir)
-            self.CreateAsBuiltInf()
             return True
 
         # build library

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


------------------------------------------------------------------------------
Slashdot TV.  
Video for Nerds.  Stuff that matters.
http://tv.slashdot.org/
_______________________________________________
edk2-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to