[Qemu-devel] [PATCH 0/2] Fix tarball builds of UEFI/EDK2 firmware

Michael Roth posted 2 patches 6 years, 2 months ago
Test docker-clang@ubuntu passed
Test FreeBSD passed
Test checkpatch passed
Test docker-quick@centos7 passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190912231202.12327-1-mdroth@linux.vnet.ibm.com
Maintainers: "Philippe Mathieu-Daudé" <philmd@redhat.com>, Laszlo Ersek <lersek@redhat.com>
roms/Makefile.edk2   | 7 ++++++-
scripts/make-release | 8 ++++++++
2 files changed, 14 insertions(+), 1 deletion(-)
[Qemu-devel] [PATCH 0/2] Fix tarball builds of UEFI/EDK2 firmware
Posted by Michael Roth 6 years, 2 months ago
Bruce noticed that we cannot build `make efi` target from the v4.1.0
tarball. This is due to a failure on the part of the make-release script
to pull in submodules nested under other submodules, as well as
Makefile.edk2's assumptions about being in a git tree.

Suggestions for more robust solutions are definitely welcome, but for
now this series takes what seems to be the most direct approach.

 roms/Makefile.edk2   | 7 ++++++-
 scripts/make-release | 8 ++++++++
 2 files changed, 14 insertions(+), 1 deletion(-)



Re: [Qemu-devel] [PATCH 0/2] Fix tarball builds of UEFI/EDK2 firmware
Posted by Philippe Mathieu-Daudé 6 years, 1 month ago
On 9/13/19 1:12 AM, Michael Roth wrote:
> Bruce noticed that we cannot build `make efi` target from the v4.1.0
> tarball. This is due to a failure on the part of the make-release script
> to pull in submodules nested under other submodules, as well as
> Makefile.edk2's assumptions about being in a git tree.

Hmm I'd expect distributions interested in building EDK2, build it
properly tuned, outside of QEMU. I consider what is here as default
config useful for testing, not for production use. I might be wrong.

Meanwhile, it is true 'make efi' should not fail.

> Suggestions for more robust solutions are definitely welcome, but for
> now this series takes what seems to be the most direct approach.

This is fine:
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

I'll queue these patches in my edk2-next branch since I'm preparing a
pull request.

Thanks,

Phil.