* [PATCH] BaseTools: Rename String to StringUtils.
@ 2018-05-19 10:50 Marvin Häuser
2018-05-21 0:59 ` Gao, Liming
0 siblings, 1 reply; 3+ messages in thread
From: Marvin Häuser @ 2018-05-19 10:50 UTC (permalink / raw)
To: edk2-devel@lists.01.org; +Cc: yonghong.zhu@intel.com, liming.gao@intel.com
For case-insensitive file systems, edk2 String.py collides with the
Python string.py, which results in build errors. This,for example,
applies to building via the Windows Subsystem for Linux from a
DriveFS file system. This patch renames String to StringUtils to
prevent conflicts for case-insensitive file systems.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Marvin Haeuser <Marvin.Haeuser@outlook.com>
---
BaseTools/Source/Python/AutoGen/AutoGen.py | 2 +-
BaseTools/Source/Python/AutoGen/BuildEngine.py | 2 +-
BaseTools/Source/Python/AutoGen/GenC.py | 2 +-
BaseTools/Source/Python/AutoGen/GenMake.py | 2 +-
BaseTools/Source/Python/AutoGen/GenPcdDb.py | 2 +-
BaseTools/Source/Python/AutoGen/IdfClassObject.py | 2 +-
BaseTools/Source/Python/AutoGen/UniClassObject.py | 2 +-
BaseTools/Source/Python/Common/Database.py | 2 +-
BaseTools/Source/Python/Common/Parsing.py | 2 +-
BaseTools/Source/Python/Common/{String.py => StringUtils.py} | 0
BaseTools/Source/Python/Common/ToolDefClassObject.py | 2 +-
BaseTools/Source/Python/Ecc/Configuration.py | 2 +-
BaseTools/Source/Python/Ecc/Ecc.py | 2 +-
BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py | 2 +-
BaseTools/Source/Python/Eot/Eot.py | 2 +-
BaseTools/Source/Python/Eot/InfParserLite.py | 2 +-
BaseTools/Source/Python/Eot/Parser.py | 4 ++--
BaseTools/Source/Python/GenFds/AprioriSection.py | 2 +-
BaseTools/Source/Python/GenFds/FdfParser.py | 4 ++--
BaseTools/Source/Python/GenFds/FfsInfStatement.py | 2 +-
BaseTools/Source/Python/GenFds/GenFds.py | 2 +-
BaseTools/Source/Python/GenFds/OptRomInfStatement.py | 2 +-
BaseTools/Source/Python/Makefile | 4 ++--
BaseTools/Source/Python/Table/TableDataModel.py | 2 +-
BaseTools/Source/Python/Table/TableDec.py | 2 +-
BaseTools/Source/Python/Table/TableDsc.py | 2 +-
BaseTools/Source/Python/Table/TableEotReport.py | 2 +-
BaseTools/Source/Python/Table/TableFdf.py | 2 +-
BaseTools/Source/Python/Table/TableFile.py | 2 +-
BaseTools/Source/Python/Table/TableFunction.py | 2 +-
BaseTools/Source/Python/Table/TableIdentifier.py | 2 +-
BaseTools/Source/Python/Table/TableInf.py | 2 +-
BaseTools/Source/Python/Table/TablePcd.py | 2 +-
BaseTools/Source/Python/Table/TableQuery.py | 2 +-
BaseTools/Source/Python/Table/TableReport.py | 2 +-
BaseTools/Source/Python/UPT/Library/{String.py => StringUtils.py} | 0
BaseTools/Source/Python/Workspace/DecBuildData.py | 2 +-
BaseTools/Source/Python/Workspace/DscBuildData.py | 2 +-
BaseTools/Source/Python/Workspace/InfBuildData.py | 2 +-
BaseTools/Source/Python/Workspace/MetaFileParser.py | 2 +-
BaseTools/Source/Python/Workspace/WorkspaceDatabase.py | 2 +-
BaseTools/Source/Python/build/BuildReport.py | 2 +-
42 files changed, 43 insertions(+), 43 deletions(-)
diff --git a/BaseTools/Source/Python/AutoGen/AutoGen.py b/BaseTools/Source/Python/AutoGen/AutoGen.py
index 1e6511cdb5d2..4304967aee42 100644
--- a/BaseTools/Source/Python/AutoGen/AutoGen.py
+++ b/BaseTools/Source/Python/AutoGen/AutoGen.py
@@ -33,7 +33,7 @@ from Common.LongFilePathSupport import CopyLongFilePath
from Common.BuildToolError import *
from Common.DataType import *
from Common.Misc import *
-from Common.String import *
+from Common.StringUtils import *
import Common.GlobalData as GlobalData
from GenFds.FdfParser import *
from CommonDataClass.CommonClass import SkuInfoClass
diff --git a/BaseTools/Source/Python/AutoGen/BuildEngine.py b/BaseTools/Source/Python/AutoGen/BuildEngine.py
index dd6301b01fce..ad1919442e6e 100644
--- a/BaseTools/Source/Python/AutoGen/BuildEngine.py
+++ b/BaseTools/Source/Python/AutoGen/BuildEngine.py
@@ -23,7 +23,7 @@ from Common.LongFilePathSupport import OpenLongFilePath as open
from Common.GlobalData import *
from Common.BuildToolError import *
from Common.Misc import tdict, PathClass
-from Common.String import NormPath
+from Common.StringUtils import NormPath
from Common.DataType import *
import Common.EdkLogger as EdkLogger
diff --git a/BaseTools/Source/Python/AutoGen/GenC.py b/BaseTools/Source/Python/AutoGen/GenC.py
index 40a343ca1057..1be27d2b89e0 100644
--- a/BaseTools/Source/Python/AutoGen/GenC.py
+++ b/BaseTools/Source/Python/AutoGen/GenC.py
@@ -21,7 +21,7 @@ from Common import EdkLogger
from Common.BuildToolError import *
from Common.DataType import *
from Common.Misc import *
-from Common.String import StringToArray
+from Common.StringUtils import StringToArray
from StrGather import *
from GenPcdDb import CreatePcdDatabaseCode
from IdfClassObject import *
diff --git a/BaseTools/Source/Python/AutoGen/GenMake.py b/BaseTools/Source/Python/AutoGen/GenMake.py
index a37350742240..6e83b3d73e61 100644
--- a/BaseTools/Source/Python/AutoGen/GenMake.py
+++ b/BaseTools/Source/Python/AutoGen/GenMake.py
@@ -22,7 +22,7 @@ from Common.LongFilePathSupport import OpenLongFilePath as open
from Common.MultipleWorkspace import MultipleWorkspace as mws
from Common.BuildToolError import *
from Common.Misc import *
-from Common.String import *
+from Common.StringUtils import *
from BuildEngine import *
import Common.GlobalData as GlobalData
from collections import OrderedDict
diff --git a/BaseTools/Source/Python/AutoGen/GenPcdDb.py b/BaseTools/Source/Python/AutoGen/GenPcdDb.py
index d2d42fe9d08e..2e05b77e14c2 100644
--- a/BaseTools/Source/Python/AutoGen/GenPcdDb.py
+++ b/BaseTools/Source/Python/AutoGen/GenPcdDb.py
@@ -12,7 +12,7 @@
#
from StringIO import StringIO
from Common.Misc import *
-from Common.String import StringToArray
+from Common.StringUtils import StringToArray
from struct import pack
from ValidCheckingInfoObject import VAR_CHECK_PCD_VARIABLE_TAB_CONTAINER
from ValidCheckingInfoObject import VAR_CHECK_PCD_VARIABLE_TAB
diff --git a/BaseTools/Source/Python/AutoGen/IdfClassObject.py b/BaseTools/Source/Python/AutoGen/IdfClassObject.py
index 769790d965b5..e5b933c2036f 100644
--- a/BaseTools/Source/Python/AutoGen/IdfClassObject.py
+++ b/BaseTools/Source/Python/AutoGen/IdfClassObject.py
@@ -16,7 +16,7 @@
import Common.EdkLogger as EdkLogger
import StringIO
from Common.BuildToolError import *
-from Common.String import GetLineNo
+from Common.StringUtils import GetLineNo
from Common.Misc import PathClass
from Common.LongFilePathSupport import LongFilePath
import re
diff --git a/BaseTools/Source/Python/AutoGen/UniClassObject.py b/BaseTools/Source/Python/AutoGen/UniClassObject.py
index 54b6fb22a08a..5a3c2547783b 100644
--- a/BaseTools/Source/Python/AutoGen/UniClassObject.py
+++ b/BaseTools/Source/Python/AutoGen/UniClassObject.py
@@ -21,7 +21,7 @@ import distutils.util
import Common.EdkLogger as EdkLogger
import StringIO
from Common.BuildToolError import *
-from Common.String import GetLineNo
+from Common.StringUtils import GetLineNo
from Common.Misc import PathClass
from Common.LongFilePathSupport import LongFilePath
from Common.GlobalData import *
diff --git a/BaseTools/Source/Python/Common/Database.py b/BaseTools/Source/Python/Common/Database.py
index a81a44731f03..d3340f5a9036 100644
--- a/BaseTools/Source/Python/Common/Database.py
+++ b/BaseTools/Source/Python/Common/Database.py
@@ -19,7 +19,7 @@ import Common.LongFilePathOs as os
import EdkLogger as EdkLogger
from CommonDataClass.DataClass import *
-from String import *
+from StringUtils import *
from DataType import *
from Table.TableDataModel import TableDataModel
diff --git a/BaseTools/Source/Python/Common/Parsing.py b/BaseTools/Source/Python/Common/Parsing.py
index 453c2039e3d9..527852a50c09 100644
--- a/BaseTools/Source/Python/Common/Parsing.py
+++ b/BaseTools/Source/Python/Common/Parsing.py
@@ -14,7 +14,7 @@
##
# Import Modules
#
-from String import *
+from StringUtils import *
from CommonDataClass.DataClass import *
from DataType import *
diff --git a/BaseTools/Source/Python/Common/String.py b/BaseTools/Source/Python/Common/StringUtils.py
similarity index 100%
rename from BaseTools/Source/Python/Common/String.py
rename to BaseTools/Source/Python/Common/StringUtils.py
diff --git a/BaseTools/Source/Python/Common/ToolDefClassObject.py b/BaseTools/Source/Python/Common/ToolDefClassObject.py
index 83359586b994..dd985ab30359 100644
--- a/BaseTools/Source/Python/Common/ToolDefClassObject.py
+++ b/BaseTools/Source/Python/Common/ToolDefClassObject.py
@@ -22,7 +22,7 @@ from BuildToolError import *
from TargetTxtClassObject import *
from Common.LongFilePathSupport import OpenLongFilePath as open
from Common.Misc import PathClass
-from Common.String import NormPath
+from Common.StringUtils import NormPath
import Common.GlobalData as GlobalData
from Common import GlobalData
from Common.MultipleWorkspace import MultipleWorkspace as mws
diff --git a/BaseTools/Source/Python/Ecc/Configuration.py b/BaseTools/Source/Python/Ecc/Configuration.py
index fee7ecb9703d..217b60f4f319 100644
--- a/BaseTools/Source/Python/Ecc/Configuration.py
+++ b/BaseTools/Source/Python/Ecc/Configuration.py
@@ -17,7 +17,7 @@
import Common.LongFilePathOs as os
import Common.EdkLogger as EdkLogger
from Common.DataType import *
-from Common.String import *
+from Common.StringUtils import *
from Common.LongFilePathSupport import OpenLongFilePath as open
_ConfigFileToInternalTranslation = {
diff --git a/BaseTools/Source/Python/Ecc/Ecc.py b/BaseTools/Source/Python/Ecc/Ecc.py
index 60dfc00260f1..e78d70372e36 100644
--- a/BaseTools/Source/Python/Ecc/Ecc.py
+++ b/BaseTools/Source/Python/Ecc/Ecc.py
@@ -24,7 +24,7 @@ from Configuration import Configuration
from Check import Check
import Common.GlobalData as GlobalData
-from Common.String import NormPath
+from Common.StringUtils import NormPath
from Common.BuildVersion import gBUILD_VERSION
from Common import BuildToolError
from Common.Misc import PathClass
diff --git a/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py b/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py
index 4d61cd1cea91..3749f6a2699e 100644
--- a/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py
+++ b/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py
@@ -26,7 +26,7 @@ import EccToolError
from CommonDataClass.DataClass import *
from Common.DataType import *
-from Common.String import *
+from Common.StringUtils import *
from Common.Misc import GuidStructureStringToGuidString, CheckPcdDatum, PathClass, AnalyzePcdData
from Common.Expression import *
from CommonDataClass.Exceptions import *
diff --git a/BaseTools/Source/Python/Eot/Eot.py b/BaseTools/Source/Python/Eot/Eot.py
index fcde8fd3e22f..30a3baa4910a 100644
--- a/BaseTools/Source/Python/Eot/Eot.py
+++ b/BaseTools/Source/Python/Eot/Eot.py
@@ -18,7 +18,7 @@ import Common.LongFilePathOs as os, time, glob
import Common.EdkLogger as EdkLogger
import EotGlobalData
from optparse import OptionParser
-from Common.String import NormPath
+from Common.StringUtils import NormPath
from Common import BuildToolError
from Common.Misc import GuidStructureStringToGuidString
from InfParserLite import *
diff --git a/BaseTools/Source/Python/Eot/InfParserLite.py b/BaseTools/Source/Python/Eot/InfParserLite.py
index 8867bb5dc23a..584a95d6f3e4 100644
--- a/BaseTools/Source/Python/Eot/InfParserLite.py
+++ b/BaseTools/Source/Python/Eot/InfParserLite.py
@@ -19,7 +19,7 @@ import Common.EdkLogger as EdkLogger
from Common.DataType import *
from CommonDataClass.DataClass import *
from Common.Identification import *
-from Common.String import *
+from Common.StringUtils import *
from Parser import *
import Database
diff --git a/BaseTools/Source/Python/Eot/Parser.py b/BaseTools/Source/Python/Eot/Parser.py
index 14c287588a01..ff88e957ad0d 100644
--- a/BaseTools/Source/Python/Eot/Parser.py
+++ b/BaseTools/Source/Python/Eot/Parser.py
@@ -19,9 +19,9 @@ import Common.LongFilePathOs as os, re
import Common.EdkLogger as EdkLogger
from Common.DataType import *
from CommonDataClass.DataClass import *
-from Common.String import CleanString, GetSplitValueList, ReplaceMacro
+from Common.StringUtils import CleanString, GetSplitValueList, ReplaceMacro
import EotGlobalData
-from Common.String import GetSplitList
+from Common.StringUtils import GetSplitList
from Common.LongFilePathSupport import OpenLongFilePath as open
## PreProcess() method
diff --git a/BaseTools/Source/Python/GenFds/AprioriSection.py b/BaseTools/Source/Python/GenFds/AprioriSection.py
index 7cf792b5b251..6b81b42620d7 100644
--- a/BaseTools/Source/Python/GenFds/AprioriSection.py
+++ b/BaseTools/Source/Python/GenFds/AprioriSection.py
@@ -21,7 +21,7 @@ import StringIO
import FfsFileStatement
from GenFdsGlobalVariable import GenFdsGlobalVariable
from CommonDataClass.FdfClass import AprioriSectionClassObject
-from Common.String import *
+from Common.StringUtils import *
from Common.Misc import SaveFileOnChange,PathClass
from Common import EdkLogger
from Common.BuildToolError import *
diff --git a/BaseTools/Source/Python/GenFds/FdfParser.py b/BaseTools/Source/Python/GenFds/FdfParser.py
index 8a9296c49d1d..bb801c02e1cb 100644
--- a/BaseTools/Source/Python/GenFds/FdfParser.py
+++ b/BaseTools/Source/Python/GenFds/FdfParser.py
@@ -48,12 +48,12 @@ from GenFdsGlobalVariable import GenFdsGlobalVariable
from Common.BuildToolError import *
from Common import EdkLogger
from Common.Misc import PathClass
-from Common.String import NormPath
+from Common.StringUtils import NormPath
import Common.GlobalData as GlobalData
from Common.Expression import *
from Common import GlobalData
from Common.DataType import *
-from Common.String import ReplaceMacro
+from Common.StringUtils import ReplaceMacro
import uuid
from Common.Misc import tdict
from Common.MultipleWorkspace import MultipleWorkspace as mws
diff --git a/BaseTools/Source/Python/GenFds/FfsInfStatement.py b/BaseTools/Source/Python/GenFds/FfsInfStatement.py
index c332eee6079d..b26821b29052 100644
--- a/BaseTools/Source/Python/GenFds/FfsInfStatement.py
+++ b/BaseTools/Source/Python/GenFds/FfsInfStatement.py
@@ -29,7 +29,7 @@ import RuleSimpleFile
import RuleComplexFile
from CommonDataClass.FdfClass import FfsInfStatementClassObject
from Common.MultipleWorkspace import MultipleWorkspace as mws
-from Common.String import *
+from Common.StringUtils import *
from Common.Misc import PathClass
from Common.Misc import GuidStructureByteArrayToGuidString
from Common.Misc import ProcessDuplicatedInf
diff --git a/BaseTools/Source/Python/GenFds/GenFds.py b/BaseTools/Source/Python/GenFds/GenFds.py
index f0b51e25dfa2..339b99867369 100644
--- a/BaseTools/Source/Python/GenFds/GenFds.py
+++ b/BaseTools/Source/Python/GenFds/GenFds.py
@@ -32,7 +32,7 @@ import Common.ToolDefClassObject as ToolDefClassObject
from Common.DataType import *
import Common.GlobalData as GlobalData
from Common import EdkLogger
-from Common.String import *
+from Common.StringUtils import *
from Common.Misc import DirCache, PathClass
from Common.Misc import SaveFileOnChange
from Common.Misc import ClearDuplicatedInf
diff --git a/BaseTools/Source/Python/GenFds/OptRomInfStatement.py b/BaseTools/Source/Python/GenFds/OptRomInfStatement.py
index a865ac4436d5..79891b3b82a4 100644
--- a/BaseTools/Source/Python/GenFds/OptRomInfStatement.py
+++ b/BaseTools/Source/Python/GenFds/OptRomInfStatement.py
@@ -22,7 +22,7 @@ import OptionRom
import Common.GlobalData as GlobalData
from Common.DataType import *
-from Common.String import *
+from Common.StringUtils import *
from FfsInfStatement import FfsInfStatement
from GenFdsGlobalVariable import GenFdsGlobalVariable
diff --git a/BaseTools/Source/Python/Makefile b/BaseTools/Source/Python/Makefile
index 8466f6ebab33..d78b12d5050d 100644
--- a/BaseTools/Source/Python/Makefile
+++ b/BaseTools/Source/Python/Makefile
@@ -46,7 +46,7 @@ COMMON_PYTHON=$(BASE_TOOLS_PATH)\Source\Python\Common\BuildToolError.py \
$(BASE_TOOLS_PATH)\Source\Python\Common\LongFilePathSupport.py \
$(BASE_TOOLS_PATH)\Source\Python\Common\Misc.py \
$(BASE_TOOLS_PATH)\Source\Python\Common\Parsing.py \
- $(BASE_TOOLS_PATH)\Source\Python\Common\String.py \
+ $(BASE_TOOLS_PATH)\Source\Python\Common\StringUtils.py \
$(BASE_TOOLS_PATH)\Source\Python\Common\TargetTxtClassObject.py \
$(BASE_TOOLS_PATH)\Source\Python\Common\ToolDefClassObject.py \
$(BASE_TOOLS_PATH)\Source\Python\Common\VpdInfoFile.py \
@@ -167,7 +167,7 @@ CMD_UPT=$(BASE_TOOLS_PATH)\Source\Python\UPT\Core\DependencyRules.py \
$(BASE_TOOLS_PATH)\Source\Python\UPT\Library\Misc.py \
$(BASE_TOOLS_PATH)\Source\Python\UPT\Library\ParserValidate.py \
$(BASE_TOOLS_PATH)\Source\Python\UPT\Library\Parsing.py \
- $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\String.py \
+ $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\StringUtils.py \
$(BASE_TOOLS_PATH)\Source\Python\UPT\Library\UniClassObject.py \
$(BASE_TOOLS_PATH)\Source\Python\UPT\Library\Xml\XmlRoutines.py \
$(BASE_TOOLS_PATH)\Source\Python\UPT\Logger\Log.py \
diff --git a/BaseTools/Source/Python/Table/TableDataModel.py b/BaseTools/Source/Python/Table/TableDataModel.py
index 9c3d7bd9345f..2c37592fc67c 100644
--- a/BaseTools/Source/Python/Table/TableDataModel.py
+++ b/BaseTools/Source/Python/Table/TableDataModel.py
@@ -17,7 +17,7 @@
import Common.EdkLogger as EdkLogger
import CommonDataClass.DataClass as DataClass
from Table import Table
-from Common.String import ConvertToSqlString
+from Common.StringUtils import ConvertToSqlString
## TableDataModel
#
diff --git a/BaseTools/Source/Python/Table/TableDec.py b/BaseTools/Source/Python/Table/TableDec.py
index 6b7d22c9384c..97139c58d893 100644
--- a/BaseTools/Source/Python/Table/TableDec.py
+++ b/BaseTools/Source/Python/Table/TableDec.py
@@ -17,7 +17,7 @@
import Common.EdkLogger as EdkLogger
import CommonDataClass.DataClass as DataClass
from Table import Table
-from Common.String import ConvertToSqlString
+from Common.StringUtils import ConvertToSqlString
## TableDec
#
diff --git a/BaseTools/Source/Python/Table/TableDsc.py b/BaseTools/Source/Python/Table/TableDsc.py
index 69477d544d8e..4ac54933aa3b 100644
--- a/BaseTools/Source/Python/Table/TableDsc.py
+++ b/BaseTools/Source/Python/Table/TableDsc.py
@@ -17,7 +17,7 @@
import Common.EdkLogger as EdkLogger
import CommonDataClass.DataClass as DataClass
from Table import Table
-from Common.String import ConvertToSqlString
+from Common.StringUtils import ConvertToSqlString
## TableDsc
#
diff --git a/BaseTools/Source/Python/Table/TableEotReport.py b/BaseTools/Source/Python/Table/TableEotReport.py
index 740105c8f99d..bccf25ca45d5 100644
--- a/BaseTools/Source/Python/Table/TableEotReport.py
+++ b/BaseTools/Source/Python/Table/TableEotReport.py
@@ -17,7 +17,7 @@
import Common.EdkLogger as EdkLogger
import Common.LongFilePathOs as os, time
from Table import Table
-from Common.String import ConvertToSqlString2
+from Common.StringUtils import ConvertToSqlString2
import Eot.EotToolError as EotToolError
import Eot.EotGlobalData as EotGlobalData
diff --git a/BaseTools/Source/Python/Table/TableFdf.py b/BaseTools/Source/Python/Table/TableFdf.py
index 927b5d1a3be6..eea8e9404d55 100644
--- a/BaseTools/Source/Python/Table/TableFdf.py
+++ b/BaseTools/Source/Python/Table/TableFdf.py
@@ -17,7 +17,7 @@
import Common.EdkLogger as EdkLogger
import CommonDataClass.DataClass as DataClass
from Table import Table
-from Common.String import ConvertToSqlString
+from Common.StringUtils import ConvertToSqlString
## TableFdf
#
diff --git a/BaseTools/Source/Python/Table/TableFile.py b/BaseTools/Source/Python/Table/TableFile.py
index caf749e9d3c5..ac762ea7fcc0 100644
--- a/BaseTools/Source/Python/Table/TableFile.py
+++ b/BaseTools/Source/Python/Table/TableFile.py
@@ -16,7 +16,7 @@
#
import Common.EdkLogger as EdkLogger
from Table import Table
-from Common.String import ConvertToSqlString
+from Common.StringUtils import ConvertToSqlString
import Common.LongFilePathOs as os
from CommonDataClass.DataClass import FileClass
diff --git a/BaseTools/Source/Python/Table/TableFunction.py b/BaseTools/Source/Python/Table/TableFunction.py
index 3d7c2d0ea5a0..3d40bd61f695 100644
--- a/BaseTools/Source/Python/Table/TableFunction.py
+++ b/BaseTools/Source/Python/Table/TableFunction.py
@@ -16,7 +16,7 @@
#
import Common.EdkLogger as EdkLogger
from Table import Table
-from Common.String import ConvertToSqlString
+from Common.StringUtils import ConvertToSqlString
## TableFunction
#
diff --git a/BaseTools/Source/Python/Table/TableIdentifier.py b/BaseTools/Source/Python/Table/TableIdentifier.py
index bcd6d6e1c152..0ec8b3c162ce 100644
--- a/BaseTools/Source/Python/Table/TableIdentifier.py
+++ b/BaseTools/Source/Python/Table/TableIdentifier.py
@@ -15,7 +15,7 @@
# Import Modules
#
import Common.EdkLogger as EdkLogger
-from Common.String import ConvertToSqlString
+from Common.StringUtils import ConvertToSqlString
from Table import Table
## TableIdentifier
diff --git a/BaseTools/Source/Python/Table/TableInf.py b/BaseTools/Source/Python/Table/TableInf.py
index b6e300b150c1..478b77776e35 100644
--- a/BaseTools/Source/Python/Table/TableInf.py
+++ b/BaseTools/Source/Python/Table/TableInf.py
@@ -17,7 +17,7 @@
import Common.EdkLogger as EdkLogger
import CommonDataClass.DataClass as DataClass
from Table import Table
-from Common.String import ConvertToSqlString
+from Common.StringUtils import ConvertToSqlString
## TableInf
#
diff --git a/BaseTools/Source/Python/Table/TablePcd.py b/BaseTools/Source/Python/Table/TablePcd.py
index 19623f98f42c..ca1c0f0c8ab2 100644
--- a/BaseTools/Source/Python/Table/TablePcd.py
+++ b/BaseTools/Source/Python/Table/TablePcd.py
@@ -16,7 +16,7 @@
#
import Common.EdkLogger as EdkLogger
from Table import Table
-from Common.String import ConvertToSqlString
+from Common.StringUtils import ConvertToSqlString
## TablePcd
#
diff --git a/BaseTools/Source/Python/Table/TableQuery.py b/BaseTools/Source/Python/Table/TableQuery.py
index e1d2537394b2..f377b10561dc 100644
--- a/BaseTools/Source/Python/Table/TableQuery.py
+++ b/BaseTools/Source/Python/Table/TableQuery.py
@@ -15,7 +15,7 @@
# Import Modules
#
import Common.EdkLogger as EdkLogger
-from Common.String import ConvertToSqlString
+from Common.StringUtils import ConvertToSqlString
from Table import Table
## TableQuery
diff --git a/BaseTools/Source/Python/Table/TableReport.py b/BaseTools/Source/Python/Table/TableReport.py
index 4af0e98d86b4..9ce1d0aa2518 100644
--- a/BaseTools/Source/Python/Table/TableReport.py
+++ b/BaseTools/Source/Python/Table/TableReport.py
@@ -17,7 +17,7 @@
import Common.EdkLogger as EdkLogger
import Common.LongFilePathOs as os, time
from Table import Table
-from Common.String import ConvertToSqlString2
+from Common.StringUtils import ConvertToSqlString2
import EccToolError as EccToolError
import EccGlobalData as EccGlobalData
from Common.LongFilePathSupport import OpenLongFilePath as open
diff --git a/BaseTools/Source/Python/UPT/Library/String.py b/BaseTools/Source/Python/UPT/Library/StringUtils.py
similarity index 100%
rename from BaseTools/Source/Python/UPT/Library/String.py
rename to BaseTools/Source/Python/UPT/Library/StringUtils.py
diff --git a/BaseTools/Source/Python/Workspace/DecBuildData.py b/BaseTools/Source/Python/Workspace/DecBuildData.py
index 1fbd095f743c..99257d08147b 100644
--- a/BaseTools/Source/Python/Workspace/DecBuildData.py
+++ b/BaseTools/Source/Python/Workspace/DecBuildData.py
@@ -11,7 +11,7 @@
# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
#
-from Common.String import *
+from Common.StringUtils import *
from Common.DataType import *
from Common.Misc import *
from types import *
diff --git a/BaseTools/Source/Python/Workspace/DscBuildData.py b/BaseTools/Source/Python/Workspace/DscBuildData.py
index 8476543c5352..a1205e4098f8 100644
--- a/BaseTools/Source/Python/Workspace/DscBuildData.py
+++ b/BaseTools/Source/Python/Workspace/DscBuildData.py
@@ -17,7 +17,7 @@
# This class is used to retrieve information stored in database and convert them
# into PlatformBuildClassObject form for easier use for AutoGen.
#
-from Common.String import *
+from Common.StringUtils import *
from Common.DataType import *
from Common.Misc import *
from types import *
diff --git a/BaseTools/Source/Python/Workspace/InfBuildData.py b/BaseTools/Source/Python/Workspace/InfBuildData.py
index bd1c84154123..836140759f21 100644
--- a/BaseTools/Source/Python/Workspace/InfBuildData.py
+++ b/BaseTools/Source/Python/Workspace/InfBuildData.py
@@ -12,7 +12,7 @@
# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
#
-from Common.String import *
+from Common.StringUtils import *
from Common.DataType import *
from Common.Misc import *
from types import *
diff --git a/BaseTools/Source/Python/Workspace/MetaFileParser.py b/BaseTools/Source/Python/Workspace/MetaFileParser.py
index 36843643ed13..8ab1dd23a096 100644
--- a/BaseTools/Source/Python/Workspace/MetaFileParser.py
+++ b/BaseTools/Source/Python/Workspace/MetaFileParser.py
@@ -26,7 +26,7 @@ import Common.GlobalData as GlobalData
from CommonDataClass.DataClass import *
from Common.DataType import *
-from Common.String import *
+from Common.StringUtils import *
from Common.Misc import GuidStructureStringToGuidString, CheckPcdDatum, PathClass, AnalyzePcdData, AnalyzeDscPcd, AnalyzePcdExpression, ParseFieldValue
from Common.Expression import *
from CommonDataClass.Exceptions import *
diff --git a/BaseTools/Source/Python/Workspace/WorkspaceDatabase.py b/BaseTools/Source/Python/Workspace/WorkspaceDatabase.py
index 14dcb1ae8136..9e055fcfc46b 100644
--- a/BaseTools/Source/Python/Workspace/WorkspaceDatabase.py
+++ b/BaseTools/Source/Python/Workspace/WorkspaceDatabase.py
@@ -16,7 +16,7 @@
# Import Modules
#
import sqlite3
-from Common.String import *
+from Common.StringUtils import *
from Common.DataType import *
from Common.Misc import *
from types import *
diff --git a/BaseTools/Source/Python/build/BuildReport.py b/BaseTools/Source/Python/build/BuildReport.py
index cf45ef173498..3c495a6b1b31 100644
--- a/BaseTools/Source/Python/build/BuildReport.py
+++ b/BaseTools/Source/Python/build/BuildReport.py
@@ -42,7 +42,7 @@ from Common.MultipleWorkspace import MultipleWorkspace as mws
import Common.GlobalData as GlobalData
from AutoGen.AutoGen import ModuleAutoGen
from Common.Misc import PathClass
-from Common.String import NormPath
+from Common.StringUtils import NormPath
from Common.DataType import *
import collections
from Common.Expression import *
--
2.17.0.windows.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] BaseTools: Rename String to StringUtils.
2018-05-19 10:50 [PATCH] BaseTools: Rename String to StringUtils Marvin Häuser
@ 2018-05-21 0:59 ` Gao, Liming
2018-05-23 2:26 ` Gao, Liming
0 siblings, 1 reply; 3+ messages in thread
From: Gao, Liming @ 2018-05-21 0:59 UTC (permalink / raw)
To: Marvin.Haeuser@outlook.com, edk2-devel@lists.01.org
Cc: Zhu, Yonghong, Gao, Liming
Marvin:
Thanks. This is a good enhancement. I will verify it in my platform and let you know the result.
Thanks
Liming
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Marvin H?user
> Sent: Saturday, May 19, 2018 6:50 PM
> To: edk2-devel@lists.01.org
> Cc: Gao, Liming <liming.gao@intel.com>
> Subject: [edk2] [PATCH] BaseTools: Rename String to StringUtils.
>
> For case-insensitive file systems, edk2 String.py collides with the
> Python string.py, which results in build errors. This,for example,
> applies to building via the Windows Subsystem for Linux from a
> DriveFS file system. This patch renames String to StringUtils to
> prevent conflicts for case-insensitive file systems.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Marvin Haeuser <Marvin.Haeuser@outlook.com>
> ---
> BaseTools/Source/Python/AutoGen/AutoGen.py | 2 +-
> BaseTools/Source/Python/AutoGen/BuildEngine.py | 2 +-
> BaseTools/Source/Python/AutoGen/GenC.py | 2 +-
> BaseTools/Source/Python/AutoGen/GenMake.py | 2 +-
> BaseTools/Source/Python/AutoGen/GenPcdDb.py | 2 +-
> BaseTools/Source/Python/AutoGen/IdfClassObject.py | 2 +-
> BaseTools/Source/Python/AutoGen/UniClassObject.py | 2 +-
> BaseTools/Source/Python/Common/Database.py | 2 +-
> BaseTools/Source/Python/Common/Parsing.py | 2 +-
> BaseTools/Source/Python/Common/{String.py => StringUtils.py} | 0
> BaseTools/Source/Python/Common/ToolDefClassObject.py | 2 +-
> BaseTools/Source/Python/Ecc/Configuration.py | 2 +-
> BaseTools/Source/Python/Ecc/Ecc.py | 2 +-
> BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py | 2 +-
> BaseTools/Source/Python/Eot/Eot.py | 2 +-
> BaseTools/Source/Python/Eot/InfParserLite.py | 2 +-
> BaseTools/Source/Python/Eot/Parser.py | 4 ++--
> BaseTools/Source/Python/GenFds/AprioriSection.py | 2 +-
> BaseTools/Source/Python/GenFds/FdfParser.py | 4 ++--
> BaseTools/Source/Python/GenFds/FfsInfStatement.py | 2 +-
> BaseTools/Source/Python/GenFds/GenFds.py | 2 +-
> BaseTools/Source/Python/GenFds/OptRomInfStatement.py | 2 +-
> BaseTools/Source/Python/Makefile | 4 ++--
> BaseTools/Source/Python/Table/TableDataModel.py | 2 +-
> BaseTools/Source/Python/Table/TableDec.py | 2 +-
> BaseTools/Source/Python/Table/TableDsc.py | 2 +-
> BaseTools/Source/Python/Table/TableEotReport.py | 2 +-
> BaseTools/Source/Python/Table/TableFdf.py | 2 +-
> BaseTools/Source/Python/Table/TableFile.py | 2 +-
> BaseTools/Source/Python/Table/TableFunction.py | 2 +-
> BaseTools/Source/Python/Table/TableIdentifier.py | 2 +-
> BaseTools/Source/Python/Table/TableInf.py | 2 +-
> BaseTools/Source/Python/Table/TablePcd.py | 2 +-
> BaseTools/Source/Python/Table/TableQuery.py | 2 +-
> BaseTools/Source/Python/Table/TableReport.py | 2 +-
> BaseTools/Source/Python/UPT/Library/{String.py => StringUtils.py} | 0
> BaseTools/Source/Python/Workspace/DecBuildData.py | 2 +-
> BaseTools/Source/Python/Workspace/DscBuildData.py | 2 +-
> BaseTools/Source/Python/Workspace/InfBuildData.py | 2 +-
> BaseTools/Source/Python/Workspace/MetaFileParser.py | 2 +-
> BaseTools/Source/Python/Workspace/WorkspaceDatabase.py | 2 +-
> BaseTools/Source/Python/build/BuildReport.py | 2 +-
> 42 files changed, 43 insertions(+), 43 deletions(-)
>
> diff --git a/BaseTools/Source/Python/AutoGen/AutoGen.py b/BaseTools/Source/Python/AutoGen/AutoGen.py
> index 1e6511cdb5d2..4304967aee42 100644
> --- a/BaseTools/Source/Python/AutoGen/AutoGen.py
> +++ b/BaseTools/Source/Python/AutoGen/AutoGen.py
> @@ -33,7 +33,7 @@ from Common.LongFilePathSupport import CopyLongFilePath
> from Common.BuildToolError import *
> from Common.DataType import *
> from Common.Misc import *
> -from Common.String import *
> +from Common.StringUtils import *
> import Common.GlobalData as GlobalData
> from GenFds.FdfParser import *
> from CommonDataClass.CommonClass import SkuInfoClass
> diff --git a/BaseTools/Source/Python/AutoGen/BuildEngine.py b/BaseTools/Source/Python/AutoGen/BuildEngine.py
> index dd6301b01fce..ad1919442e6e 100644
> --- a/BaseTools/Source/Python/AutoGen/BuildEngine.py
> +++ b/BaseTools/Source/Python/AutoGen/BuildEngine.py
> @@ -23,7 +23,7 @@ from Common.LongFilePathSupport import OpenLongFilePath as open
> from Common.GlobalData import *
> from Common.BuildToolError import *
> from Common.Misc import tdict, PathClass
> -from Common.String import NormPath
> +from Common.StringUtils import NormPath
> from Common.DataType import *
>
> import Common.EdkLogger as EdkLogger
> diff --git a/BaseTools/Source/Python/AutoGen/GenC.py b/BaseTools/Source/Python/AutoGen/GenC.py
> index 40a343ca1057..1be27d2b89e0 100644
> --- a/BaseTools/Source/Python/AutoGen/GenC.py
> +++ b/BaseTools/Source/Python/AutoGen/GenC.py
> @@ -21,7 +21,7 @@ from Common import EdkLogger
> from Common.BuildToolError import *
> from Common.DataType import *
> from Common.Misc import *
> -from Common.String import StringToArray
> +from Common.StringUtils import StringToArray
> from StrGather import *
> from GenPcdDb import CreatePcdDatabaseCode
> from IdfClassObject import *
> diff --git a/BaseTools/Source/Python/AutoGen/GenMake.py b/BaseTools/Source/Python/AutoGen/GenMake.py
> index a37350742240..6e83b3d73e61 100644
> --- a/BaseTools/Source/Python/AutoGen/GenMake.py
> +++ b/BaseTools/Source/Python/AutoGen/GenMake.py
> @@ -22,7 +22,7 @@ from Common.LongFilePathSupport import OpenLongFilePath as open
> from Common.MultipleWorkspace import MultipleWorkspace as mws
> from Common.BuildToolError import *
> from Common.Misc import *
> -from Common.String import *
> +from Common.StringUtils import *
> from BuildEngine import *
> import Common.GlobalData as GlobalData
> from collections import OrderedDict
> diff --git a/BaseTools/Source/Python/AutoGen/GenPcdDb.py b/BaseTools/Source/Python/AutoGen/GenPcdDb.py
> index d2d42fe9d08e..2e05b77e14c2 100644
> --- a/BaseTools/Source/Python/AutoGen/GenPcdDb.py
> +++ b/BaseTools/Source/Python/AutoGen/GenPcdDb.py
> @@ -12,7 +12,7 @@
> #
> from StringIO import StringIO
> from Common.Misc import *
> -from Common.String import StringToArray
> +from Common.StringUtils import StringToArray
> from struct import pack
> from ValidCheckingInfoObject import VAR_CHECK_PCD_VARIABLE_TAB_CONTAINER
> from ValidCheckingInfoObject import VAR_CHECK_PCD_VARIABLE_TAB
> diff --git a/BaseTools/Source/Python/AutoGen/IdfClassObject.py b/BaseTools/Source/Python/AutoGen/IdfClassObject.py
> index 769790d965b5..e5b933c2036f 100644
> --- a/BaseTools/Source/Python/AutoGen/IdfClassObject.py
> +++ b/BaseTools/Source/Python/AutoGen/IdfClassObject.py
> @@ -16,7 +16,7 @@
> import Common.EdkLogger as EdkLogger
> import StringIO
> from Common.BuildToolError import *
> -from Common.String import GetLineNo
> +from Common.StringUtils import GetLineNo
> from Common.Misc import PathClass
> from Common.LongFilePathSupport import LongFilePath
> import re
> diff --git a/BaseTools/Source/Python/AutoGen/UniClassObject.py b/BaseTools/Source/Python/AutoGen/UniClassObject.py
> index 54b6fb22a08a..5a3c2547783b 100644
> --- a/BaseTools/Source/Python/AutoGen/UniClassObject.py
> +++ b/BaseTools/Source/Python/AutoGen/UniClassObject.py
> @@ -21,7 +21,7 @@ import distutils.util
> import Common.EdkLogger as EdkLogger
> import StringIO
> from Common.BuildToolError import *
> -from Common.String import GetLineNo
> +from Common.StringUtils import GetLineNo
> from Common.Misc import PathClass
> from Common.LongFilePathSupport import LongFilePath
> from Common.GlobalData import *
> diff --git a/BaseTools/Source/Python/Common/Database.py b/BaseTools/Source/Python/Common/Database.py
> index a81a44731f03..d3340f5a9036 100644
> --- a/BaseTools/Source/Python/Common/Database.py
> +++ b/BaseTools/Source/Python/Common/Database.py
> @@ -19,7 +19,7 @@ import Common.LongFilePathOs as os
>
> import EdkLogger as EdkLogger
> from CommonDataClass.DataClass import *
> -from String import *
> +from StringUtils import *
> from DataType import *
>
> from Table.TableDataModel import TableDataModel
> diff --git a/BaseTools/Source/Python/Common/Parsing.py b/BaseTools/Source/Python/Common/Parsing.py
> index 453c2039e3d9..527852a50c09 100644
> --- a/BaseTools/Source/Python/Common/Parsing.py
> +++ b/BaseTools/Source/Python/Common/Parsing.py
> @@ -14,7 +14,7 @@
> ##
> # Import Modules
> #
> -from String import *
> +from StringUtils import *
> from CommonDataClass.DataClass import *
> from DataType import *
>
> diff --git a/BaseTools/Source/Python/Common/String.py b/BaseTools/Source/Python/Common/StringUtils.py
> similarity index 100%
> rename from BaseTools/Source/Python/Common/String.py
> rename to BaseTools/Source/Python/Common/StringUtils.py
> diff --git a/BaseTools/Source/Python/Common/ToolDefClassObject.py b/BaseTools/Source/Python/Common/ToolDefClassObject.py
> index 83359586b994..dd985ab30359 100644
> --- a/BaseTools/Source/Python/Common/ToolDefClassObject.py
> +++ b/BaseTools/Source/Python/Common/ToolDefClassObject.py
> @@ -22,7 +22,7 @@ from BuildToolError import *
> from TargetTxtClassObject import *
> from Common.LongFilePathSupport import OpenLongFilePath as open
> from Common.Misc import PathClass
> -from Common.String import NormPath
> +from Common.StringUtils import NormPath
> import Common.GlobalData as GlobalData
> from Common import GlobalData
> from Common.MultipleWorkspace import MultipleWorkspace as mws
> diff --git a/BaseTools/Source/Python/Ecc/Configuration.py b/BaseTools/Source/Python/Ecc/Configuration.py
> index fee7ecb9703d..217b60f4f319 100644
> --- a/BaseTools/Source/Python/Ecc/Configuration.py
> +++ b/BaseTools/Source/Python/Ecc/Configuration.py
> @@ -17,7 +17,7 @@
> import Common.LongFilePathOs as os
> import Common.EdkLogger as EdkLogger
> from Common.DataType import *
> -from Common.String import *
> +from Common.StringUtils import *
> from Common.LongFilePathSupport import OpenLongFilePath as open
>
> _ConfigFileToInternalTranslation = {
> diff --git a/BaseTools/Source/Python/Ecc/Ecc.py b/BaseTools/Source/Python/Ecc/Ecc.py
> index 60dfc00260f1..e78d70372e36 100644
> --- a/BaseTools/Source/Python/Ecc/Ecc.py
> +++ b/BaseTools/Source/Python/Ecc/Ecc.py
> @@ -24,7 +24,7 @@ from Configuration import Configuration
> from Check import Check
> import Common.GlobalData as GlobalData
>
> -from Common.String import NormPath
> +from Common.StringUtils import NormPath
> from Common.BuildVersion import gBUILD_VERSION
> from Common import BuildToolError
> from Common.Misc import PathClass
> diff --git a/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py
> b/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py
> index 4d61cd1cea91..3749f6a2699e 100644
> --- a/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py
> +++ b/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py
> @@ -26,7 +26,7 @@ import EccToolError
>
> from CommonDataClass.DataClass import *
> from Common.DataType import *
> -from Common.String import *
> +from Common.StringUtils import *
> from Common.Misc import GuidStructureStringToGuidString, CheckPcdDatum, PathClass, AnalyzePcdData
> from Common.Expression import *
> from CommonDataClass.Exceptions import *
> diff --git a/BaseTools/Source/Python/Eot/Eot.py b/BaseTools/Source/Python/Eot/Eot.py
> index fcde8fd3e22f..30a3baa4910a 100644
> --- a/BaseTools/Source/Python/Eot/Eot.py
> +++ b/BaseTools/Source/Python/Eot/Eot.py
> @@ -18,7 +18,7 @@ import Common.LongFilePathOs as os, time, glob
> import Common.EdkLogger as EdkLogger
> import EotGlobalData
> from optparse import OptionParser
> -from Common.String import NormPath
> +from Common.StringUtils import NormPath
> from Common import BuildToolError
> from Common.Misc import GuidStructureStringToGuidString
> from InfParserLite import *
> diff --git a/BaseTools/Source/Python/Eot/InfParserLite.py b/BaseTools/Source/Python/Eot/InfParserLite.py
> index 8867bb5dc23a..584a95d6f3e4 100644
> --- a/BaseTools/Source/Python/Eot/InfParserLite.py
> +++ b/BaseTools/Source/Python/Eot/InfParserLite.py
> @@ -19,7 +19,7 @@ import Common.EdkLogger as EdkLogger
> from Common.DataType import *
> from CommonDataClass.DataClass import *
> from Common.Identification import *
> -from Common.String import *
> +from Common.StringUtils import *
> from Parser import *
> import Database
>
> diff --git a/BaseTools/Source/Python/Eot/Parser.py b/BaseTools/Source/Python/Eot/Parser.py
> index 14c287588a01..ff88e957ad0d 100644
> --- a/BaseTools/Source/Python/Eot/Parser.py
> +++ b/BaseTools/Source/Python/Eot/Parser.py
> @@ -19,9 +19,9 @@ import Common.LongFilePathOs as os, re
> import Common.EdkLogger as EdkLogger
> from Common.DataType import *
> from CommonDataClass.DataClass import *
> -from Common.String import CleanString, GetSplitValueList, ReplaceMacro
> +from Common.StringUtils import CleanString, GetSplitValueList, ReplaceMacro
> import EotGlobalData
> -from Common.String import GetSplitList
> +from Common.StringUtils import GetSplitList
> from Common.LongFilePathSupport import OpenLongFilePath as open
>
> ## PreProcess() method
> diff --git a/BaseTools/Source/Python/GenFds/AprioriSection.py b/BaseTools/Source/Python/GenFds/AprioriSection.py
> index 7cf792b5b251..6b81b42620d7 100644
> --- a/BaseTools/Source/Python/GenFds/AprioriSection.py
> +++ b/BaseTools/Source/Python/GenFds/AprioriSection.py
> @@ -21,7 +21,7 @@ import StringIO
> import FfsFileStatement
> from GenFdsGlobalVariable import GenFdsGlobalVariable
> from CommonDataClass.FdfClass import AprioriSectionClassObject
> -from Common.String import *
> +from Common.StringUtils import *
> from Common.Misc import SaveFileOnChange,PathClass
> from Common import EdkLogger
> from Common.BuildToolError import *
> diff --git a/BaseTools/Source/Python/GenFds/FdfParser.py b/BaseTools/Source/Python/GenFds/FdfParser.py
> index 8a9296c49d1d..bb801c02e1cb 100644
> --- a/BaseTools/Source/Python/GenFds/FdfParser.py
> +++ b/BaseTools/Source/Python/GenFds/FdfParser.py
> @@ -48,12 +48,12 @@ from GenFdsGlobalVariable import GenFdsGlobalVariable
> from Common.BuildToolError import *
> from Common import EdkLogger
> from Common.Misc import PathClass
> -from Common.String import NormPath
> +from Common.StringUtils import NormPath
> import Common.GlobalData as GlobalData
> from Common.Expression import *
> from Common import GlobalData
> from Common.DataType import *
> -from Common.String import ReplaceMacro
> +from Common.StringUtils import ReplaceMacro
> import uuid
> from Common.Misc import tdict
> from Common.MultipleWorkspace import MultipleWorkspace as mws
> diff --git a/BaseTools/Source/Python/GenFds/FfsInfStatement.py b/BaseTools/Source/Python/GenFds/FfsInfStatement.py
> index c332eee6079d..b26821b29052 100644
> --- a/BaseTools/Source/Python/GenFds/FfsInfStatement.py
> +++ b/BaseTools/Source/Python/GenFds/FfsInfStatement.py
> @@ -29,7 +29,7 @@ import RuleSimpleFile
> import RuleComplexFile
> from CommonDataClass.FdfClass import FfsInfStatementClassObject
> from Common.MultipleWorkspace import MultipleWorkspace as mws
> -from Common.String import *
> +from Common.StringUtils import *
> from Common.Misc import PathClass
> from Common.Misc import GuidStructureByteArrayToGuidString
> from Common.Misc import ProcessDuplicatedInf
> diff --git a/BaseTools/Source/Python/GenFds/GenFds.py b/BaseTools/Source/Python/GenFds/GenFds.py
> index f0b51e25dfa2..339b99867369 100644
> --- a/BaseTools/Source/Python/GenFds/GenFds.py
> +++ b/BaseTools/Source/Python/GenFds/GenFds.py
> @@ -32,7 +32,7 @@ import Common.ToolDefClassObject as ToolDefClassObject
> from Common.DataType import *
> import Common.GlobalData as GlobalData
> from Common import EdkLogger
> -from Common.String import *
> +from Common.StringUtils import *
> from Common.Misc import DirCache, PathClass
> from Common.Misc import SaveFileOnChange
> from Common.Misc import ClearDuplicatedInf
> diff --git a/BaseTools/Source/Python/GenFds/OptRomInfStatement.py b/BaseTools/Source/Python/GenFds/OptRomInfStatement.py
> index a865ac4436d5..79891b3b82a4 100644
> --- a/BaseTools/Source/Python/GenFds/OptRomInfStatement.py
> +++ b/BaseTools/Source/Python/GenFds/OptRomInfStatement.py
> @@ -22,7 +22,7 @@ import OptionRom
> import Common.GlobalData as GlobalData
>
> from Common.DataType import *
> -from Common.String import *
> +from Common.StringUtils import *
> from FfsInfStatement import FfsInfStatement
> from GenFdsGlobalVariable import GenFdsGlobalVariable
>
> diff --git a/BaseTools/Source/Python/Makefile b/BaseTools/Source/Python/Makefile
> index 8466f6ebab33..d78b12d5050d 100644
> --- a/BaseTools/Source/Python/Makefile
> +++ b/BaseTools/Source/Python/Makefile
> @@ -46,7 +46,7 @@ COMMON_PYTHON=$(BASE_TOOLS_PATH)\Source\Python\Common\BuildToolError.py \
> $(BASE_TOOLS_PATH)\Source\Python\Common\LongFilePathSupport.py \
> $(BASE_TOOLS_PATH)\Source\Python\Common\Misc.py \
> $(BASE_TOOLS_PATH)\Source\Python\Common\Parsing.py \
> - $(BASE_TOOLS_PATH)\Source\Python\Common\String.py \
> + $(BASE_TOOLS_PATH)\Source\Python\Common\StringUtils.py \
> $(BASE_TOOLS_PATH)\Source\Python\Common\TargetTxtClassObject.py \
> $(BASE_TOOLS_PATH)\Source\Python\Common\ToolDefClassObject.py \
> $(BASE_TOOLS_PATH)\Source\Python\Common\VpdInfoFile.py \
> @@ -167,7 +167,7 @@ CMD_UPT=$(BASE_TOOLS_PATH)\Source\Python\UPT\Core\DependencyRules.py \
> $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\Misc.py \
> $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\ParserValidate.py \
> $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\Parsing.py \
> - $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\String.py \
> + $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\StringUtils.py \
> $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\UniClassObject.py \
> $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\Xml\XmlRoutines.py \
> $(BASE_TOOLS_PATH)\Source\Python\UPT\Logger\Log.py \
> diff --git a/BaseTools/Source/Python/Table/TableDataModel.py b/BaseTools/Source/Python/Table/TableDataModel.py
> index 9c3d7bd9345f..2c37592fc67c 100644
> --- a/BaseTools/Source/Python/Table/TableDataModel.py
> +++ b/BaseTools/Source/Python/Table/TableDataModel.py
> @@ -17,7 +17,7 @@
> import Common.EdkLogger as EdkLogger
> import CommonDataClass.DataClass as DataClass
> from Table import Table
> -from Common.String import ConvertToSqlString
> +from Common.StringUtils import ConvertToSqlString
>
> ## TableDataModel
> #
> diff --git a/BaseTools/Source/Python/Table/TableDec.py b/BaseTools/Source/Python/Table/TableDec.py
> index 6b7d22c9384c..97139c58d893 100644
> --- a/BaseTools/Source/Python/Table/TableDec.py
> +++ b/BaseTools/Source/Python/Table/TableDec.py
> @@ -17,7 +17,7 @@
> import Common.EdkLogger as EdkLogger
> import CommonDataClass.DataClass as DataClass
> from Table import Table
> -from Common.String import ConvertToSqlString
> +from Common.StringUtils import ConvertToSqlString
>
> ## TableDec
> #
> diff --git a/BaseTools/Source/Python/Table/TableDsc.py b/BaseTools/Source/Python/Table/TableDsc.py
> index 69477d544d8e..4ac54933aa3b 100644
> --- a/BaseTools/Source/Python/Table/TableDsc.py
> +++ b/BaseTools/Source/Python/Table/TableDsc.py
> @@ -17,7 +17,7 @@
> import Common.EdkLogger as EdkLogger
> import CommonDataClass.DataClass as DataClass
> from Table import Table
> -from Common.String import ConvertToSqlString
> +from Common.StringUtils import ConvertToSqlString
>
> ## TableDsc
> #
> diff --git a/BaseTools/Source/Python/Table/TableEotReport.py b/BaseTools/Source/Python/Table/TableEotReport.py
> index 740105c8f99d..bccf25ca45d5 100644
> --- a/BaseTools/Source/Python/Table/TableEotReport.py
> +++ b/BaseTools/Source/Python/Table/TableEotReport.py
> @@ -17,7 +17,7 @@
> import Common.EdkLogger as EdkLogger
> import Common.LongFilePathOs as os, time
> from Table import Table
> -from Common.String import ConvertToSqlString2
> +from Common.StringUtils import ConvertToSqlString2
> import Eot.EotToolError as EotToolError
> import Eot.EotGlobalData as EotGlobalData
>
> diff --git a/BaseTools/Source/Python/Table/TableFdf.py b/BaseTools/Source/Python/Table/TableFdf.py
> index 927b5d1a3be6..eea8e9404d55 100644
> --- a/BaseTools/Source/Python/Table/TableFdf.py
> +++ b/BaseTools/Source/Python/Table/TableFdf.py
> @@ -17,7 +17,7 @@
> import Common.EdkLogger as EdkLogger
> import CommonDataClass.DataClass as DataClass
> from Table import Table
> -from Common.String import ConvertToSqlString
> +from Common.StringUtils import ConvertToSqlString
>
> ## TableFdf
> #
> diff --git a/BaseTools/Source/Python/Table/TableFile.py b/BaseTools/Source/Python/Table/TableFile.py
> index caf749e9d3c5..ac762ea7fcc0 100644
> --- a/BaseTools/Source/Python/Table/TableFile.py
> +++ b/BaseTools/Source/Python/Table/TableFile.py
> @@ -16,7 +16,7 @@
> #
> import Common.EdkLogger as EdkLogger
> from Table import Table
> -from Common.String import ConvertToSqlString
> +from Common.StringUtils import ConvertToSqlString
> import Common.LongFilePathOs as os
> from CommonDataClass.DataClass import FileClass
>
> diff --git a/BaseTools/Source/Python/Table/TableFunction.py b/BaseTools/Source/Python/Table/TableFunction.py
> index 3d7c2d0ea5a0..3d40bd61f695 100644
> --- a/BaseTools/Source/Python/Table/TableFunction.py
> +++ b/BaseTools/Source/Python/Table/TableFunction.py
> @@ -16,7 +16,7 @@
> #
> import Common.EdkLogger as EdkLogger
> from Table import Table
> -from Common.String import ConvertToSqlString
> +from Common.StringUtils import ConvertToSqlString
>
> ## TableFunction
> #
> diff --git a/BaseTools/Source/Python/Table/TableIdentifier.py b/BaseTools/Source/Python/Table/TableIdentifier.py
> index bcd6d6e1c152..0ec8b3c162ce 100644
> --- a/BaseTools/Source/Python/Table/TableIdentifier.py
> +++ b/BaseTools/Source/Python/Table/TableIdentifier.py
> @@ -15,7 +15,7 @@
> # Import Modules
> #
> import Common.EdkLogger as EdkLogger
> -from Common.String import ConvertToSqlString
> +from Common.StringUtils import ConvertToSqlString
> from Table import Table
>
> ## TableIdentifier
> diff --git a/BaseTools/Source/Python/Table/TableInf.py b/BaseTools/Source/Python/Table/TableInf.py
> index b6e300b150c1..478b77776e35 100644
> --- a/BaseTools/Source/Python/Table/TableInf.py
> +++ b/BaseTools/Source/Python/Table/TableInf.py
> @@ -17,7 +17,7 @@
> import Common.EdkLogger as EdkLogger
> import CommonDataClass.DataClass as DataClass
> from Table import Table
> -from Common.String import ConvertToSqlString
> +from Common.StringUtils import ConvertToSqlString
>
> ## TableInf
> #
> diff --git a/BaseTools/Source/Python/Table/TablePcd.py b/BaseTools/Source/Python/Table/TablePcd.py
> index 19623f98f42c..ca1c0f0c8ab2 100644
> --- a/BaseTools/Source/Python/Table/TablePcd.py
> +++ b/BaseTools/Source/Python/Table/TablePcd.py
> @@ -16,7 +16,7 @@
> #
> import Common.EdkLogger as EdkLogger
> from Table import Table
> -from Common.String import ConvertToSqlString
> +from Common.StringUtils import ConvertToSqlString
>
> ## TablePcd
> #
> diff --git a/BaseTools/Source/Python/Table/TableQuery.py b/BaseTools/Source/Python/Table/TableQuery.py
> index e1d2537394b2..f377b10561dc 100644
> --- a/BaseTools/Source/Python/Table/TableQuery.py
> +++ b/BaseTools/Source/Python/Table/TableQuery.py
> @@ -15,7 +15,7 @@
> # Import Modules
> #
> import Common.EdkLogger as EdkLogger
> -from Common.String import ConvertToSqlString
> +from Common.StringUtils import ConvertToSqlString
> from Table import Table
>
> ## TableQuery
> diff --git a/BaseTools/Source/Python/Table/TableReport.py b/BaseTools/Source/Python/Table/TableReport.py
> index 4af0e98d86b4..9ce1d0aa2518 100644
> --- a/BaseTools/Source/Python/Table/TableReport.py
> +++ b/BaseTools/Source/Python/Table/TableReport.py
> @@ -17,7 +17,7 @@
> import Common.EdkLogger as EdkLogger
> import Common.LongFilePathOs as os, time
> from Table import Table
> -from Common.String import ConvertToSqlString2
> +from Common.StringUtils import ConvertToSqlString2
> import EccToolError as EccToolError
> import EccGlobalData as EccGlobalData
> from Common.LongFilePathSupport import OpenLongFilePath as open
> diff --git a/BaseTools/Source/Python/UPT/Library/String.py b/BaseTools/Source/Python/UPT/Library/StringUtils.py
> similarity index 100%
> rename from BaseTools/Source/Python/UPT/Library/String.py
> rename to BaseTools/Source/Python/UPT/Library/StringUtils.py
> diff --git a/BaseTools/Source/Python/Workspace/DecBuildData.py b/BaseTools/Source/Python/Workspace/DecBuildData.py
> index 1fbd095f743c..99257d08147b 100644
> --- a/BaseTools/Source/Python/Workspace/DecBuildData.py
> +++ b/BaseTools/Source/Python/Workspace/DecBuildData.py
> @@ -11,7 +11,7 @@
> # THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
> # WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> #
> -from Common.String import *
> +from Common.StringUtils import *
> from Common.DataType import *
> from Common.Misc import *
> from types import *
> diff --git a/BaseTools/Source/Python/Workspace/DscBuildData.py b/BaseTools/Source/Python/Workspace/DscBuildData.py
> index 8476543c5352..a1205e4098f8 100644
> --- a/BaseTools/Source/Python/Workspace/DscBuildData.py
> +++ b/BaseTools/Source/Python/Workspace/DscBuildData.py
> @@ -17,7 +17,7 @@
> # This class is used to retrieve information stored in database and convert them
> # into PlatformBuildClassObject form for easier use for AutoGen.
> #
> -from Common.String import *
> +from Common.StringUtils import *
> from Common.DataType import *
> from Common.Misc import *
> from types import *
> diff --git a/BaseTools/Source/Python/Workspace/InfBuildData.py b/BaseTools/Source/Python/Workspace/InfBuildData.py
> index bd1c84154123..836140759f21 100644
> --- a/BaseTools/Source/Python/Workspace/InfBuildData.py
> +++ b/BaseTools/Source/Python/Workspace/InfBuildData.py
> @@ -12,7 +12,7 @@
> # WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> #
>
> -from Common.String import *
> +from Common.StringUtils import *
> from Common.DataType import *
> from Common.Misc import *
> from types import *
> diff --git a/BaseTools/Source/Python/Workspace/MetaFileParser.py b/BaseTools/Source/Python/Workspace/MetaFileParser.py
> index 36843643ed13..8ab1dd23a096 100644
> --- a/BaseTools/Source/Python/Workspace/MetaFileParser.py
> +++ b/BaseTools/Source/Python/Workspace/MetaFileParser.py
> @@ -26,7 +26,7 @@ import Common.GlobalData as GlobalData
>
> from CommonDataClass.DataClass import *
> from Common.DataType import *
> -from Common.String import *
> +from Common.StringUtils import *
> from Common.Misc import GuidStructureStringToGuidString, CheckPcdDatum, PathClass, AnalyzePcdData, AnalyzeDscPcd,
> AnalyzePcdExpression, ParseFieldValue
> from Common.Expression import *
> from CommonDataClass.Exceptions import *
> diff --git a/BaseTools/Source/Python/Workspace/WorkspaceDatabase.py
> b/BaseTools/Source/Python/Workspace/WorkspaceDatabase.py
> index 14dcb1ae8136..9e055fcfc46b 100644
> --- a/BaseTools/Source/Python/Workspace/WorkspaceDatabase.py
> +++ b/BaseTools/Source/Python/Workspace/WorkspaceDatabase.py
> @@ -16,7 +16,7 @@
> # Import Modules
> #
> import sqlite3
> -from Common.String import *
> +from Common.StringUtils import *
> from Common.DataType import *
> from Common.Misc import *
> from types import *
> diff --git a/BaseTools/Source/Python/build/BuildReport.py b/BaseTools/Source/Python/build/BuildReport.py
> index cf45ef173498..3c495a6b1b31 100644
> --- a/BaseTools/Source/Python/build/BuildReport.py
> +++ b/BaseTools/Source/Python/build/BuildReport.py
> @@ -42,7 +42,7 @@ from Common.MultipleWorkspace import MultipleWorkspace as mws
> import Common.GlobalData as GlobalData
> from AutoGen.AutoGen import ModuleAutoGen
> from Common.Misc import PathClass
> -from Common.String import NormPath
> +from Common.StringUtils import NormPath
> from Common.DataType import *
> import collections
> from Common.Expression import *
> --
> 2.17.0.windows.1
>
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] BaseTools: Rename String to StringUtils.
2018-05-21 0:59 ` Gao, Liming
@ 2018-05-23 2:26 ` Gao, Liming
0 siblings, 0 replies; 3+ messages in thread
From: Gao, Liming @ 2018-05-23 2:26 UTC (permalink / raw)
To: Marvin.Haeuser@outlook.com, edk2-devel@lists.01.org
This patch passes my platform.
Reviewed-by: Liming Gao <liming.gao@intel.com>
> -----Original Message-----
> From: Gao, Liming
> Sent: Monday, May 21, 2018 9:00 AM
> To: Marvin.Haeuser@outlook.com; edk2-devel@lists.01.org
> Cc: Zhu, Yonghong <yonghong.zhu@intel.com>; Gao, Liming <liming.gao@intel.com>
> Subject: RE: [PATCH] BaseTools: Rename String to StringUtils.
>
> Marvin:
> Thanks. This is a good enhancement. I will verify it in my platform and let you know the result.
>
> Thanks
> Liming
> > -----Original Message-----
> > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Marvin H?user
> > Sent: Saturday, May 19, 2018 6:50 PM
> > To: edk2-devel@lists.01.org
> > Cc: Gao, Liming <liming.gao@intel.com>
> > Subject: [edk2] [PATCH] BaseTools: Rename String to StringUtils.
> >
> > For case-insensitive file systems, edk2 String.py collides with the
> > Python string.py, which results in build errors. This,for example,
> > applies to building via the Windows Subsystem for Linux from a
> > DriveFS file system. This patch renames String to StringUtils to
> > prevent conflicts for case-insensitive file systems.
> >
> > Contributed-under: TianoCore Contribution Agreement 1.1
> > Signed-off-by: Marvin Haeuser <Marvin.Haeuser@outlook.com>
> > ---
> > BaseTools/Source/Python/AutoGen/AutoGen.py | 2 +-
> > BaseTools/Source/Python/AutoGen/BuildEngine.py | 2 +-
> > BaseTools/Source/Python/AutoGen/GenC.py | 2 +-
> > BaseTools/Source/Python/AutoGen/GenMake.py | 2 +-
> > BaseTools/Source/Python/AutoGen/GenPcdDb.py | 2 +-
> > BaseTools/Source/Python/AutoGen/IdfClassObject.py | 2 +-
> > BaseTools/Source/Python/AutoGen/UniClassObject.py | 2 +-
> > BaseTools/Source/Python/Common/Database.py | 2 +-
> > BaseTools/Source/Python/Common/Parsing.py | 2 +-
> > BaseTools/Source/Python/Common/{String.py => StringUtils.py} | 0
> > BaseTools/Source/Python/Common/ToolDefClassObject.py | 2 +-
> > BaseTools/Source/Python/Ecc/Configuration.py | 2 +-
> > BaseTools/Source/Python/Ecc/Ecc.py | 2 +-
> > BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py | 2 +-
> > BaseTools/Source/Python/Eot/Eot.py | 2 +-
> > BaseTools/Source/Python/Eot/InfParserLite.py | 2 +-
> > BaseTools/Source/Python/Eot/Parser.py | 4 ++--
> > BaseTools/Source/Python/GenFds/AprioriSection.py | 2 +-
> > BaseTools/Source/Python/GenFds/FdfParser.py | 4 ++--
> > BaseTools/Source/Python/GenFds/FfsInfStatement.py | 2 +-
> > BaseTools/Source/Python/GenFds/GenFds.py | 2 +-
> > BaseTools/Source/Python/GenFds/OptRomInfStatement.py | 2 +-
> > BaseTools/Source/Python/Makefile | 4 ++--
> > BaseTools/Source/Python/Table/TableDataModel.py | 2 +-
> > BaseTools/Source/Python/Table/TableDec.py | 2 +-
> > BaseTools/Source/Python/Table/TableDsc.py | 2 +-
> > BaseTools/Source/Python/Table/TableEotReport.py | 2 +-
> > BaseTools/Source/Python/Table/TableFdf.py | 2 +-
> > BaseTools/Source/Python/Table/TableFile.py | 2 +-
> > BaseTools/Source/Python/Table/TableFunction.py | 2 +-
> > BaseTools/Source/Python/Table/TableIdentifier.py | 2 +-
> > BaseTools/Source/Python/Table/TableInf.py | 2 +-
> > BaseTools/Source/Python/Table/TablePcd.py | 2 +-
> > BaseTools/Source/Python/Table/TableQuery.py | 2 +-
> > BaseTools/Source/Python/Table/TableReport.py | 2 +-
> > BaseTools/Source/Python/UPT/Library/{String.py => StringUtils.py} | 0
> > BaseTools/Source/Python/Workspace/DecBuildData.py | 2 +-
> > BaseTools/Source/Python/Workspace/DscBuildData.py | 2 +-
> > BaseTools/Source/Python/Workspace/InfBuildData.py | 2 +-
> > BaseTools/Source/Python/Workspace/MetaFileParser.py | 2 +-
> > BaseTools/Source/Python/Workspace/WorkspaceDatabase.py | 2 +-
> > BaseTools/Source/Python/build/BuildReport.py | 2 +-
> > 42 files changed, 43 insertions(+), 43 deletions(-)
> >
> > diff --git a/BaseTools/Source/Python/AutoGen/AutoGen.py b/BaseTools/Source/Python/AutoGen/AutoGen.py
> > index 1e6511cdb5d2..4304967aee42 100644
> > --- a/BaseTools/Source/Python/AutoGen/AutoGen.py
> > +++ b/BaseTools/Source/Python/AutoGen/AutoGen.py
> > @@ -33,7 +33,7 @@ from Common.LongFilePathSupport import CopyLongFilePath
> > from Common.BuildToolError import *
> > from Common.DataType import *
> > from Common.Misc import *
> > -from Common.String import *
> > +from Common.StringUtils import *
> > import Common.GlobalData as GlobalData
> > from GenFds.FdfParser import *
> > from CommonDataClass.CommonClass import SkuInfoClass
> > diff --git a/BaseTools/Source/Python/AutoGen/BuildEngine.py b/BaseTools/Source/Python/AutoGen/BuildEngine.py
> > index dd6301b01fce..ad1919442e6e 100644
> > --- a/BaseTools/Source/Python/AutoGen/BuildEngine.py
> > +++ b/BaseTools/Source/Python/AutoGen/BuildEngine.py
> > @@ -23,7 +23,7 @@ from Common.LongFilePathSupport import OpenLongFilePath as open
> > from Common.GlobalData import *
> > from Common.BuildToolError import *
> > from Common.Misc import tdict, PathClass
> > -from Common.String import NormPath
> > +from Common.StringUtils import NormPath
> > from Common.DataType import *
> >
> > import Common.EdkLogger as EdkLogger
> > diff --git a/BaseTools/Source/Python/AutoGen/GenC.py b/BaseTools/Source/Python/AutoGen/GenC.py
> > index 40a343ca1057..1be27d2b89e0 100644
> > --- a/BaseTools/Source/Python/AutoGen/GenC.py
> > +++ b/BaseTools/Source/Python/AutoGen/GenC.py
> > @@ -21,7 +21,7 @@ from Common import EdkLogger
> > from Common.BuildToolError import *
> > from Common.DataType import *
> > from Common.Misc import *
> > -from Common.String import StringToArray
> > +from Common.StringUtils import StringToArray
> > from StrGather import *
> > from GenPcdDb import CreatePcdDatabaseCode
> > from IdfClassObject import *
> > diff --git a/BaseTools/Source/Python/AutoGen/GenMake.py b/BaseTools/Source/Python/AutoGen/GenMake.py
> > index a37350742240..6e83b3d73e61 100644
> > --- a/BaseTools/Source/Python/AutoGen/GenMake.py
> > +++ b/BaseTools/Source/Python/AutoGen/GenMake.py
> > @@ -22,7 +22,7 @@ from Common.LongFilePathSupport import OpenLongFilePath as open
> > from Common.MultipleWorkspace import MultipleWorkspace as mws
> > from Common.BuildToolError import *
> > from Common.Misc import *
> > -from Common.String import *
> > +from Common.StringUtils import *
> > from BuildEngine import *
> > import Common.GlobalData as GlobalData
> > from collections import OrderedDict
> > diff --git a/BaseTools/Source/Python/AutoGen/GenPcdDb.py b/BaseTools/Source/Python/AutoGen/GenPcdDb.py
> > index d2d42fe9d08e..2e05b77e14c2 100644
> > --- a/BaseTools/Source/Python/AutoGen/GenPcdDb.py
> > +++ b/BaseTools/Source/Python/AutoGen/GenPcdDb.py
> > @@ -12,7 +12,7 @@
> > #
> > from StringIO import StringIO
> > from Common.Misc import *
> > -from Common.String import StringToArray
> > +from Common.StringUtils import StringToArray
> > from struct import pack
> > from ValidCheckingInfoObject import VAR_CHECK_PCD_VARIABLE_TAB_CONTAINER
> > from ValidCheckingInfoObject import VAR_CHECK_PCD_VARIABLE_TAB
> > diff --git a/BaseTools/Source/Python/AutoGen/IdfClassObject.py b/BaseTools/Source/Python/AutoGen/IdfClassObject.py
> > index 769790d965b5..e5b933c2036f 100644
> > --- a/BaseTools/Source/Python/AutoGen/IdfClassObject.py
> > +++ b/BaseTools/Source/Python/AutoGen/IdfClassObject.py
> > @@ -16,7 +16,7 @@
> > import Common.EdkLogger as EdkLogger
> > import StringIO
> > from Common.BuildToolError import *
> > -from Common.String import GetLineNo
> > +from Common.StringUtils import GetLineNo
> > from Common.Misc import PathClass
> > from Common.LongFilePathSupport import LongFilePath
> > import re
> > diff --git a/BaseTools/Source/Python/AutoGen/UniClassObject.py b/BaseTools/Source/Python/AutoGen/UniClassObject.py
> > index 54b6fb22a08a..5a3c2547783b 100644
> > --- a/BaseTools/Source/Python/AutoGen/UniClassObject.py
> > +++ b/BaseTools/Source/Python/AutoGen/UniClassObject.py
> > @@ -21,7 +21,7 @@ import distutils.util
> > import Common.EdkLogger as EdkLogger
> > import StringIO
> > from Common.BuildToolError import *
> > -from Common.String import GetLineNo
> > +from Common.StringUtils import GetLineNo
> > from Common.Misc import PathClass
> > from Common.LongFilePathSupport import LongFilePath
> > from Common.GlobalData import *
> > diff --git a/BaseTools/Source/Python/Common/Database.py b/BaseTools/Source/Python/Common/Database.py
> > index a81a44731f03..d3340f5a9036 100644
> > --- a/BaseTools/Source/Python/Common/Database.py
> > +++ b/BaseTools/Source/Python/Common/Database.py
> > @@ -19,7 +19,7 @@ import Common.LongFilePathOs as os
> >
> > import EdkLogger as EdkLogger
> > from CommonDataClass.DataClass import *
> > -from String import *
> > +from StringUtils import *
> > from DataType import *
> >
> > from Table.TableDataModel import TableDataModel
> > diff --git a/BaseTools/Source/Python/Common/Parsing.py b/BaseTools/Source/Python/Common/Parsing.py
> > index 453c2039e3d9..527852a50c09 100644
> > --- a/BaseTools/Source/Python/Common/Parsing.py
> > +++ b/BaseTools/Source/Python/Common/Parsing.py
> > @@ -14,7 +14,7 @@
> > ##
> > # Import Modules
> > #
> > -from String import *
> > +from StringUtils import *
> > from CommonDataClass.DataClass import *
> > from DataType import *
> >
> > diff --git a/BaseTools/Source/Python/Common/String.py b/BaseTools/Source/Python/Common/StringUtils.py
> > similarity index 100%
> > rename from BaseTools/Source/Python/Common/String.py
> > rename to BaseTools/Source/Python/Common/StringUtils.py
> > diff --git a/BaseTools/Source/Python/Common/ToolDefClassObject.py b/BaseTools/Source/Python/Common/ToolDefClassObject.py
> > index 83359586b994..dd985ab30359 100644
> > --- a/BaseTools/Source/Python/Common/ToolDefClassObject.py
> > +++ b/BaseTools/Source/Python/Common/ToolDefClassObject.py
> > @@ -22,7 +22,7 @@ from BuildToolError import *
> > from TargetTxtClassObject import *
> > from Common.LongFilePathSupport import OpenLongFilePath as open
> > from Common.Misc import PathClass
> > -from Common.String import NormPath
> > +from Common.StringUtils import NormPath
> > import Common.GlobalData as GlobalData
> > from Common import GlobalData
> > from Common.MultipleWorkspace import MultipleWorkspace as mws
> > diff --git a/BaseTools/Source/Python/Ecc/Configuration.py b/BaseTools/Source/Python/Ecc/Configuration.py
> > index fee7ecb9703d..217b60f4f319 100644
> > --- a/BaseTools/Source/Python/Ecc/Configuration.py
> > +++ b/BaseTools/Source/Python/Ecc/Configuration.py
> > @@ -17,7 +17,7 @@
> > import Common.LongFilePathOs as os
> > import Common.EdkLogger as EdkLogger
> > from Common.DataType import *
> > -from Common.String import *
> > +from Common.StringUtils import *
> > from Common.LongFilePathSupport import OpenLongFilePath as open
> >
> > _ConfigFileToInternalTranslation = {
> > diff --git a/BaseTools/Source/Python/Ecc/Ecc.py b/BaseTools/Source/Python/Ecc/Ecc.py
> > index 60dfc00260f1..e78d70372e36 100644
> > --- a/BaseTools/Source/Python/Ecc/Ecc.py
> > +++ b/BaseTools/Source/Python/Ecc/Ecc.py
> > @@ -24,7 +24,7 @@ from Configuration import Configuration
> > from Check import Check
> > import Common.GlobalData as GlobalData
> >
> > -from Common.String import NormPath
> > +from Common.StringUtils import NormPath
> > from Common.BuildVersion import gBUILD_VERSION
> > from Common import BuildToolError
> > from Common.Misc import PathClass
> > diff --git a/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py
> > b/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py
> > index 4d61cd1cea91..3749f6a2699e 100644
> > --- a/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py
> > +++ b/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py
> > @@ -26,7 +26,7 @@ import EccToolError
> >
> > from CommonDataClass.DataClass import *
> > from Common.DataType import *
> > -from Common.String import *
> > +from Common.StringUtils import *
> > from Common.Misc import GuidStructureStringToGuidString, CheckPcdDatum, PathClass, AnalyzePcdData
> > from Common.Expression import *
> > from CommonDataClass.Exceptions import *
> > diff --git a/BaseTools/Source/Python/Eot/Eot.py b/BaseTools/Source/Python/Eot/Eot.py
> > index fcde8fd3e22f..30a3baa4910a 100644
> > --- a/BaseTools/Source/Python/Eot/Eot.py
> > +++ b/BaseTools/Source/Python/Eot/Eot.py
> > @@ -18,7 +18,7 @@ import Common.LongFilePathOs as os, time, glob
> > import Common.EdkLogger as EdkLogger
> > import EotGlobalData
> > from optparse import OptionParser
> > -from Common.String import NormPath
> > +from Common.StringUtils import NormPath
> > from Common import BuildToolError
> > from Common.Misc import GuidStructureStringToGuidString
> > from InfParserLite import *
> > diff --git a/BaseTools/Source/Python/Eot/InfParserLite.py b/BaseTools/Source/Python/Eot/InfParserLite.py
> > index 8867bb5dc23a..584a95d6f3e4 100644
> > --- a/BaseTools/Source/Python/Eot/InfParserLite.py
> > +++ b/BaseTools/Source/Python/Eot/InfParserLite.py
> > @@ -19,7 +19,7 @@ import Common.EdkLogger as EdkLogger
> > from Common.DataType import *
> > from CommonDataClass.DataClass import *
> > from Common.Identification import *
> > -from Common.String import *
> > +from Common.StringUtils import *
> > from Parser import *
> > import Database
> >
> > diff --git a/BaseTools/Source/Python/Eot/Parser.py b/BaseTools/Source/Python/Eot/Parser.py
> > index 14c287588a01..ff88e957ad0d 100644
> > --- a/BaseTools/Source/Python/Eot/Parser.py
> > +++ b/BaseTools/Source/Python/Eot/Parser.py
> > @@ -19,9 +19,9 @@ import Common.LongFilePathOs as os, re
> > import Common.EdkLogger as EdkLogger
> > from Common.DataType import *
> > from CommonDataClass.DataClass import *
> > -from Common.String import CleanString, GetSplitValueList, ReplaceMacro
> > +from Common.StringUtils import CleanString, GetSplitValueList, ReplaceMacro
> > import EotGlobalData
> > -from Common.String import GetSplitList
> > +from Common.StringUtils import GetSplitList
> > from Common.LongFilePathSupport import OpenLongFilePath as open
> >
> > ## PreProcess() method
> > diff --git a/BaseTools/Source/Python/GenFds/AprioriSection.py b/BaseTools/Source/Python/GenFds/AprioriSection.py
> > index 7cf792b5b251..6b81b42620d7 100644
> > --- a/BaseTools/Source/Python/GenFds/AprioriSection.py
> > +++ b/BaseTools/Source/Python/GenFds/AprioriSection.py
> > @@ -21,7 +21,7 @@ import StringIO
> > import FfsFileStatement
> > from GenFdsGlobalVariable import GenFdsGlobalVariable
> > from CommonDataClass.FdfClass import AprioriSectionClassObject
> > -from Common.String import *
> > +from Common.StringUtils import *
> > from Common.Misc import SaveFileOnChange,PathClass
> > from Common import EdkLogger
> > from Common.BuildToolError import *
> > diff --git a/BaseTools/Source/Python/GenFds/FdfParser.py b/BaseTools/Source/Python/GenFds/FdfParser.py
> > index 8a9296c49d1d..bb801c02e1cb 100644
> > --- a/BaseTools/Source/Python/GenFds/FdfParser.py
> > +++ b/BaseTools/Source/Python/GenFds/FdfParser.py
> > @@ -48,12 +48,12 @@ from GenFdsGlobalVariable import GenFdsGlobalVariable
> > from Common.BuildToolError import *
> > from Common import EdkLogger
> > from Common.Misc import PathClass
> > -from Common.String import NormPath
> > +from Common.StringUtils import NormPath
> > import Common.GlobalData as GlobalData
> > from Common.Expression import *
> > from Common import GlobalData
> > from Common.DataType import *
> > -from Common.String import ReplaceMacro
> > +from Common.StringUtils import ReplaceMacro
> > import uuid
> > from Common.Misc import tdict
> > from Common.MultipleWorkspace import MultipleWorkspace as mws
> > diff --git a/BaseTools/Source/Python/GenFds/FfsInfStatement.py b/BaseTools/Source/Python/GenFds/FfsInfStatement.py
> > index c332eee6079d..b26821b29052 100644
> > --- a/BaseTools/Source/Python/GenFds/FfsInfStatement.py
> > +++ b/BaseTools/Source/Python/GenFds/FfsInfStatement.py
> > @@ -29,7 +29,7 @@ import RuleSimpleFile
> > import RuleComplexFile
> > from CommonDataClass.FdfClass import FfsInfStatementClassObject
> > from Common.MultipleWorkspace import MultipleWorkspace as mws
> > -from Common.String import *
> > +from Common.StringUtils import *
> > from Common.Misc import PathClass
> > from Common.Misc import GuidStructureByteArrayToGuidString
> > from Common.Misc import ProcessDuplicatedInf
> > diff --git a/BaseTools/Source/Python/GenFds/GenFds.py b/BaseTools/Source/Python/GenFds/GenFds.py
> > index f0b51e25dfa2..339b99867369 100644
> > --- a/BaseTools/Source/Python/GenFds/GenFds.py
> > +++ b/BaseTools/Source/Python/GenFds/GenFds.py
> > @@ -32,7 +32,7 @@ import Common.ToolDefClassObject as ToolDefClassObject
> > from Common.DataType import *
> > import Common.GlobalData as GlobalData
> > from Common import EdkLogger
> > -from Common.String import *
> > +from Common.StringUtils import *
> > from Common.Misc import DirCache, PathClass
> > from Common.Misc import SaveFileOnChange
> > from Common.Misc import ClearDuplicatedInf
> > diff --git a/BaseTools/Source/Python/GenFds/OptRomInfStatement.py b/BaseTools/Source/Python/GenFds/OptRomInfStatement.py
> > index a865ac4436d5..79891b3b82a4 100644
> > --- a/BaseTools/Source/Python/GenFds/OptRomInfStatement.py
> > +++ b/BaseTools/Source/Python/GenFds/OptRomInfStatement.py
> > @@ -22,7 +22,7 @@ import OptionRom
> > import Common.GlobalData as GlobalData
> >
> > from Common.DataType import *
> > -from Common.String import *
> > +from Common.StringUtils import *
> > from FfsInfStatement import FfsInfStatement
> > from GenFdsGlobalVariable import GenFdsGlobalVariable
> >
> > diff --git a/BaseTools/Source/Python/Makefile b/BaseTools/Source/Python/Makefile
> > index 8466f6ebab33..d78b12d5050d 100644
> > --- a/BaseTools/Source/Python/Makefile
> > +++ b/BaseTools/Source/Python/Makefile
> > @@ -46,7 +46,7 @@ COMMON_PYTHON=$(BASE_TOOLS_PATH)\Source\Python\Common\BuildToolError.py \
> > $(BASE_TOOLS_PATH)\Source\Python\Common\LongFilePathSupport.py \
> > $(BASE_TOOLS_PATH)\Source\Python\Common\Misc.py \
> > $(BASE_TOOLS_PATH)\Source\Python\Common\Parsing.py \
> > - $(BASE_TOOLS_PATH)\Source\Python\Common\String.py \
> > + $(BASE_TOOLS_PATH)\Source\Python\Common\StringUtils.py \
> > $(BASE_TOOLS_PATH)\Source\Python\Common\TargetTxtClassObject.py \
> > $(BASE_TOOLS_PATH)\Source\Python\Common\ToolDefClassObject.py \
> > $(BASE_TOOLS_PATH)\Source\Python\Common\VpdInfoFile.py \
> > @@ -167,7 +167,7 @@ CMD_UPT=$(BASE_TOOLS_PATH)\Source\Python\UPT\Core\DependencyRules.py \
> > $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\Misc.py \
> > $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\ParserValidate.py \
> > $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\Parsing.py \
> > - $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\String.py \
> > + $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\StringUtils.py \
> > $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\UniClassObject.py \
> > $(BASE_TOOLS_PATH)\Source\Python\UPT\Library\Xml\XmlRoutines.py \
> > $(BASE_TOOLS_PATH)\Source\Python\UPT\Logger\Log.py \
> > diff --git a/BaseTools/Source/Python/Table/TableDataModel.py b/BaseTools/Source/Python/Table/TableDataModel.py
> > index 9c3d7bd9345f..2c37592fc67c 100644
> > --- a/BaseTools/Source/Python/Table/TableDataModel.py
> > +++ b/BaseTools/Source/Python/Table/TableDataModel.py
> > @@ -17,7 +17,7 @@
> > import Common.EdkLogger as EdkLogger
> > import CommonDataClass.DataClass as DataClass
> > from Table import Table
> > -from Common.String import ConvertToSqlString
> > +from Common.StringUtils import ConvertToSqlString
> >
> > ## TableDataModel
> > #
> > diff --git a/BaseTools/Source/Python/Table/TableDec.py b/BaseTools/Source/Python/Table/TableDec.py
> > index 6b7d22c9384c..97139c58d893 100644
> > --- a/BaseTools/Source/Python/Table/TableDec.py
> > +++ b/BaseTools/Source/Python/Table/TableDec.py
> > @@ -17,7 +17,7 @@
> > import Common.EdkLogger as EdkLogger
> > import CommonDataClass.DataClass as DataClass
> > from Table import Table
> > -from Common.String import ConvertToSqlString
> > +from Common.StringUtils import ConvertToSqlString
> >
> > ## TableDec
> > #
> > diff --git a/BaseTools/Source/Python/Table/TableDsc.py b/BaseTools/Source/Python/Table/TableDsc.py
> > index 69477d544d8e..4ac54933aa3b 100644
> > --- a/BaseTools/Source/Python/Table/TableDsc.py
> > +++ b/BaseTools/Source/Python/Table/TableDsc.py
> > @@ -17,7 +17,7 @@
> > import Common.EdkLogger as EdkLogger
> > import CommonDataClass.DataClass as DataClass
> > from Table import Table
> > -from Common.String import ConvertToSqlString
> > +from Common.StringUtils import ConvertToSqlString
> >
> > ## TableDsc
> > #
> > diff --git a/BaseTools/Source/Python/Table/TableEotReport.py b/BaseTools/Source/Python/Table/TableEotReport.py
> > index 740105c8f99d..bccf25ca45d5 100644
> > --- a/BaseTools/Source/Python/Table/TableEotReport.py
> > +++ b/BaseTools/Source/Python/Table/TableEotReport.py
> > @@ -17,7 +17,7 @@
> > import Common.EdkLogger as EdkLogger
> > import Common.LongFilePathOs as os, time
> > from Table import Table
> > -from Common.String import ConvertToSqlString2
> > +from Common.StringUtils import ConvertToSqlString2
> > import Eot.EotToolError as EotToolError
> > import Eot.EotGlobalData as EotGlobalData
> >
> > diff --git a/BaseTools/Source/Python/Table/TableFdf.py b/BaseTools/Source/Python/Table/TableFdf.py
> > index 927b5d1a3be6..eea8e9404d55 100644
> > --- a/BaseTools/Source/Python/Table/TableFdf.py
> > +++ b/BaseTools/Source/Python/Table/TableFdf.py
> > @@ -17,7 +17,7 @@
> > import Common.EdkLogger as EdkLogger
> > import CommonDataClass.DataClass as DataClass
> > from Table import Table
> > -from Common.String import ConvertToSqlString
> > +from Common.StringUtils import ConvertToSqlString
> >
> > ## TableFdf
> > #
> > diff --git a/BaseTools/Source/Python/Table/TableFile.py b/BaseTools/Source/Python/Table/TableFile.py
> > index caf749e9d3c5..ac762ea7fcc0 100644
> > --- a/BaseTools/Source/Python/Table/TableFile.py
> > +++ b/BaseTools/Source/Python/Table/TableFile.py
> > @@ -16,7 +16,7 @@
> > #
> > import Common.EdkLogger as EdkLogger
> > from Table import Table
> > -from Common.String import ConvertToSqlString
> > +from Common.StringUtils import ConvertToSqlString
> > import Common.LongFilePathOs as os
> > from CommonDataClass.DataClass import FileClass
> >
> > diff --git a/BaseTools/Source/Python/Table/TableFunction.py b/BaseTools/Source/Python/Table/TableFunction.py
> > index 3d7c2d0ea5a0..3d40bd61f695 100644
> > --- a/BaseTools/Source/Python/Table/TableFunction.py
> > +++ b/BaseTools/Source/Python/Table/TableFunction.py
> > @@ -16,7 +16,7 @@
> > #
> > import Common.EdkLogger as EdkLogger
> > from Table import Table
> > -from Common.String import ConvertToSqlString
> > +from Common.StringUtils import ConvertToSqlString
> >
> > ## TableFunction
> > #
> > diff --git a/BaseTools/Source/Python/Table/TableIdentifier.py b/BaseTools/Source/Python/Table/TableIdentifier.py
> > index bcd6d6e1c152..0ec8b3c162ce 100644
> > --- a/BaseTools/Source/Python/Table/TableIdentifier.py
> > +++ b/BaseTools/Source/Python/Table/TableIdentifier.py
> > @@ -15,7 +15,7 @@
> > # Import Modules
> > #
> > import Common.EdkLogger as EdkLogger
> > -from Common.String import ConvertToSqlString
> > +from Common.StringUtils import ConvertToSqlString
> > from Table import Table
> >
> > ## TableIdentifier
> > diff --git a/BaseTools/Source/Python/Table/TableInf.py b/BaseTools/Source/Python/Table/TableInf.py
> > index b6e300b150c1..478b77776e35 100644
> > --- a/BaseTools/Source/Python/Table/TableInf.py
> > +++ b/BaseTools/Source/Python/Table/TableInf.py
> > @@ -17,7 +17,7 @@
> > import Common.EdkLogger as EdkLogger
> > import CommonDataClass.DataClass as DataClass
> > from Table import Table
> > -from Common.String import ConvertToSqlString
> > +from Common.StringUtils import ConvertToSqlString
> >
> > ## TableInf
> > #
> > diff --git a/BaseTools/Source/Python/Table/TablePcd.py b/BaseTools/Source/Python/Table/TablePcd.py
> > index 19623f98f42c..ca1c0f0c8ab2 100644
> > --- a/BaseTools/Source/Python/Table/TablePcd.py
> > +++ b/BaseTools/Source/Python/Table/TablePcd.py
> > @@ -16,7 +16,7 @@
> > #
> > import Common.EdkLogger as EdkLogger
> > from Table import Table
> > -from Common.String import ConvertToSqlString
> > +from Common.StringUtils import ConvertToSqlString
> >
> > ## TablePcd
> > #
> > diff --git a/BaseTools/Source/Python/Table/TableQuery.py b/BaseTools/Source/Python/Table/TableQuery.py
> > index e1d2537394b2..f377b10561dc 100644
> > --- a/BaseTools/Source/Python/Table/TableQuery.py
> > +++ b/BaseTools/Source/Python/Table/TableQuery.py
> > @@ -15,7 +15,7 @@
> > # Import Modules
> > #
> > import Common.EdkLogger as EdkLogger
> > -from Common.String import ConvertToSqlString
> > +from Common.StringUtils import ConvertToSqlString
> > from Table import Table
> >
> > ## TableQuery
> > diff --git a/BaseTools/Source/Python/Table/TableReport.py b/BaseTools/Source/Python/Table/TableReport.py
> > index 4af0e98d86b4..9ce1d0aa2518 100644
> > --- a/BaseTools/Source/Python/Table/TableReport.py
> > +++ b/BaseTools/Source/Python/Table/TableReport.py
> > @@ -17,7 +17,7 @@
> > import Common.EdkLogger as EdkLogger
> > import Common.LongFilePathOs as os, time
> > from Table import Table
> > -from Common.String import ConvertToSqlString2
> > +from Common.StringUtils import ConvertToSqlString2
> > import EccToolError as EccToolError
> > import EccGlobalData as EccGlobalData
> > from Common.LongFilePathSupport import OpenLongFilePath as open
> > diff --git a/BaseTools/Source/Python/UPT/Library/String.py b/BaseTools/Source/Python/UPT/Library/StringUtils.py
> > similarity index 100%
> > rename from BaseTools/Source/Python/UPT/Library/String.py
> > rename to BaseTools/Source/Python/UPT/Library/StringUtils.py
> > diff --git a/BaseTools/Source/Python/Workspace/DecBuildData.py b/BaseTools/Source/Python/Workspace/DecBuildData.py
> > index 1fbd095f743c..99257d08147b 100644
> > --- a/BaseTools/Source/Python/Workspace/DecBuildData.py
> > +++ b/BaseTools/Source/Python/Workspace/DecBuildData.py
> > @@ -11,7 +11,7 @@
> > # THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
> > # WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> > #
> > -from Common.String import *
> > +from Common.StringUtils import *
> > from Common.DataType import *
> > from Common.Misc import *
> > from types import *
> > diff --git a/BaseTools/Source/Python/Workspace/DscBuildData.py b/BaseTools/Source/Python/Workspace/DscBuildData.py
> > index 8476543c5352..a1205e4098f8 100644
> > --- a/BaseTools/Source/Python/Workspace/DscBuildData.py
> > +++ b/BaseTools/Source/Python/Workspace/DscBuildData.py
> > @@ -17,7 +17,7 @@
> > # This class is used to retrieve information stored in database and convert them
> > # into PlatformBuildClassObject form for easier use for AutoGen.
> > #
> > -from Common.String import *
> > +from Common.StringUtils import *
> > from Common.DataType import *
> > from Common.Misc import *
> > from types import *
> > diff --git a/BaseTools/Source/Python/Workspace/InfBuildData.py b/BaseTools/Source/Python/Workspace/InfBuildData.py
> > index bd1c84154123..836140759f21 100644
> > --- a/BaseTools/Source/Python/Workspace/InfBuildData.py
> > +++ b/BaseTools/Source/Python/Workspace/InfBuildData.py
> > @@ -12,7 +12,7 @@
> > # WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> > #
> >
> > -from Common.String import *
> > +from Common.StringUtils import *
> > from Common.DataType import *
> > from Common.Misc import *
> > from types import *
> > diff --git a/BaseTools/Source/Python/Workspace/MetaFileParser.py b/BaseTools/Source/Python/Workspace/MetaFileParser.py
> > index 36843643ed13..8ab1dd23a096 100644
> > --- a/BaseTools/Source/Python/Workspace/MetaFileParser.py
> > +++ b/BaseTools/Source/Python/Workspace/MetaFileParser.py
> > @@ -26,7 +26,7 @@ import Common.GlobalData as GlobalData
> >
> > from CommonDataClass.DataClass import *
> > from Common.DataType import *
> > -from Common.String import *
> > +from Common.StringUtils import *
> > from Common.Misc import GuidStructureStringToGuidString, CheckPcdDatum, PathClass, AnalyzePcdData, AnalyzeDscPcd,
> > AnalyzePcdExpression, ParseFieldValue
> > from Common.Expression import *
> > from CommonDataClass.Exceptions import *
> > diff --git a/BaseTools/Source/Python/Workspace/WorkspaceDatabase.py
> > b/BaseTools/Source/Python/Workspace/WorkspaceDatabase.py
> > index 14dcb1ae8136..9e055fcfc46b 100644
> > --- a/BaseTools/Source/Python/Workspace/WorkspaceDatabase.py
> > +++ b/BaseTools/Source/Python/Workspace/WorkspaceDatabase.py
> > @@ -16,7 +16,7 @@
> > # Import Modules
> > #
> > import sqlite3
> > -from Common.String import *
> > +from Common.StringUtils import *
> > from Common.DataType import *
> > from Common.Misc import *
> > from types import *
> > diff --git a/BaseTools/Source/Python/build/BuildReport.py b/BaseTools/Source/Python/build/BuildReport.py
> > index cf45ef173498..3c495a6b1b31 100644
> > --- a/BaseTools/Source/Python/build/BuildReport.py
> > +++ b/BaseTools/Source/Python/build/BuildReport.py
> > @@ -42,7 +42,7 @@ from Common.MultipleWorkspace import MultipleWorkspace as mws
> > import Common.GlobalData as GlobalData
> > from AutoGen.AutoGen import ModuleAutoGen
> > from Common.Misc import PathClass
> > -from Common.String import NormPath
> > +from Common.StringUtils import NormPath
> > from Common.DataType import *
> > import collections
> > from Common.Expression import *
> > --
> > 2.17.0.windows.1
> >
> > _______________________________________________
> > edk2-devel mailing list
> > edk2-devel@lists.01.org
> > https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-05-23 2:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-05-19 10:50 [PATCH] BaseTools: Rename String to StringUtils Marvin Häuser
2018-05-21 0:59 ` Gao, Liming
2018-05-23 2:26 ` Gao, Liming
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox