BaseTools: Remove EDK_SOURCE keyword from ECC Tool
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1350 Remove EDK_SOURCE keyword from ECC Tool. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng <bob.c.feng@intel.com> Cc: Liming Gao <liming.gao@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com>
This commit is contained in:
@ -68,9 +68,6 @@ class Ecc(object):
|
|||||||
self.ParseOption()
|
self.ParseOption()
|
||||||
EdkLogger.info(time.strftime("%H:%M:%S, %b.%d %Y ", time.localtime()) + "[00:00]" + "\n")
|
EdkLogger.info(time.strftime("%H:%M:%S, %b.%d %Y ", time.localtime()) + "[00:00]" + "\n")
|
||||||
|
|
||||||
#
|
|
||||||
# Check EFI_SOURCE (Edk build convention). EDK_SOURCE will always point to ECP
|
|
||||||
#
|
|
||||||
WorkspaceDir = os.path.normcase(os.path.normpath(os.environ["WORKSPACE"]))
|
WorkspaceDir = os.path.normcase(os.path.normpath(os.environ["WORKSPACE"]))
|
||||||
os.environ["WORKSPACE"] = WorkspaceDir
|
os.environ["WORKSPACE"] = WorkspaceDir
|
||||||
|
|
||||||
@ -78,33 +75,9 @@ class Ecc(object):
|
|||||||
PackagesPath = os.getenv("PACKAGES_PATH")
|
PackagesPath = os.getenv("PACKAGES_PATH")
|
||||||
mws.setWs(WorkspaceDir, PackagesPath)
|
mws.setWs(WorkspaceDir, PackagesPath)
|
||||||
|
|
||||||
if "ECP_SOURCE" not in os.environ:
|
|
||||||
os.environ["ECP_SOURCE"] = mws.join(WorkspaceDir, GlobalData.gEdkCompatibilityPkg)
|
|
||||||
if "EFI_SOURCE" not in os.environ:
|
|
||||||
os.environ["EFI_SOURCE"] = os.environ["ECP_SOURCE"]
|
|
||||||
if "EDK_SOURCE" not in os.environ:
|
|
||||||
os.environ["EDK_SOURCE"] = os.environ["ECP_SOURCE"]
|
|
||||||
|
|
||||||
#
|
|
||||||
# Unify case of characters on case-insensitive systems
|
|
||||||
#
|
|
||||||
EfiSourceDir = os.path.normcase(os.path.normpath(os.environ["EFI_SOURCE"]))
|
|
||||||
EdkSourceDir = os.path.normcase(os.path.normpath(os.environ["EDK_SOURCE"]))
|
|
||||||
EcpSourceDir = os.path.normcase(os.path.normpath(os.environ["ECP_SOURCE"]))
|
|
||||||
|
|
||||||
os.environ["EFI_SOURCE"] = EfiSourceDir
|
|
||||||
os.environ["EDK_SOURCE"] = EdkSourceDir
|
|
||||||
os.environ["ECP_SOURCE"] = EcpSourceDir
|
|
||||||
|
|
||||||
GlobalData.gWorkspace = WorkspaceDir
|
GlobalData.gWorkspace = WorkspaceDir
|
||||||
GlobalData.gEfiSource = EfiSourceDir
|
|
||||||
GlobalData.gEdkSource = EdkSourceDir
|
|
||||||
GlobalData.gEcpSource = EcpSourceDir
|
|
||||||
|
|
||||||
GlobalData.gGlobalDefines["WORKSPACE"] = WorkspaceDir
|
GlobalData.gGlobalDefines["WORKSPACE"] = WorkspaceDir
|
||||||
GlobalData.gGlobalDefines["EFI_SOURCE"] = EfiSourceDir
|
|
||||||
GlobalData.gGlobalDefines["EDK_SOURCE"] = EdkSourceDir
|
|
||||||
GlobalData.gGlobalDefines["ECP_SOURCE"] = EcpSourceDir
|
|
||||||
|
|
||||||
EdkLogger.info("Loading ECC configuration ... done")
|
EdkLogger.info("Loading ECC configuration ... done")
|
||||||
# Generate checkpoints list
|
# Generate checkpoints list
|
||||||
|
@ -610,17 +610,6 @@ class InfParser(MetaFileParser):
|
|||||||
if not Value:
|
if not Value:
|
||||||
continue
|
continue
|
||||||
|
|
||||||
if Value.upper().find('$(EFI_SOURCE)\Edk'.upper()) > -1 or Value.upper().find('$(EFI_SOURCE)/Edk'.upper()) > -1:
|
|
||||||
Value = '$(EDK_SOURCE)' + Value[17:]
|
|
||||||
if Value.find('$(EFI_SOURCE)') > -1 or Value.find('$(EDK_SOURCE)') > -1:
|
|
||||||
pass
|
|
||||||
elif Value.startswith('.'):
|
|
||||||
pass
|
|
||||||
elif Value.startswith('$('):
|
|
||||||
pass
|
|
||||||
else:
|
|
||||||
Value = '$(EFI_SOURCE)/' + Value
|
|
||||||
|
|
||||||
self._ValueList[Index] = ReplaceMacro(Value, Macros)
|
self._ValueList[Index] = ReplaceMacro(Value, Macros)
|
||||||
|
|
||||||
## Parse [Sources] section
|
## Parse [Sources] section
|
||||||
@ -1359,16 +1348,7 @@ class DscParser(MetaFileParser):
|
|||||||
# Allow using system environment variables in path after !include
|
# Allow using system environment variables in path after !include
|
||||||
#
|
#
|
||||||
__IncludeMacros['WORKSPACE'] = GlobalData.gGlobalDefines['WORKSPACE']
|
__IncludeMacros['WORKSPACE'] = GlobalData.gGlobalDefines['WORKSPACE']
|
||||||
if "ECP_SOURCE" in GlobalData.gGlobalDefines.keys():
|
|
||||||
__IncludeMacros['ECP_SOURCE'] = GlobalData.gGlobalDefines['ECP_SOURCE']
|
|
||||||
#
|
|
||||||
# During GenFds phase call DSC parser, will go into this branch.
|
|
||||||
#
|
|
||||||
elif "ECP_SOURCE" in GlobalData.gCommandLineDefines.keys():
|
|
||||||
__IncludeMacros['ECP_SOURCE'] = GlobalData.gCommandLineDefines['ECP_SOURCE']
|
|
||||||
|
|
||||||
__IncludeMacros['EFI_SOURCE'] = GlobalData.gGlobalDefines['EFI_SOURCE']
|
|
||||||
__IncludeMacros['EDK_SOURCE'] = GlobalData.gGlobalDefines['EDK_SOURCE']
|
|
||||||
#
|
#
|
||||||
# Allow using MACROs comes from [Defines] section to keep compatible.
|
# Allow using MACROs comes from [Defines] section to keep compatible.
|
||||||
#
|
#
|
||||||
|
Reference in New Issue
Block a user