File 0006-LU-16353-config-enable_foo-variables-mustn... of Package lustre_2_15

Overview Repositories Revisions Requests Users Attributes Meta

File 0006-LU-16353-config-enable_foo-variables-mustn-t-contain.patch of Package lustre_2_15

From 97d599d328172861df25b4e867154103749f044d Mon Sep 17 00:00:00 2001
From: Mr NeilBrown <neilb@suse.de>
Date: Wed, 30 Nov 2022 11:47:03 +1100
Subject: [PATCH 06/30] LU-16353 config: enable_foo variables mustn't contains
 space

$enable_crypto is in some circumstances set to "embedded llcrypt"
which contains a space.
When the code from lustre-build.m4 then tests the value with:

if test x$enablecrypto = xyes

we get a syntax error from ./configure

We could add quotes to this comment, but for consistency we would need
to add quotes to ever other test for an enable_foo variable.

It is simpler just to ensure we don't add spaces.  So change the space
to a hyphen.

Test-Parameters: trivial
Signed-off-by: Mr NeilBrown <neilb@suse.de>
Change-Id: I097e857409d6ec48a765ccda1cc470d28b90e601
---
 lustre/autoconf/lustre-core.m4 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lustre/autoconf/lustre-core.m4 b/lustre/autoconf/lustre-core.m4
index b6c4977433f2..35fd86213fa8 100644
--- a/lustre/autoconf/lustre-core.m4
+++ b/lustre/autoconf/lustre-core.m4
@@ -2964,7 +2964,7 @@ AS_IF([test "x$enable_crypto" = xin-kernel], [
 	      AC_DEFINE(HAVE_LUSTRE_CRYPTO, 1, [Enable Lustre client crypto via embedded llcrypt])
 	      AC_DEFINE(CONFIG_LL_ENCRYPTION, 1, [embedded llcrypt])
 	      AC_DEFINE(HAVE_FSCRYPT_DUMMY_CONTEXT_ENABLED, 1, [embedded llcrypt uses llcrypt_dummy_context_enabled()])
-	      enable_crypto="embedded llcrypt"
+	      enable_crypto="embedded-llcrypt"
 	      enable_llcrypt=yes], [
 	      AS_IF([test "x$enable_crypto" = xyes],
 	            [AC_MSG_ERROR([Lustre client crypto cannot be enabled because of lack of encryption support in your kernel.])])
-- 
2.38.1

Places

File 0006-LU-16353-config-enable_foo-variables-mustn-t-contain.patch of Package lustre_2_15

Places