From patchwork Fri Sep 1 14:23:31 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Bruce Richardson X-Patchwork-Id: 131096 X-Patchwork-Delegate: david.marchand@redhat.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id AB44A42278; Fri, 1 Sep 2023 16:23:43 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A685D4029B; Fri, 1 Sep 2023 16:23:43 +0200 (CEST) Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.100]) by mails.dpdk.org (Postfix) with ESMTP id 2F1FE40285 for ; Fri, 1 Sep 2023 16:23:42 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1693578222; x=1725114222; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=QgjfzsTMOcSiOckYs0liWgiC8HNJ7shz+FZG37/RncY=; b=KPHAyJz0rlaWao8x7eYYkEtsYcJKuXsbAHf7azU/20WXJFKOOPFbULFK wcSsnzv46krHjy17kwLayV0jBsOPrqpPEbTT5upXC3mdx+EzImtBErqNR RdyjbK0xG77MgOeDp9dtaeQ8qSi8jlW7grmZzOACabK5s/xwW098XJP1F KCFS6hnDAkr/jnyaGPaQm39RG2ptSYSXu+1jvm471hBqmloqDOW78v78m 43uQVXmaAHg2M0kw77yYV3hjzrqOp3UqiTGHFoLfRHakI4F1W7e/9Sbni oTXFadTHK9xubn6MrLPoKFWxQCtfcw6zTGBouctaap1Vodu6DkkTgvnsq g==; X-IronPort-AV: E=McAfee;i="6600,9927,10819"; a="442624643" X-IronPort-AV: E=Sophos;i="6.02,219,1688454000"; d="scan'208";a="442624643" Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 01 Sep 2023 07:23:40 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10819"; a="883199261" X-IronPort-AV: E=Sophos;i="6.02,219,1688454000"; d="scan'208";a="883199261" Received: from silpixa00401385.ir.intel.com ([10.237.214.152]) by fmsmga001.fm.intel.com with ESMTP; 01 Sep 2023 07:23:42 -0700 From: Bruce Richardson To: dev@dpdk.org Cc: Bruce Richardson Subject: [PATCH 1/2] build: fail if explicitly requested lib is unbuildable Date: Fri, 1 Sep 2023 15:23:31 +0100 Message-Id: <20230901142332.588856-1-bruce.richardson@intel.com> X-Mailer: git-send-email 2.39.2 MIME-Version: 1.0 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org When the user passes a list of desired libraries to build via the "enable_libs" option, the expectation is that those libraries should be part of the build. However, if those libs have either external or internal dependencies, they still may be silently disabled, for example: running "meson setup -Denable_libs=security build" will successfully run, but the security lib will not be configured as "cryptodev" is missing. We can fix this by setting a flag to indicate when the libraries are specified via an enable_libs flag. If so, then we error out when a library is unbuildable, giving a suitable error message. For the above example case, the "meson setup" run fails with: Message: Disabling security [lib/security]: missing internal dependency "cryptodev" lib/meson.build:218:16: ERROR: Problem encountered: Cannot build explicitly requested lib "security". Please add missing dependency "cryptodev" to "enable_libs" option Signed-off-by: Bruce Richardson Acked-by: Morten Brørup Acked-by: David Marchand --- lib/meson.build | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/lib/meson.build b/lib/meson.build index 099b0ed18a..cf4aa63630 100644 --- a/lib/meson.build +++ b/lib/meson.build @@ -131,7 +131,9 @@ endforeach disable_libs = run_command(list_dir_globs, get_option('disable_libs'), check: true).stdout().split() enable_libs = run_command(list_dir_globs, get_option('enable_libs'), check: true).stdout().split() +require_libs = true if enable_libs.length() == 0 + require_libs = false enable_libs += optional_libs endif enable_libs += always_enable @@ -189,6 +191,10 @@ foreach l:libraries if build subdir(l) + if not build and require_libs + error('Cannot build explicitly requested lib "@0@".\n'.format(name) + +'\tReason: ' + reason) + endif endif if name != l warning('Library name, "@0@", and directory name, "@1@", do not match'.format(name, l)) @@ -208,6 +214,10 @@ foreach l:libraries endif message('Disabling @1@ [@2@]: missing internal dependency "@0@"' .format(d, name, 'lib/' + l)) + if require_libs + error('Cannot build explicitly requested lib "@0@".\n'.format(name) + + '\tPlease add missing dependency "@0@" to "enable_libs" option'.format(d)) + endif else shared_deps += [get_variable('shared_rte_' + d)] static_deps += [get_variable('static_rte_' + d)]