From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from NAM11-CO1-obe.outbound.protection.outlook.com (NAM11-CO1-obe.outbound.protection.outlook.com [40.92.18.14]) by mx.groups.io with SMTP id smtpd.web11.30796.1683143437665368399 for ; Wed, 03 May 2023 12:50:38 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@outlook.com header.s=selector1 header.b=mLWQhUsz; spf=pass (domain: outlook.com, ip: 40.92.18.14, mailfrom: spbrogan@outlook.com) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZY59didOirCSj4VEt/sYVZLeEIiHjJGScG5/2bFllLKf2g6AmGfLgeByM1Ey8EW29/GTofHo/3LJmeulYovCHwra+BLcpgUH4Ya8HY+J6SAHtUDHVjB6Rijv/GqyG0C7f+23RVwh2Ip7i/pOd+JIGfM0fTHoHLnKTsdvt2OSibQevOjppCNIi3QziB/5MT4zUr9OyhabXneECb/0I7/etDt8P9e6kw57OI/Mib+tCGgCbv4fLiakli0d+b8+8eYI6HcNVhao9MVo3P8zfnd0qRtgNOzPx2b3r1G0bhCCUlPA0MBc7nvtYiO3RYyxOJxeAI5LTHRSAjpBnB9/cVmjfQ== 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=Pis7xZ6yeQcuFI+JqpDCEFsAtKETfQi3lx6drkTXHP4=; b=H+XlKp/Dw+hSJcn+ywoBtq6awq6f1l3ZoJCedVqhuRbhSYUgC9EdOrre3FguZHXbEdvyNmBu7QWKTOvjtpR9ZS9ku27bUPgUBaBoIotgR0yI16clnwesKt51UOPpCA+tdQ/DKW+sWgZlVQSI8zHMuMipPqfCJC25C83KaRdATLZiqMrXNad84m1TO27yCZcDhlDWFUzqjYzeJeE7K8E1PS0ftkPDnJ5T96XpIpq9Zj8Xql6RRaX7R0OHpTuo7ghugHSQ9hNQZk4RdhY8Z6WBMlCE90E2t6L1J4iegPOgVt3c4RYtl0iG0eEnCz4NtcP+ox8s1Owg3Zg9d/t7ACnPfA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none 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=Pis7xZ6yeQcuFI+JqpDCEFsAtKETfQi3lx6drkTXHP4=; b=mLWQhUsz4UH+Gk9uwuFvGZW77LOqx1mItzR6695jo4y3sUG5wn+ItdzBKbn1BWHqh0RsH6IM5f3mTYK3U1sXzsNKgDytalOI+9z9x93KvwTNIBj9jYMe9h8UC7QICByAWYFH01YS6DbZDFe5VaaVkDWSq0tqop7PlYkNcOwmVnQOkD/T+GaeqBgXg6Rrt2YtwvN/Qpn5txtjrMdOb0B6g8z65xDV/5FFISPxk1Y14uiQtNSNylHyqxDbSqJ+ew1h7Uu821+4puWOGTlSV7CNkephZCh57cI8AmiFEBxwMzTwQhNI7PGFKvR0hcEt+f1feEuXOqjA0RTiPP1ExkB7jQ== Received: from BY3PR19MB4900.namprd19.prod.outlook.com (2603:10b6:a03:354::11) by SA1PR19MB7133.namprd19.prod.outlook.com (2603:10b6:806:2ba::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6363.20; Wed, 3 May 2023 19:50:35 +0000 Received: from BY3PR19MB4900.namprd19.prod.outlook.com ([fe80::358d:ef31:7691:6ec4]) by BY3PR19MB4900.namprd19.prod.outlook.com ([fe80::358d:ef31:7691:6ec4%6]) with mapi id 15.20.6387.008; Wed, 3 May 2023 19:50:35 +0000 Message-ID: Date: Wed, 3 May 2023 12:50:34 -0700 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.10.1 Subject: Re: [edk2-devel] OvmfPkg PlatformCI: Should iasl dependency be updated from 20190215.0.0 ? To: devel@edk2.groups.io, rebecca@bsdio.com, Ard Biesheuvel , Jiewen Yao , Jordan Justen , Gerd Hoffmann Cc: Sean Brogan , Michael Kubacki , Michael D Kinney , Liming Gao References: <5970efe0-b520-f92d-db2f-6ed809ce4e55@bsdio.com> From: "Sean" In-Reply-To: <5970efe0-b520-f92d-db2f-6ed809ce4e55@bsdio.com> X-TMN: [blKKe8wFrggt/8MirwNYypww/MMSE4tnkKlpmeukVN/MhCwz577Q+TLEkZfiTg69] X-ClientProxiedBy: CY5PR14CA0009.namprd14.prod.outlook.com (2603:10b6:930:2::31) To BY3PR19MB4900.namprd19.prod.outlook.com (2603:10b6:a03:354::11) Return-Path: spbrogan@outlook.com X-Microsoft-Original-Message-ID: MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BY3PR19MB4900:EE_|SA1PR19MB7133:EE_ X-MS-Office365-Filtering-Correlation-Id: 10446e04-412c-4415-4e1a-08db4c0fa953 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: viljRx3NQQJW6vywOqUaHrj8UWDIc5/PIJM90niU4nEcQEFR2Ozf+cORHUBHw3pNWUmyaR605QZZmkU3hpMw6H3pAN2btjDC9KzFXQRkxEjSR3Ejh/ORAsaPRddCkMRazgN3NGWHP9WF93i5fJTY+I7TiUXA30ZzSE5/keK1fzdKn2tE7l968QIAEsr6a5YGIMaJNCGWNRMZUs7DTuG4UUA+L/4DC+j7NclIFeYTdxTPVtwcANACAJFD9kx5R88jImJmsxQqyDOBOPLQdx/839TKxKrfP3OTjEptNA98J1Mj9qoaQMsMd8TCd6q0dtV9dSFpguKCTTCNXK2MR8RVPku8DHx6Khq/1xsS+CQCWyVXt+f4VMXxWTulooN1ombj/Ui81vqC+dMNyU6Gw1y9E051FvWB3NrtDl9rKp03++WIeOwpM29JHOJNUhJ0olx97Fyl3Fuea2tm18mOGr4n7akhS+1xgJMZs/O5ejnVMJ2BsF3a4UqZZ6rgcdsLhQeFOausPyDPeXKPMLPTFr/mCCXlV3qdrqC/jyvK/xpK2a0+D0JpNSyuV6g9dsOcFZN+ocdDf9+QqM1DJCRXSsSoVzoZsPbQSlwi4Hbo6QzzSXDfe1f3gpnmz8mBNAPMYVFKuZTOnx5usModeisvvOq6kMkszbGOxws5DAdPJYMGCMI= X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?V0pnMFo5Y0J1cVhFL1hialpCMy9pOCtQSkh6UnhDSEJvalJuZHltdU9rTzEr?= =?utf-8?B?Tnd1aEdsOW1zT2kwSXpjdGxDdU11T2V0YnhxazZoMXBBOW5lZjk2eWk0Q210?= =?utf-8?B?MC9PNXNzeFROR2wvWmZ4RUozZFNCMHh5b3lDNVJqaWgxS2VXeFVnd2cxQjlh?= =?utf-8?B?a3VIb0s2cmVseVIwOThnYWwwNVFMTm9uSGFpWHhiZk9tNWtRcmV0MlJwQnFI?= =?utf-8?B?SmtPdXNZNXNLdi9WREs4WHpuV1BmSTh1K2pZYmNaTkU1OXZKTDdXbks4eHoz?= =?utf-8?B?YlBwSDA3L2NDUmhObzNxSzZVZ28zTWdscFQ4RVBqQm1hL0dKQVhwQ1NhV3o0?= =?utf-8?B?MGRPRFpqOVA4dUF4dlhqNjBHeUNoYUZ3SEt1WElkeTNuRFYzS0RCeG1WU2dB?= =?utf-8?B?MnhhZFlOc1MwU3k4c0dmL1krcmg1dnZCNmJnVjZmOG13ZnVGc2RzQVJFRjB0?= =?utf-8?B?NUpKWXdxejFUdGVnSkZJczBURUxaYjYvcGVzZjBHdVNERUxVQk1tcHRFQzU1?= =?utf-8?B?N2l4LzYwYkN4ellCYTJEbVpOYTVqSlg1V0VxR2xqNGw5SHJXYWRxdXA1STlX?= =?utf-8?B?cTU3WnVVZGo2MkhycE5qRExtdGp0UmU1eXlaS25oTjNzMDkxYTJibEEzS1Av?= =?utf-8?B?bERDbHVuQTBoMDIzakxiN3RrZEJFak05VUhoamFteWlJN0NBOE9xeks2OElm?= =?utf-8?B?SnlyR2NWaG9CWHAvTGUxdmMybEJWZjZ2L0VJMjdwUms1MzE5aEg3RHVMVnBs?= =?utf-8?B?QnhLZWcyd0JDWDh0QzhCT2U3WTJXVjhLNUIrc203SUZsZHhCT3U2S1RFaTIw?= =?utf-8?B?cGVUT2o4dmtjcWlTcFZiWXd6eGdZSmhVaSt6T2ttdWhzRlFxdi85NExqb2NR?= =?utf-8?B?NEptZllzWVVaSUhHd21qNVo0b3lOdUtkS2FUOVBwQkh2aTRjeVdHZEhiZ2V5?= =?utf-8?B?RmVPMTd5MkhQVVFXLzlIdUtRVG1NQUswUDVQUnJtRDFIRWpnM2FnR0lUQXo0?= =?utf-8?B?ZThuR21vc1FGckFoVVBwaWFOSWUwNVBscXh2dkZpbi8xeFR0N25MdkM3Zjhi?= =?utf-8?B?TkQ4SHNyN2ZYdjlsUDNrTW5DTDNTWmZhRGlucHhXMUdoT3R5V1k0bWxscWNN?= =?utf-8?B?eXU5ZUFkdUNNQU5BbkxTYVZBbTVEK1dxcWcySEdRVnQ0SVpURnRXZUJLQ2Nl?= =?utf-8?B?VEVDcHZCMXpMMTRBRURTRHlORjZvSElkUlJhMDU4aXIxL3NDeEcwUENLcHBt?= =?utf-8?B?S2diaUZXTUF3SlovK0cyUlR2Znp3aUIzTkhYU0NrK2xDM1doVGpITjBZTFJO?= =?utf-8?B?Mi8xNFBNdWt0b0ovSGV2aElPYkVLK0lWVjE4Sk9hcFgrU0NUcXQwMHM1cm5F?= =?utf-8?B?a05vZmRIdmFhK1pjSDc2eXUxbTM5RGwxZzFLTVhqUG83dTBPMHhYZlpZVnZx?= =?utf-8?B?akZPWVdJWGFCZGVJS2I2eEpwaHN3Ty9WMU5NbXBKdDhXSEhuQ2FsZnUyUFEx?= =?utf-8?B?SEJ0ZnVvZ0phVWFDTzZKcVBUTkE5d0ZVN0ZYMEM1YnV1bGk4ZyttSHh6Qnls?= =?utf-8?B?VzdlOVRkWnZ4QzRleUFwL21CZ0EyR1lEQTdzQkg5eDlYU3EzWitzUm80UmFO?= =?utf-8?B?cFRiS2RVTWNMRm84ZHNGT0huUFJiZ0Zvb2NiSXN0SmRzL0VubzhyS1VOeVEr?= =?utf-8?B?Skt6STNDOVRld2VONWxGMDBkTEx3L3lVN3BhclFLbHZjUUJ5VFFVUDhnPT0=?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 10446e04-412c-4415-4e1a-08db4c0fa953 X-MS-Exchange-CrossTenant-AuthSource: BY3PR19MB4900.namprd19.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 May 2023 19:50:35.7085 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR19MB7133 Content-Type: multipart/alternative; boundary="------------K6Oc6fZW7RZLPi1ParwQOSeb" Content-Language: en-US --------------K6Oc6fZW7RZLPi1ParwQOSeb Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Rebecca, I agree the iasl dependency for CI has not been managed consistently.   When all of the CI was setup we decided that iasl should be controlled by the platform and thus EmulatorPkg, ArmVirt, and OVMF have their own extdep.  This gives those platforms control to rev their version as necessary for their platform.   We have found it very common in platform development for platforms to have different required versions of iasl.exe. For Core CI (meaning just package builds) we managed iasl in Basetools.  We decided that this would only need to be updated when newer features were used by core components (not very often given the very little ASL in edk2 tree). As for the feed.  Yes they are inconsistent.   We were moving away from a global nuget.org feed as it just didn't seem necessary to push to nuget.org.  But now we are evaluating ways to move entirely away from nuget.  Nuget.exe worked pretty well for Windows development and our initial use cases but has definitely created a headache on Linux, MacOS and other.  There really isn't a generic package management solution that is supported cross platform that has free/high quality/secure hosting.  If anyone has ideas please share. So my suggestion is to hold off for a couple of weeks (unless something is broken) and lets see if we can use web downloads from github releases.  This should still allow consistency with tools, work cross platform, and give the flexibility needed per platform. Regarding the steps in that document.  In that example it doesn't call out all the steps needed as that would just rehash the section before.  Instead it relies on a user having followed the generic steps in the section above (How to Build With Stuart · tianocore/tianocore.github.io Wiki ). For example the user would need to have also done:  setup python virtual env, install pypi dependencies, and clone source + submodules. Thanks Sean On 5/3/2023 12:34 PM, Rebecca Cran wrote: > I noticed OvmfPkg/PlatformCI/iasl_ext_dep.yaml specifies iasl version > 20190215.0.0 while BaseTools/Bin/iasl_ext_dep.yaml has the newer > 20200717.0.0, and "mono .../edk2toolext/bin/NuGet.exe list -Source > https://pkgs.dev.azure.com/projectmu/acpica/_packaging/mu_iasl/nuget/v3/index.json" > shows there's version 20210105.0.6 available. > > > Though OvmfPkg is using source https://api.nuget.org/v3/index.json > while BaseTools uses > https://pkgs.dev.azure.com/projectmu/acpica/_packaging/mu_iasl/nuget/v3/index.json > - I don't know why they're different. > > > I was wondering if iasl_ext_dep.yaml should be updated? > > > Also, the example in > https://github.com/tianocore/tianocore.github.io/wiki/How-to-Build-With-Stuart > of using stuart_build to build OVMF seems to be missing a step: > running "stuart_update -c PlatformBuild.py TOOL_CHAIN_TAG=GCC5 -a X64" > appears to be required otherwise stuart_build will complain that the > iasl dependency hasn't been met. > > --------------K6Oc6fZW7RZLPi1ParwQOSeb Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: 8bit

Rebecca,

I agree the iasl dependency for CI has not been managed consistently.   When all of the CI was setup we decided that iasl should be controlled by the platform and thus EmulatorPkg, ArmVirt, and OVMF have their own extdep.  This gives those platforms control to rev their version as necessary for their platform.   We have found it very common in platform development for platforms to have different required versions of iasl.exe. 

For Core CI (meaning just package builds) we managed iasl in Basetools.  We decided that this would only need to be updated when newer features were used by core components (not very often given the very little ASL in edk2 tree). 


As for the feed.  Yes they are inconsistent.   We were moving away from a global nuget.org feed as it just didn't seem necessary to push to nuget.org.  But now we are evaluating ways to move entirely away from nuget.  Nuget.exe worked pretty well for Windows development and our initial use cases but has definitely created a headache on Linux, MacOS and other.  There really isn't a generic package management solution that is supported cross platform that has free/high quality/secure hosting.  If anyone has ideas please share. 


So my suggestion is to hold off for a couple of weeks (unless something is broken) and lets see if we can use web downloads from github releases.  This should still allow consistency with tools, work cross platform, and give the flexibility needed per platform. 


Regarding the steps in that document.  In that example it doesn't call out all the steps needed as that would just rehash the section before.  Instead it relies on a user having followed the generic steps in the section above (How to Build With Stuart · tianocore/tianocore.github.io Wiki).  For example the user would need to have also done:  setup python virtual env, install pypi dependencies, and clone source + submodules. 


Thanks

Sean




On 5/3/2023 12:34 PM, Rebecca Cran wrote:
I noticed OvmfPkg/PlatformCI/iasl_ext_dep.yaml specifies iasl version 20190215.0.0 while BaseTools/Bin/iasl_ext_dep.yaml has the newer 20200717.0.0, and "mono .../edk2toolext/bin/NuGet.exe list -Source https://pkgs.dev.azure.com/projectmu/acpica/_packaging/mu_iasl/nuget/v3/index.json" shows there's version 20210105.0.6 available.


Though OvmfPkg is using source https://api.nuget.org/v3/index.json while BaseTools uses https://pkgs.dev.azure.com/projectmu/acpica/_packaging/mu_iasl/nuget/v3/index.json - I don't know why they're different.


I was wondering if iasl_ext_dep.yaml should be updated?


Also, the example in https://github.com/tianocore/tianocore.github.io/wiki/How-to-Build-With-Stuart of using stuart_build to build OVMF seems to be missing a step: running "stuart_update -c PlatformBuild.py TOOL_CHAIN_TAG=GCC5 -a X64" appears to be required otherwise stuart_build will complain that the iasl dependency hasn't been met.


--------------K6Oc6fZW7RZLPi1ParwQOSeb--