zcache - trivial tweak to build-time error message wording

zcache - trivial tweak to build-time error message wording

[Valdis.Kletnieks]

Clarify that the requirement is "at least one of" - the error message could
potentially be read as both being required.

Yes, the line is over 80 chars.  You can't get 79 chars of text and a #error
into 80 chars :)

Patch against linux-next-20110831

Signed-off-by: Valdis Kletnieks <valdis.kletnieks@vt.edu>

--- linux-next/drivers/staging/zcache/zcache-main.c.dist	2011-08-31 13:31:27.054696837 -0400
+++ linux-next/drivers/staging/zcache/zcache-main.c	2011-08-31 16:16:46.055659964 -0400
@@ -34,7 +34,7 @@
 #include "../zram/xvmalloc.h" /* if built in drivers/staging */
 
 #if (!defined(CONFIG_CLEANCACHE) && !defined(CONFIG_FRONTSWAP))
-#error "zcache is useless without CONFIG_CLEANCACHE or CONFIG_FRONTSWAP"
+#error "zcache is useless without at least one of CONFIG_CLEANCACHE or CONFIG_FRONTSWAP"
 #endif
 #ifdef CONFIG_CLEANCACHE
 #include <linux/cleancache.h>

RE: zcache - trivial tweak to build-time error message wording

[Dan Magenheimer]

> From: Valdis.Kletnieks@vt.edu [mailto:Valdis.Kletnieks@vt.edu]
> Sent: Wednesday, August 31, 2011 2:23 PM
> To: Dan Magenheimer
> Cc: linux-kernel@vger.kernel.org
> Subject: zcache - trivial tweak to build-time error message wording
> 
> Clarify that the requirement is "at least one of" - the error message could
> potentially be read as both being required.
> 
> Yes, the line is over 80 chars.  You can't get 79 chars of text and a #error
> into 80 chars :)
> 
> Patch against linux-next-20110831
> 
> Signed-off-by: Valdis Kletnieks <valdis.kletnieks@vt.edu>
> 
> --- linux-next/drivers/staging/zcache/zcache-main.c.dist	2011-08-31 13:31:27.054696837 -0400
> +++ linux-next/drivers/staging/zcache/zcache-main.c	2011-08-31 16:16:46.055659964 -0400
> @@ -34,7 +34,7 @@
>  #include "../zram/xvmalloc.h" /* if built in drivers/staging */
> 
>  #if (!defined(CONFIG_CLEANCACHE) && !defined(CONFIG_FRONTSWAP))
> -#error "zcache is useless without CONFIG_CLEANCACHE or CONFIG_FRONTSWAP"
> +#error "zcache is useless without at least one of CONFIG_CLEANCACHE or CONFIG_FRONTSWAP"

Actually, if the reader interprets that both are required,
that's fine with me.  Cleancache and frontswap were split
up for upstreaming purposes but, except in unusual circumstances,
should always be used together.  That is difficult, of course,
if frontswap is not yet merged in your build tree, but after
frontswap is merged in Linus's tree, I might be inclined to
require both CONFIG_CLEANCACHE *and* CONFIG_FRONTSWAP in
the future Kconfig for zcache.