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

[Michael D Kinney]

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 at edk2.groups.io <devel at edk2.groups.io> On Behalf Of PierreGondois
> Sent: Monday, November 15, 2021 9:09 AM
> To: devel at edk2.groups.io
> Cc: Ard Biesheuvel <ardb+tianocore at kernel.org>; Leif Lindholm <leif at nuviainc.com>; Sean Brogan
> <sean.brogan at microsoft.com>; Bret Barkelew <Bret.Barkelew at microsoft.com>; Kinney, Michael D <michael.d.kinney at intel.com>;
> Liming Gao <gaoliming at byosoft.com.cn>; Sami Mujawar <sami.mujawar at arm.com>; Pierre Gondois <Pierre.Gondois at arm.com>
> Subject: [edk2-devel] [PATCH edk2-platforms v5 1/6] edk2-platforms: add edk2 repository as a submodule
> 
> From: Pierre Gondois <Pierre.Gondois at 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 at kernel.org>
> Cc: Leif Lindholm <leif at nuviainc.com>
> Cc: Sean Brogan <sean.brogan at microsoft.com>
> Cc: Bret Barkelew <Bret.Barkelew at microsoft.com>
> Cc: Michael D Kinney <michael.d.kinney at intel.com>
> Cc: Liming Gao <gaoliming at byosoft.com.cn>
> Cc: Sami Mujawar <sami.mujawar at arm.com>
> Co-authored-by: Sami Mujawar <sami.mujawar at arm.com>
> Signed-off-by: Pierre Gondois <Pierre.Gondois at 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
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#83760): https://edk2.groups.io/g/devel/message/83760
Mute This Topic: https://groups.io/mt/87074374/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-