From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by mx.groups.io with SMTP id smtpd.web11.11220.1634030852152644871 for ; Tue, 12 Oct 2021 02:27:32 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=LHg32s8B; spf=pass (domain: intel.com, ip: 134.134.136.24, mailfrom: bob.c.feng@intel.com) X-IronPort-AV: E=McAfee;i="6200,9189,10134"; a="226990127" X-IronPort-AV: E=Sophos;i="5.85,367,1624345200"; d="scan'208";a="226990127" Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 Oct 2021 02:27:31 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.85,367,1624345200"; d="scan'208";a="716782430" Received: from orsmsx603.amr.corp.intel.com ([10.22.229.16]) by fmsmga005.fm.intel.com with ESMTP; 12 Oct 2021 02:27:30 -0700 Received: from orsmsx610.amr.corp.intel.com (10.22.229.23) by ORSMSX603.amr.corp.intel.com (10.22.229.16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Tue, 12 Oct 2021 02:27:30 -0700 Received: from orsmsx611.amr.corp.intel.com (10.22.229.24) by ORSMSX610.amr.corp.intel.com (10.22.229.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Tue, 12 Oct 2021 02:27:29 -0700 Received: from ORSEDG601.ED.cps.intel.com (10.7.248.6) by orsmsx611.amr.corp.intel.com (10.22.229.24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12 via Frontend Transport; Tue, 12 Oct 2021 02:27:29 -0700 Received: from NAM10-BN7-obe.outbound.protection.outlook.com (104.47.70.101) by edgegateway.intel.com (134.134.137.102) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2242.12; Tue, 12 Oct 2021 02:26:19 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=JzAzpDrrwwL3WasTRPQmsFeQEggCW/pngwHpqY+s0KwoSlrJv3Ci9atJcQyI2T+/trXEqZTYtTYWJ6hw8m+u7Yguv8R1NpP25A8HDZC/Nfmbpj+SkKBh77QJVdAoQkQ7UxGKjzn6wPmTCaQbg7MmXYg/t7C+yT+4cI9YDCHyoPveYSE1Y3AxYfkZNuRd1Udo+GnJ9k8i/x4/97/jEXXCmLE92GVStGIZPjZUmceGSo8D0RrZzus8k0nS4/zdnjfl21740cu2C08hyN9RWymvk8HYLvr28o5sFyusO1M96NCYHNNsmKkP+DmWHkrfHFIhc4l565AlsRSNc4YyN9WFUg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=jIFZn3+Kli/g0kJ8HfhlrQTVKPrnGkJIoQreFf3IhUY=; b=ZLKlUaY6Uyqe9S1IHyYz7Sb7CrJVAwFoPpbZeufOZ24CukbCNiiD95fZah7h5mnoj4IzF7F8zOzAzHonfxlRCSmZuzpaI+rT7HC2BK662v1vSFk/CBROB6Ns52nqJvLWg89shekW7AfMKpEuI24bq2VhB2iToEyzJNEWXpYTAv7v2th6o15MHIwiHaSW2in/fRtfp+Y7ABmzgmUQrABBMjgDVrDki1yrJqowZYbzJvqPglKCgxitke2zu71W/2cdQZZ3h0ufHLKa/FKVkIfeKB+QS3jnhhRutd/J9elmAoLhvZXDRU4TiIRThtGNzgJUuxFbZFUMTnvJnWYTdZK4qg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=jIFZn3+Kli/g0kJ8HfhlrQTVKPrnGkJIoQreFf3IhUY=; b=LHg32s8Bog34paEdOi+Yoksn7ukqzMJnAK2p7v83lg/YndQv5SQjrWdGnnaG+h+wKSDHigEci8yHM0FeQuaAlcn1l+sNrrDT6KqZHi92ZxONlpoo6MLXCSQYQG1AfIBUxr+iey1kr6X3dw3yDSUH7K7BYbwjHadZik7sqAhmMko= Received: from DM6PR11MB4073.namprd11.prod.outlook.com (2603:10b6:5:19f::22) by DM6PR11MB2540.namprd11.prod.outlook.com (2603:10b6:5:c5::30) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4587.25; Tue, 12 Oct 2021 09:26:17 +0000 Received: from DM6PR11MB4073.namprd11.prod.outlook.com ([fe80::b083:17f4:8441:efed]) by DM6PR11MB4073.namprd11.prod.outlook.com ([fe80::b083:17f4:8441:efed%3]) with mapi id 15.20.4587.026; Tue, 12 Oct 2021 09:26:17 +0000 From: "Bob Feng" To: "Chen, Christine" , "devel@edk2.groups.io" CC: Liming Gao Subject: Re: [Patch V4 1/1] BaseTools: Change RealPath to AbsPath Thread-Topic: [Patch V4 1/1] BaseTools: Change RealPath to AbsPath Thread-Index: AQHXvx66mSg7JfTo302BloGRPHZwqKvPGBow Date: Tue, 12 Oct 2021 09:26:17 +0000 Message-ID: References: <20211012040738.1656-1-yuwei.chen@intel.com> In-Reply-To: <20211012040738.1656-1-yuwei.chen@intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=intel.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 83fb7cf8-2b60-42ad-9476-08d98d62584a x-ms-traffictypediagnostic: DM6PR11MB2540: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:6430; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: rNh/KVjBpp4deWEfs96cRNiKHfYmrxTMtQ9++8H1wkwS7qVdnALALBRqohymi1kA6C5OCD19e8bLpZLaJ4K6r+RaMBWWXjAhnzm8dFkNk+HL/bJcWHLkaBLEkALTwpV+NuPsqaduukGoMQrGnzIfLVHwq0WtSAeFGfKgoW0xEBGvIruidAdVZgOwoS+0eUXZk0IC51Ru//TwEYOBHD7Pq0CQwvAp7BTOJsjfXbw86wVUViQBbax8HC343zvKuNxQhaJQ7Jm5JFbYtlobYNHO5I7dHkbWq8jG11ZuiTFL0m+R723ZHsjgr52GWJ0OAR3GU+hnxAxKCBjAlMdytm/Pn+ZaJDFcD/SikcX0h6s11zhWp+hy7sq/ogf74XNjVBu2DHdDHLA9rWeB+kbq6ewAGNX9MoLRyP2NPYOuWyxwEidE+wVYSRqNY/qhzRs5bgONsWg5UmPKBk4+Wh+Wr+VC22L7MzTArDvMnfyT7rXKoUEC3g4jKgEKF0CfPXguhznDALYTajBJh0EsdoVTXyKW6GauC7xpvpiwyBL5uNfou6SLR6IkKHV4RbKgUoC/Krrww7epxzQI6zSCJAfzhDK1jJFp6EwcOLA4ovPbhwpCctMIgoUsNFsGDrDfuc8gEkM+lArUtCm8UveMWGudQaEjWaFkQicrM+Ozna5VgM5HvITeLzm/dMzwbqg2SsWEBiUn2xZRi93djxiiYkovTc5Syw== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DM6PR11MB4073.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(366004)(2906002)(9686003)(186003)(55016002)(83380400001)(64756008)(66476007)(66946007)(66446008)(71200400001)(76116006)(38070700005)(66556008)(8936002)(53546011)(110136005)(8676002)(316002)(38100700002)(122000001)(26005)(52536014)(33656002)(6506007)(86362001)(508600001)(4326008)(5660300002)(7696005);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?23517sX580yg3zq3y7dhmN3Q7U8byPjelTHJ2e903Ckb4BhS8WjHel25sxb7?= =?us-ascii?Q?eOSow9/wMrlf1oC22v1ANhshIQhYvkCPt2Ao3Hl6jqwKewUzw4RCglJHC85z?= =?us-ascii?Q?DFyAuxoo7ZQrzSpBUhMKSuXxQKGlRRRARNrtThG6GfVbKrFkpPJFyW3CxVW9?= =?us-ascii?Q?LzZYVd5evxGeWoyZdE7KKF9QQgieakDuEQgeZmyte4jjxnQ6EWzYCnoNlBp3?= =?us-ascii?Q?CMET3iFSoHgx+qAPI5A7j3CQqxTqSkJy80uxYVXgIbrxhuTQ3QKFlpn47+Vr?= =?us-ascii?Q?RwtQMhPbSKfHApUFbfe8s72WCtMCZ46Cbi26MMowN6MqwrojtK95JETbyBzr?= =?us-ascii?Q?rXEqx/sewGnWXx/iCdDUdBs3s7W/LpHAXIaDtSBKZw+CK1hoOEmlPcus1igA?= =?us-ascii?Q?5o31AkGXiZBC6x1C9zpTf6ML1wf15x2PpOhqyTkOFA/lNJ9XvyGKA5cIxRR/?= =?us-ascii?Q?yiDQgXdSjDviyMVg54Ueo6kURwZHVJxokWHZWRHLMaz3aLaTMpCQM9j5QIkS?= =?us-ascii?Q?EGeCIrLzohBvQJ2rozO3Z4RFWA2av+RMmwTgonbbZXFano8ilifa/lA2mE2u?= =?us-ascii?Q?ITU30cHpvtVNi7u0c+BIYPZplUBXUxSPAKoMV6w5fiOfUBThe3TKtYt6QJ+x?= =?us-ascii?Q?pvv3D1FLrGYMsjIN31vb1JVPFztjjLUzwZZD0cChwBcD4d19GN9vhhpOyWHl?= =?us-ascii?Q?bI6do8AlJK4R8q7z4fP9UZdDThodCNG9L5DFaSkjHNfe90b1nPVoAEsbR4WI?= =?us-ascii?Q?o8qO596V9wTYyvDjZkaOvP3xZdQt59OaCP3bdeT93q8WmsdsYM5x0KKpdnyI?= =?us-ascii?Q?JpiTfdBrl3Eqt5pAbevzd9Za0DTwiEzAbKdiKLmyIrEadcn00UVjDx6nJJxn?= =?us-ascii?Q?iVR6aQm2N5Lbun2Hd2qDmvW+v+oR929aI5x43Lx36OkE9dfk+jabitC5ffoU?= =?us-ascii?Q?4+jhiRE39WyzLgJcjK4iE3To17zfBnfKW1VK/Clx/+Fnr41dDcRrm8oLbAkK?= =?us-ascii?Q?1gdSspWD7nnJFUwWlyO8/KdtRXs0Hjdg6BHwsGETsHWCCBZ2CrZ8myzhpOHq?= =?us-ascii?Q?37MoOatDK6p+GS4JccBIl5kixydXtyGj8euJ51K5luZZY72VkGSjlmPafuJX?= =?us-ascii?Q?PxtsK7oEi7xiL13fXcsUiXw25H3hdrL2EY8Kr46+LewneLgKktp0uYmLp5as?= =?us-ascii?Q?kBTrNpHuGy8LMsW9mv6kpZwhe3EtBNvjDeeuvi7bEDSWkMoWDPDBerwOdYkF?= =?us-ascii?Q?5TCEJd1uUMOyQEBYMQRk0Webq3io8zHZphxT09brpUmFoWgZ8YgVRlzm52Qc?= =?us-ascii?Q?OKeMrLCMQJtk3SmF0dMwf6gG?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB4073.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 83fb7cf8-2b60-42ad-9476-08d98d62584a X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Oct 2021 09:26:17.6410 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: aOb9ezcmKuaYfK9eB/5qEvP29APXn4ukCrtrj964qUg9snuPNDkjczzhNhqGhTbog7e6TaoWXk8YpiA58gqoiw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB2540 Return-Path: bob.c.feng@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Reviewed-by: Bob Feng -----Original Message----- From: Chen, Christine =20 Sent: Tuesday, October 12, 2021 12:08 PM To: devel@edk2.groups.io Cc: Feng, Bob C ; Liming Gao Subject: [Patch V4 1/1] BaseTools: Change RealPath to AbsPath Currently the realpath is used when parse modules, which shows the path wit= h a drive letter in build log. In Windows 'subst' comand is used to associa= tes a path with a drive letter, when use the mapped drive letter for build,= with realpath function the build log will have different disk letter info = which will cause confusion. In this situation, if use adspath function to s= how the path info, it will keep same letter with the mapped drive letter, w= hich avoids confusion. This patch modifies the realpath to abspath. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Yuwei Chen --- BaseTools/Source/Python/Ecc/EccMain.py | 2 +- BaseTools/Source/Python/GenFds/FfsInfStatement.py | 4 ++-- BaseTools/Source/Python/GenFds/GenFds.py | 6 +++--- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/BaseTools/Source/Python/Ecc/EccMain.py b/BaseTools/Source/Pyth= on/Ecc/EccMain.py index 72edbea3b883..a349cd80147f 100644 --- a/BaseTools/Source/Python/Ecc/EccMain.py +++ b/BaseTools/Source/Python/Ecc/EccMain.py @@ -105,7 +105,7 @@ class Ecc(object): =20 def InitDefaultConfigIni(self): paths =3D map(lambda p: os.path.join(p, 'Ecc', 'config.ini'), sys.= path) - paths =3D (os.path.realpath('config.ini'),) + tuple(paths) + paths =3D (os.path.abspath('config.ini'),) + tuple(paths) for path in paths: if os.path.exists(path): self.ConfigFile =3D path diff --git a/BaseTools/Source/Python/GenFds/FfsInfStatement.py b/BaseTools/= Source/Python/GenFds/FfsInfStatement.py index 20573ca28d2f..568efb6d7685 100644 --- a/BaseTools/Source/Python/GenFds/FfsInfStatement.py +++ b/BaseTools/Source/Python/GenFds/FfsInfStatement.py @@ -707,8 +707,8 @@ class FfsInfStatement(FfsInfStatementClassObject): FileName, 'DEBUG' ) - OutputPath =3D os.path.realpath(OutputPath) - DebugPath =3D os.path.realpath(DebugPath) + OutputPath =3D os.path.abspath(OutputPath) + DebugPath =3D os.path.abspath(DebugPath) return OutputPath, DebugPath =20 ## __GenSimpleFileSection__() method diff --git a/BaseTools/Source/Pyt= hon/GenFds/GenFds.py b/BaseTools/Source/Python/GenFds/GenFds.py index c34104500059..17b71b7cd347 100644 --- a/BaseTools/Source/Python/GenFds/GenFds.py +++ b/BaseTools/Source/Python/GenFds/GenFds.py @@ -153,7 +153,7 @@ def GenFdsApi(FdsCommandDict, WorkSpaceDataBase=3DNone)= : FdfFilename =3D GenFdsGlobalVariable.ReplaceWorkspaceMacro(Fdf= Filename) =20 if FdfFilename[0:2] =3D=3D '..': - FdfFilename =3D os.path.realpath(FdfFilename) + FdfFilename =3D os.path.abspath(FdfFilename) if not os.path.isabs(FdfFilename): FdfFilename =3D mws.join(GenFdsGlobalVariable.WorkSpaceDir= , FdfFilename) if not os.path.exists(FdfFilename): @@ -175,7 +175,7 @@ def GenFdsApi(FdsCommandDict, WorkSpaceDataBase=3DNone)= : ActivePlatform =3D GenFdsGlobalVariable.ReplaceWorkspaceMacro(= ActivePlatform) =20 if ActivePlatform[0:2] =3D=3D '..': - ActivePlatform =3D os.path.realpath(ActivePlatform) + ActivePlatform =3D os.path.abspath(ActivePlatform) =20 if not os.path.isabs (ActivePlatform): ActivePlatform =3D mws.join(GenFdsGlobalVariable.WorkSpace= Dir, ActivePlatform) @@ -299,7 +299,7 @@ def GenFdsApi(FdsCommandDict, Work= SpaceDataBase=3DNone): for Key in GenFdsGlobalVariable.OutputDirDict: OutputDir =3D GenFdsGlobalVariable.OutputDirDict[Key] if OutputDir[0:2] =3D=3D '..': - OutputDir =3D os.path.realpath(OutputDir) + OutputDir =3D os.path.abspath(OutputDir) =20 if OutputDir[1] !=3D ':': OutputDir =3D os.path.join (GenFdsGlobalVariable.WorkSpace= Dir, OutputDir) -- 2.27.0.windows.1