On 1/14/21 12:45 PM, Philippe Mathieu-Daudé wrote:
The --enable-bzip2/--disable-bzip2 configure arguments are
somehow misleading, they check for the bzip2 library, not
the bzip2 program.


Well. I was mislead, but they're fine, really.

We need the bzip2 program to install the EDK2 firmware blobs
(see commit 623ef637a2e "configure: Check bzip2 is available").

Check if the bzip2 program in the global meson.build to avoid
the configuration to succeed, but a later when trying to install
the firmware blobs:

     ../pc-bios/meson.build:5:2: ERROR: Program 'bzip2' not found

Reported-by: John Snow <js...@redhat.com>
Suggested-by: Paolo Bonzini <pbonz...@redhat.com>
Fixes: c8d5450bba3 ("configure: move install_blobs from configure to meson")
Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com>
---
  meson.build         | 2 ++
  pc-bios/meson.build | 1 -
  2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/meson.build b/meson.build
index ecc45d04d6a..954152c90fe 100644
--- a/meson.build
+++ b/meson.build
@@ -96,6 +96,8 @@
    endforeach
  endif
+bzip2 = find_program('bzip2', required: install_edk2_blobs)
+

Seems good though :)

  ##################
  # Compiler flags #
  ##################
diff --git a/pc-bios/meson.build b/pc-bios/meson.build
index 6a341b6cea0..af95c5d1f1d 100644
--- a/pc-bios/meson.build
+++ b/pc-bios/meson.build
@@ -1,5 +1,4 @@
  if install_edk2_blobs
-  bzip2 = find_program('bzip2', required: true)
    fds = [
      'edk2-aarch64-code.fd',
      'edk2-arm-code.fd',



Reply via email to