From 932a958672b1239d2fb872573f723aa99bf8991b Mon Sep 17 00:00:00 2001 From: "Yann E. MORIN" Date: Wed, 8 Apr 2015 19:08:35 +0200 Subject: [PATCH] infra/pkg-kconfig: require an non-empty KCONFIG_FILE Currently, we only check that the variable is defined, which is not enough since we really want it to be non-empty. We however can't check it points to an existing file, because the package might well not be extracted yet, and we may use an internal defconfig. If that file does not eventually exist, there will be a failure down the road at build time when we try to copy it... Signed-off-by: "Yann E. MORIN" Cc: Thomas Petazzoni Cc: Thomas De Schampheleire Reviewed-by: Arnout Vandecappelle (Essensium/Mind) Signed-off-by: Thomas Petazzoni --- package/pkg-kconfig.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package/pkg-kconfig.mk b/package/pkg-kconfig.mk index dcaed53dd2..7a00fff393 100644 --- a/package/pkg-kconfig.mk +++ b/package/pkg-kconfig.mk @@ -70,7 +70,7 @@ $$($(2)_TARGET_CONFIGURE): $$($(2)_DIR)/.stamp_kconfig_fixup_done ifeq ($$($$($(2)_KCONFIG_VAR)),y) # FOO_KCONFIG_FILE is required -ifndef $(2)_KCONFIG_FILE +ifeq ($$($(2)_KCONFIG_FILE),) $$(error Internal error: no value specified for $(2)_KCONFIG_FILE) endif -- 2.30.2