Re: [edk2-devel] [PATCH edk2-platforms v5 1/6] edk2-platforms: add edk2 repository as a submodule

["Michael D Kinney" <michael.d.kinney@intel.com>]

Hi Pierre,

There are many ways to pull multiple repos into a CI build.

This method of adding edk2 as a submodule of edk2-platforms is functional,
but may cause some CI system to fetch the edk2 repo multiple times if they
are using a different methods to manage multiple repos.

PyTools, EdkRepo, and CI agent scripts are some of the alternate methods.

I think this topic requires a discussion with all the edk2-platforms platform
owners to determine how TianoCore can provide a consistent way to describe the
repository dependencies and allow some platforms to use latest edk2/master
and other platforms to use branch/tag/sha such as an edk2 stable tag release.
We need this to be simple and easy to use for both TianoCore developers and
easy to consume by the4 TianoCore downstream consumers.

In order to make progress on enabling CI for this specific platform now,
is it possible to fetch of edk2 repo using Azure Pipelines scripts?  Or
other techniques that do not add edk2 as a submodule dependency at this time?

Thanks,

Mike

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of PierreGondois
> Sent: Monday, November 15, 2021 9:09 AM
> To: devel@edk2.groups.io
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>; Leif Lindholm <leif@nuviainc.com>; Sean Brogan
> <sean.brogan@microsoft.com>; Bret Barkelew <Bret.Barkelew@microsoft.com>; Kinney, Michael D <michael.d.kinney@intel.com>;
> Liming Gao <gaoliming@byosoft.com.cn>; Sami Mujawar <sami.mujawar@arm.com>; Pierre Gondois <Pierre.Gondois@arm.com>
> Subject: [edk2-devel] [PATCH edk2-platforms v5 1/6] edk2-platforms: add edk2 repository as a submodule
> 
> From: Pierre Gondois <Pierre.Gondois@arm.com>
> 
> Add the edk2 repository as a submodule:
> https://github.com/tianocore/edk2
> 
> Platforms in edk2-platforms often relies on modules available
> in the edk2 repository. In order to enable an upstream CI
> for edk2-platforms, adding edk2 as a submodule is a convenient
> way to advertise this dependency.
> 
> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3509
> 
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Sean Brogan <sean.brogan@microsoft.com>
> Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Sami Mujawar <sami.mujawar@arm.com>
> Co-authored-by: Sami Mujawar <sami.mujawar@arm.com>
> Signed-off-by: Pierre Gondois <Pierre.Gondois@arm.com>
> ---
>  .gitmodules | 3 +++
>  edk2        | 1 +
>  2 files changed, 4 insertions(+)
>  create mode 160000 edk2
> 
> diff --git a/.gitmodules b/.gitmodules
> index 88aafaf15820..ed4b2d436cdb 100644
> --- a/.gitmodules
> +++ b/.gitmodules
> @@ -1,3 +1,6 @@
>  [submodule "Silicon/RISC-V/ProcessorPkg/Library/RiscVOpensbiLib/opensbi"]
>    path = Silicon/RISC-V/ProcessorPkg/Library/RiscVOpensbiLib/opensbi
>    url = https://github.com/riscv/opensbi
> +[submodule "edk2"]
> +	path = edk2
> +	url = https://github.com/tianocore/edk2/
> diff --git a/edk2 b/edk2
> new file mode 160000
> index 000000000000..e1e7306b5414
> --- /dev/null
> +++ b/edk2
> @@ -0,0 +1 @@
> +Subproject commit e1e7306b54147e65cb7347b060e94f336d4a82d2
> --
> 2.17.1
> 
> 
> 
> 
>