[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 05/10] roms/edk2: update submodule from edk2-stable201905 to
From: |
Laszlo Ersek |
Subject: |
Re: [PATCH 05/10] roms/edk2: update submodule from edk2-stable201905 to edk2-stable202008 |
Date: |
Tue, 8 Sep 2020 14:08:59 +0200 |
On 09/08/20 10:22, Philippe Mathieu-Daudé wrote:
> Hi Laszlo,
>
> On 9/8/20 9:29 AM, Laszlo Ersek wrote:
>> Update the edk2 submodule from release edk2-stable201905 to
>> edk2-stable202008. The release notes can be read at
> [...]
>>
>> Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
>> Ref: https://bugs.launchpad.net/qemu/+bug/1852196
>> Signed-off-by: Laszlo Ersek <lersek@redhat.com>
>> ---
>> roms/edk2 | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/roms/edk2 b/roms/edk2
>> index 20d2e5a125e3..06dc822d045c 160000
>> --- a/roms/edk2
>> +++ b/roms/edk2
>> @@ -1 +1 @@
>> -Subproject commit 20d2e5a125e34fc8501026613a71549b2a1a3e54
>> +Subproject commit 06dc822d045c2bb42e497487935485302486e151
>
> FYI applying this I got:
>
> Fetching submodule roms/edk2
> Fetching submodule roms/edk2/CryptoPkg/Library/OpensslLib/openssl
> From https://github.com/openssl/openssl
> Could not access submodule 'krb5'
> Errors during submodule fetch:
> CryptoPkg/Library/OpensslLib/openssl
> Errors during submodule fetch:
> roms/edk2
>
> Probably harmless, as if one care about this submodule,
> will run "make edk2-basetools" which runs 'git submodule
> update --init --force' and silently fixes the submodules.
>
Did you pass the "--recursive" option to the "git submodule" perhaps?
(Because, krb5 is a submodule of edk2's openssl submodule.)
"--recursive" should not be used. See "ReadMe.rst" in edk2:
"""
Note: When cloning submodule repos, '--recursive' option is not
recommended. EDK II itself will not use any code/feature from
submodules in above submodules. So using '--recursive' adds a
dependency on being able to reach servers we do not actually want
any code from, as well as needlessly downloading code we will not
use.
"""
QEMU's current submodules do not require the usage of "--recursive".
Thanks
Laszlo
- [PATCH 04/10] tests: acpi: tolerate "virt/SSDT.memhp" mismatch temporarily, (continued)
- [PATCH 04/10] tests: acpi: tolerate "virt/SSDT.memhp" mismatch temporarily, Laszlo Ersek, 2020/09/08
- [PATCH 06/10] roms/Makefile.edk2: complete replacing TPM2*_ENABLE macros, Laszlo Ersek, 2020/09/08
- [PATCH 07/10] roms/Makefile.edk2: enable new ARM/AARCH64 flags up to edk2-stable202008, Laszlo Ersek, 2020/09/08
- [PATCH 08/10] pc-bios: refresh edk2 build artifacts for edk2-stable202008, Laszlo Ersek, 2020/09/08
- [PATCH 05/10] roms/edk2: update submodule from edk2-stable201905 to edk2-stable202008, Laszlo Ersek, 2020/09/08
[PATCH 09/10] pc-bios: update the README file with edk2-stable202008 information, Laszlo Ersek, 2020/09/08
[PATCH 10/10] tests: acpi: update "virt/SSDT.memhp" for edk2-stable202008, Laszlo Ersek, 2020/09/08