Re: Questions and remarks on draft-ietf-sieve-include-01.txt

[Stephan Bosch <[e-mail]>]

Hi Aaron,

Aaron Stone wrote:
> I'm posting draft-ietf-managesieve-02 that addresses these comments. Thank
> you!
> 
Ok, looks nice. I'll adjust my implementation as soon as I have some time.

However, as far as I can see, you did not address every issue yet:

- For the global command I would expect text stating that the variables 
extension is required when it is used.

- The scope of the :once modifier could be a bit confusing. I am 
assuming it holds for the whole Sieve execution and not only for the 
identical include commands within the current script. What is the 
interaction with multiscript in this case? I am still not sure what to 
do with this.

- The script examples have various small typos, syntax problems and 
missing require items.

Also, you completely lifted the global command's placement requirements. 
I am not fundamentally opposed to this, but this can make the inclusion 
of a global variable conditional, i.e. inside an if statement. Until I 
actually implement this I won't really know for sure whether there will 
be implementation issues.

And if the possibility for a conditional global command is desired, I 
would expect an explicit mention of this possibility somewhere in the 
command explanation. Now it could come to a surprise to the more naive 
implementer. This also implies that a global variable is not included 
into the local scope until after the first global command that specifies 
it.

Finally, I saw one new typo: Section 4, second paragraph, 'acess'.

Regards,

-- 
Stephan Bosch
[e-mail]