From: "Sean" <spbrogan@outlook.com>
To: devel@edk2.groups.io, rebecca@bsdio.com,
Ard Biesheuvel <ardb+tianocore@kernel.org>,
Jiewen Yao <jiewen.yao@intel.com>,
Jordan Justen <jordan.l.justen@intel.com>,
Gerd Hoffmann <kraxel@redhat.com>
Cc: Sean Brogan <sean.brogan@microsoft.com>,
Michael Kubacki <mikuback@linux.microsoft.com>,
Michael D Kinney <michael.d.kinney@intel.com>,
Liming Gao <gaoliming@byosoft.com.cn>
Subject: Re: [edk2-devel] OvmfPkg PlatformCI: Should iasl dependency be updated from 20190215.0.0 ?
Date: Wed, 3 May 2023 12:50:34 -0700 [thread overview]
Message-ID: <BY3PR19MB490092E74E33E6A58C74C3C4C86C9@BY3PR19MB4900.namprd19.prod.outlook.com> (raw)
In-Reply-To: <5970efe0-b520-f92d-db2f-6ed809ce4e55@bsdio.com>
[-- Attachment #1: Type: text/plain, Size: 3109 bytes --]
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
<https://github.com/tianocore/tianocore.github.io/wiki/How-to-Build-With-Stuart#how-to-build-in-edk2-with-stuart>).
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.
>
>
[-- Attachment #2: Type: text/html, Size: 4549 bytes --]
next prev parent reply other threads:[~2023-05-03 19:50 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-05-03 19:34 OvmfPkg PlatformCI: Should iasl dependency be updated from 20190215.0.0 ? Rebecca Cran
2023-05-03 19:50 ` Sean [this message]
2023-05-03 20:06 ` [edk2-devel] " Rebecca Cran
2023-05-04 5:56 ` Gerd Hoffmann
2023-05-05 3:13 ` Sean
2023-05-04 17:21 ` Michael D Kinney
2023-05-05 2:17 ` Michael Kubacki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=BY3PR19MB490092E74E33E6A58C74C3C4C86C9@BY3PR19MB4900.namprd19.prod.outlook.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox