From 2fcf65a7444ae52a38cd1cd9e5d0ea1b72e1ce1e Mon Sep 17 00:00:00 2001
From: Stephan Bosch <stephan@rename-it.nl>
Date: Tue, 4 Aug 2009 17:13:50 +0200
Subject: [PATCH] Include: removed variables-related FIXME and substituted a
 permanent error message.

---
 src/lib-sieve/plugins/include/cmd-include.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/lib-sieve/plugins/include/cmd-include.c b/src/lib-sieve/plugins/include/cmd-include.c
index 8ee6a3309..b8a506d28 100644
--- a/src/lib-sieve/plugins/include/cmd-include.c
+++ b/src/lib-sieve/plugins/include/cmd-include.c
@@ -216,13 +216,12 @@ static bool cmd_include_validate(struct sieve_validator *validator,
 	if ( !sieve_validator_argument_activate(validator, cmd, arg, FALSE) )
 		return FALSE;
 
-	/* FIXME: We can currently only handle string literal argument, so
-	 * variables are not allowed.
+	/* 
+	 * Variables are not allowed.
 	 */
 	if ( !sieve_argument_is_string_literal(arg) ) {
 		sieve_argument_validate_error(validator, arg, 
-			"this Sieve implementation currently only supports "
-			"a literal string argument for the include command");
+			"the include command requires a constant string for its value argument");
 		return FALSE;
 	}
 
-- 
GitLab