From: Yann E. MORIN Date: Thu, 15 Jul 2021 21:29:17 +0000 (+0200) Subject: boot/uboot: add missing dependency to host-pkgconf X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=d0edfec1e27b7b1b1cb46221e71cae1537f58440;p=buildroot.git boot/uboot: add missing dependency to host-pkgconf Commit 2eaa6d0f3629 (boot/uboot: fix uboot building host tools on x86 architecture) added use of $(PKG_CONFIG_HOST_BINARY), but forgot to add the corresponding build-ordr dependency. Add this missing depenency now. Additionally, the associated test had an explicit host pkgconf enbled in its configuration. This is superfluous now that uboot properly depends on host-pkgconf, so drop that from the test. Note: it hapenned to work, because host-pkgconf, when explicitly enabled in the configuration, and without per-package directories, would build before uboot and thus be available. This would fail with PPD, though, and thus would break for TLPB. Reported-by: Peter Korsgaard Signed-off-by: Yann E. MORIN Cc: Kory Maincent Signed-off-by: Peter Korsgaard --- diff --git a/boot/uboot/uboot.mk b/boot/uboot/uboot.mk index a7eaedb06f..a8a8bedc01 100644 --- a/boot/uboot/uboot.mk +++ b/boot/uboot/uboot.mk @@ -17,7 +17,7 @@ UBOOT_CPE_ID_PRODUCT = u-boot UBOOT_INSTALL_IMAGES = YES # u-boot 2020.01+ needs make 4.0+ -UBOOT_DEPENDENCIES = $(BR2_MAKE_HOST_DEPENDENCY) +UBOOT_DEPENDENCIES = host-pkgconf $(BR2_MAKE_HOST_DEPENDENCY) UBOOT_MAKE = $(BR2_MAKE) ifeq ($(UBOOT_VERSION),custom) diff --git a/support/testing/tests/package/test_uboot_openssl_pkgconfig.py b/support/testing/tests/package/test_uboot_openssl_pkgconfig.py index de2d83f5b8..55a45f44fc 100644 --- a/support/testing/tests/package/test_uboot_openssl_pkgconfig.py +++ b/support/testing/tests/package/test_uboot_openssl_pkgconfig.py @@ -12,7 +12,6 @@ class TestUbootOpensslPkgConfig(infra.basetest.BRTest): BR2_TARGET_UBOOT=y BR2_TARGET_UBOOT_BOARD_DEFCONFIG="efi-x86_payload64" BR2_TARGET_UBOOT_NEEDS_OPENSSL=y - BR2_PACKAGE_HOST_PKGCONF=y """ def test_run(self):