From nobody Tue Apr 30 20:00:39 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; 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; dmarc=fail(p=none dis=none) header.from=outlook.com Return-Path: Received: from ml01.01.org (ml01.01.org [198.145.21.10]) by mx.zohomail.com with SMTPS id 1526683402574484.9311674472847; Fri, 18 May 2018 15:43:22 -0700 (PDT) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 4C4FD2096DCFA; Fri, 18 May 2018 15:43:22 -0700 (PDT) Received: from EUR03-AM5-obe.outbound.protection.outlook.com (mail-oln040092070076.outbound.protection.outlook.com [40.92.70.76]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 10BD720961828 for ; Fri, 18 May 2018 15:43:20 -0700 (PDT) Received: from AM5EUR03FT039.eop-EUR03.prod.protection.outlook.com (10.152.16.53) by AM5EUR03HT204.eop-EUR03.prod.protection.outlook.com (10.152.17.9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.20.735.16; Fri, 18 May 2018 22:43:18 +0000 Received: from VI1PR0801MB1790.eurprd08.prod.outlook.com (10.152.16.53) by AM5EUR03FT039.mail.protection.outlook.com (10.152.17.185) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.20.755.15 via Frontend Transport; Fri, 18 May 2018 22:43:18 +0000 Received: from VI1PR0801MB1790.eurprd08.prod.outlook.com ([fe80::7010:5d5e:8dcf:d92e]) by VI1PR0801MB1790.eurprd08.prod.outlook.com ([fe80::7010:5d5e:8dcf:d92e%17]) with mapi id 15.20.0776.010; Fri, 18 May 2018 22:43:18 +0000 X-Original-To: edk2-devel@lists.01.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; Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=40.92.70.76; helo=eur03-am5-obe.outbound.protection.outlook.com; envelope-from=marvin.haeuser@outlook.com; receiver=edk2-devel@lists.01.org DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=iqtbh32UQwuie0qSU60bUMJmWo84hC5Z4jMkDjRiPQg=; b=JYL49BfY+TJBUQPxgnEL7qLZjb9LcRRZy/P9lPhJIG4sF/UxwTWp12fUnBn0Tx9ppgVCy2JMQraLrH6PL1MryNKBHEkaUlSxMEdgdrHuNcmg0GxCMFqnRrNI6VYQxrOta+/8FjrlI37wYwyuJwX6y4AEcG6SSRjWoih8R2BeSnYDov3x3p6Gpx2lXI7Yjidnpd9IynHMRftv8xczhtbtKm95V9wvuEOX6JbkoUU+UOk5xo5KA1ofGqCw0gxjkWkwtkQyOQ+lv939g03+tVV9xqnmaD7wFZxpXCDVjCsBpImtAXtc0s2l7OavAjjNKB+yxwNF9EuKAiQV/kmtqqdLJw== From: =?iso-8859-1?Q?Marvin_H=E4user?= To: "edk2-devel@lists.01.org" Thread-Topic: [PATCH] BaseTools/Workspace: Fix ValueChain set Thread-Index: AQHT7vmdMiu5Nl+8DkWkXeAtQCpjQw== Date: Fri, 18 May 2018 22:43:18 +0000 Message-ID: Accept-Language: de-DE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-clientproxiedby: AM5PR0701CA0071.eurprd07.prod.outlook.com (2603:10a6:203:2::33) To VI1PR0801MB1790.eurprd08.prod.outlook.com (2603:10a6:800:5b::15) x-incomingtopheadermarker: OriginalChecksum:CA7FB67ED6C7781E2666F128ED3DD9A76CD6272A639AC82DECACA5DFEB2FB09B; UpperCasedChecksum:051041CD4BA0AA33E0921E373A559B17760090584FDE9E3F30D2611730FF6824; SizeAsReceived:7491; Count:48 x-ms-exchange-messagesentrepresentingtype: 1 x-tmn: [7cyeXtFyMJnE2iDCpSvRxsxAbC8WIY6i] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1; AM5EUR03HT204; 7:CmQ+RpdG6f8xPW1ce6MRGCcPFX3f1d9QffHEl9IG9qXG5V1zat1s4anCP6q9d8q9uBQmfXqLPR9miox02K18bHtzV1+lfVvT3/YjOb+9CtfHTE03HaRR2u6IoJoTy3STlQdPzgjoJ9WnDo69SG9DFqlfGUSjy3rgc51hP+eb6FzSUP7CySGoY9jMFSaLTFMGWyZydoY25/sZd35olzbEU6dWdGyxRlZUiEOsOhVm158SvBSP086otoIv4gJhqbGJ x-incomingheadercount: 48 x-eopattributedmessage: 0 x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(7020095)(201702061078)(5061506573)(5061507331)(1603103135)(2017031320274)(2017031324274)(2017031323274)(2017031322404)(1601125466)(1603101448)(1701031045); SRVR:AM5EUR03HT204; x-ms-traffictypediagnostic: AM5EUR03HT204: x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(444000031); SRVR:AM5EUR03HT204; BCL:0; PCL:0; RULEID:; SRVR:AM5EUR03HT204; x-forefront-prvs: 0676F530A9 x-forefront-antispam-report: SFV:NSPM; SFS:(7070007)(189003)(199004)(305945005)(6916009)(4326008)(2351001)(86362001)(2900100001)(5660300001)(97736004)(82202002)(3280700002)(5640700003)(74316002)(3660700001)(55016002)(2501003)(33656002)(5250100002)(8936002)(54906003)(68736007)(8676002)(81156014)(106356001)(105586002)(7696005)(476003)(104016004)(87572001)(486006)(72206003)(426003)(99286004)(25786009)(26005)(1857600001)(14454004)(102836004)(6346003)(386003)(20460500001)(6436002); DIR:OUT; SFP:1901; SCL:1; SRVR:AM5EUR03HT204; H:VI1PR0801MB1790.eurprd08.prod.outlook.com; FPR:; SPF:None; PTR:InfoNoRecords; MX:1; A:1; LANG:; received-spf: None (protection.outlook.com: outlook.com does not designate permitted sender hosts) authentication-results: spf=none (sender IP is ) smtp.mailfrom=Marvin.Haeuser@outlook.com; x-microsoft-antispam-message-info: fIdlkAoBBIfzwa8LH2dHQ0tHGt4LytNO02T3uY0jrjVR5/3sIxbw8fvxZnhLWxAaln4d1MduSS01xrYJT9ayzJG9EN6mar1py1Gd4T8JmNx8TN7xJLD+iBjr5YxWE5YPzAlgundbGZwBqfQd8PrtBu/30aG6eBLBVPjYIqcZfxd+6AUKP1HMmRZPwirosICA MIME-Version: 1.0 X-MS-Office365-Filtering-Correlation-Id: 325e0b58-44b9-4edf-d2d2-08d5bd10bff2 X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 7181d4b0-87d6-4f4e-ba33-0d3746212cec X-MS-Exchange-CrossTenant-Network-Message-Id: 325e0b58-44b9-4edf-d2d2-08d5bd10bff2 X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 7181d4b0-87d6-4f4e-ba33-0d3746212cec X-MS-Exchange-CrossTenant-originalarrivaltime: 18 May 2018 22:43:18.6844 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Internet X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM5EUR03HT204 Subject: [edk2] [PATCH] BaseTools/Workspace: Fix ValueChain set X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: "jaben.carsey@intel.com" , "liming.gao@intel.com" Content-Transfer-Encoding: quoted-printable Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) X-ZohoMail: RDKM_2 RSF_4 Z_629925259 SPT_0 Content-Type: text/plain; charset="utf-8" Commit 88252a90d1ca7846731cd2e4e8e860454f7d97a3 changed ValueChain from a dict to a set, but also changed the (former) key type from a touple to two separate values, which was probably unintended and also breaks build for packages involving Structured PCDs, because add() only takes one argument. This commit changes the values back to touples. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Marvin Haeuser --- BaseTools/Source/Python/Workspace/DscBuildData.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/BaseTools/Source/Python/Workspace/DscBuildData.py b/BaseTools/= Source/Python/Workspace/DscBuildData.py index 8476543c5352..0262a91ed855 100644 --- a/BaseTools/Source/Python/Workspace/DscBuildData.py +++ b/BaseTools/Source/Python/Workspace/DscBuildData.py @@ -1347,7 +1347,7 @@ class DscBuildData(PlatformBuildClassObject): nextskuid =3D self.SkuIdMgr.GetNextSkuId(nextskuid) stru_pcd.SkuOverrideValues[skuid] =3D copy.deepcopy(st= ru_pcd.SkuOverrideValues[nextskuid]) if not NoDefault else copy.deepcopy({d= efaultstorename: stru_pcd.DefaultValues for defaultstorename in DefaultStor= es} if DefaultStores else {TAB_DEFAULT_STORES_DEFAULT:stru_pcd.DefaultValue= s}) if not NoDefault: - stru_pcd.ValueChain.add(skuid,'') + stru_pcd.ValueChain.add((skuid,'')) if stru_pcd.Type in [self._PCD_TYPE_STRING_[MODEL_PCD_DYNAMIC_= HII], self._PCD_TYPE_STRING_[MODEL_PCD_DYNAMIC_EX_HII]]: for skuid in SkuIds: nextskuid =3D skuid @@ -1366,7 +1366,7 @@ class DscBuildData(PlatformBuildClassObject): for defaultstoreid in DefaultStores: if defaultstoreid not in stru_pcd.SkuOverrideValue= s[skuid]: stru_pcd.SkuOverrideValues[skuid][defaultstore= id] =3D copy.deepcopy(stru_pcd.SkuOverrideValues[nextskuid][mindefaultstore= name]) - stru_pcd.ValueChain.add(skuid,defaultstoreid) + stru_pcd.ValueChain.add((skuid,defaultstoreid)) S_pcd_set =3D DscBuildData.OverrideByFdfComm(S_pcd_set) Str_Pcd_Values =3D self.GenerateByteArrayValue(S_pcd_set) if Str_Pcd_Values: --=20 2.17.0.windows.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel