From nobody Tue Apr 30 07:26:51 2024 Delivered-To: importer@patchew.org Received-SPF: none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) client-ip=198.145.21.10; envelope-from=edk2-devel-bounces@lists.01.org; helo=ml01.01.org; Authentication-Results: mx.zohomail.com; spf=none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) smtp.mailfrom=edk2-devel-bounces@lists.01.org Return-Path: Received: from ml01.01.org (ml01.01.org [198.145.21.10]) by mx.zohomail.com with SMTPS id 1503467626637179.7966452348645; Tue, 22 Aug 2017 22:53:46 -0700 (PDT) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id B420A2095DCB7; Tue, 22 Aug 2017 22:51:08 -0700 (PDT) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 3D10D21AEB0A1 for ; Tue, 22 Aug 2017 22:51:07 -0700 (PDT) Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Aug 2017 22:53:40 -0700 Received: from tiano01.ccr.corp.intel.com ([10.239.9.150]) by orsmga005.jf.intel.com with ESMTP; 22 Aug 2017 22:53:39 -0700 X-Original-To: edk2-devel@lists.01.org X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.41,415,1498546800"; d="scan'208";a="141000169" From: hesschen To: edk2-devel@lists.01.org Date: Wed, 23 Aug 2017 13:53:36 +0800 Message-Id: <1503467616-15412-1-git-send-email-hesheng.chen@intel.com> X-Mailer: git-send-email 2.7.2.windows.1 Subject: [edk2] [patch] BaseTools/UPT: Fix UNI file name issue X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" X-ZohoMail: RSF_4 Z_629925259 SPT_0 Content-Type: text/plain; charset="utf-8" Fix the issue of creating duplicate UNI file names Fix the issue of removing packages Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: hesschen Reviewed-by: Yonghong Zhu =20 --- BaseTools/Source/Python/UPT/Core/DependencyRules.py | 2 ++ BaseTools/Source/Python/UPT/GenMetaFile/GenInfFile.py | 6 ++++-- BaseTools/Source/Python/UPT/Library/String.py | 4 +++- BaseTools/Source/Python/UPT/Object/POM/ModuleObject.py | 8 +++++++- 4 files changed, 16 insertions(+), 4 deletions(-) diff --git a/BaseTools/Source/Python/UPT/Core/DependencyRules.py b/BaseTool= s/Source/Python/UPT/Core/DependencyRules.py index 909c584..26c5a97 100644 --- a/BaseTools/Source/Python/UPT/Core/DependencyRules.py +++ b/BaseTools/Source/Python/UPT/Core/DependencyRules.py @@ -55,6 +55,8 @@ class DependencyRules(object): self.PkgsToBeDepend.extend(self.GenToBeInstalledPkgList(ToBeInstal= ledPkgList)) =20 def GenToBeInstalledPkgList(self, ToBeInstalledPkgList): + if not ToBeInstalledPkgList: + return [] RtnList =3D [] for Dist in ToBeInstalledPkgList: for Package in Dist.PackageSurfaceArea: diff --git a/BaseTools/Source/Python/UPT/GenMetaFile/GenInfFile.py b/BaseTo= ols/Source/Python/UPT/GenMetaFile/GenInfFile.py index a376f56..d7eaf3e 100644 --- a/BaseTools/Source/Python/UPT/GenMetaFile/GenInfFile.py +++ b/BaseTools/Source/Python/UPT/GenMetaFile/GenInfFile.py @@ -140,7 +140,9 @@ def ModuleToInf(ModuleObject, PackageObject=3DNone, Dis= tHeader=3DNone): # FileHeader =3D GenHeaderCommentSection(ModuleAbstract, ModuleDescripti= on, ModuleCopyright, ModuleLicense, False, \ DT.TAB_COMMENT_EDK1_SPLIT) - GenModuleUNIEncodeFile(ModuleObject, FileHeader) + ModuleUniFile =3D GenModuleUNIEncodeFile(ModuleObject, FileHeader) + if ModuleUniFile: + ModuleObject.SetModuleUniFile(os.path.basename(ModuleUniFile)) =20 # # Judge whether the INF file is an AsBuild INF. @@ -310,7 +312,7 @@ def GenDefines(ModuleObject): # TAB_INF_DEFINES_VERSION_STRING if ModuleObject.UNIFlag: Statement =3D (u'%s ' % DT.TAB_INF_DEFINES_MODULE_UNI_FILE).ljust(= LeftOffset) + \ - u'=3D %s' % ModuleObject.GetBaseName() + '.uni' + u'=3D %s' % ModuleObject.GetModuleUniFile() SpecialStatementList.append(Statement) =20 # TAB_INF_DEFINES_MODULE_TYPE diff --git a/BaseTools/Source/Python/UPT/Library/String.py b/BaseTools/Sour= ce/Python/UPT/Library/String.py index 89371db..278073e 100644 --- a/BaseTools/Source/Python/UPT/Library/String.py +++ b/BaseTools/Source/Python/UPT/Library/String.py @@ -969,6 +969,7 @@ def GetUniFileName(FilePath, FileName): pass =20 LargestIndex =3D -1 + IndexNotFound =3D True for File in Files: if File.upper().startswith(FileName.upper()) and File.upper().ends= with('.UNI'): Index =3D File.upper().replace(FileName.upper(), '').replace('= .UNI', '') @@ -978,11 +979,12 @@ def GetUniFileName(FilePath, FileName): except Exception: Index =3D -1 else: + IndexNotFound =3D False Index =3D 0 if Index > LargestIndex: LargestIndex =3D Index + 1 =20 - if LargestIndex > -1: + if LargestIndex > -1 and not IndexNotFound: return os.path.normpath(os.path.join(FilePath, FileName + str(Larg= estIndex) + '.uni')) else: return os.path.normpath(os.path.join(FilePath, FileName + '.uni')) diff --git a/BaseTools/Source/Python/UPT/Object/POM/ModuleObject.py b/BaseT= ools/Source/Python/UPT/Object/POM/ModuleObject.py index e85b8fa..4052d28 100644 --- a/BaseTools/Source/Python/UPT/Object/POM/ModuleObject.py +++ b/BaseTools/Source/Python/UPT/Object/POM/ModuleObject.py @@ -1,7 +1,7 @@ ## @file # This file is used to define a class object to describe a module # -# Copyright (c) 2011 - 2014, Intel Corporation. All rights reserved.
+# Copyright (c) 2011 - 2017, Intel Corporation. All rights reserved.
# # This program and the accompanying materials are licensed and made availa= ble=20 # under the terms and conditions of the BSD License which accompanies this=20 @@ -105,6 +105,7 @@ class ModuleHeaderObject(IdentificationObject, CommonHe= aderObject, BinaryHeaderO self.PiSpecificationVersion =3D '' self.UefiSpecificationVersion =3D '' self.UNIFlag =3D False + self.ModuleUniFile =3D '' # # SpecObject # @@ -208,6 +209,11 @@ class ModuleHeaderObject(IdentificationObject, CommonH= eaderObject, BinaryHeaderO def GetSupArchList(self): return self.SupArchList =20 + def SetModuleUniFile(self, ModuleUniFile): + self.ModuleUniFile =3D ModuleUniFile + + def GetModuleUniFile(self): + return self.ModuleUniFile ## # SourceFileObject # --=20 2.7.2.windows.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel