Initializing multiple Sieve instances would hang.

Used errno instead of error variable.

Made normal compile error an info log item in the administrator log.

Forgot to increment count for the clone.

Setting skip_newline is useless when the connection is transfered to another process after successful login.

If a duplicate is detected the implicit keep is canceled, as though the redirect was successful. This prevents getting local deliveries.
The original SMTP recipient is used when it is available to augment the entry in the LDA duplicate database. This way, duplicates are only detected when (initially) addressed to the same recipient.
The main goal of the duplicate detection is mail loop prevention, but this also has the effect that mere duplicate deliveries are handled specially whithout a good reason. We should fix this in a future version.

Added LIBDOVECOT_STORAGE to library dependencies in src/lib-sieve/Makefile.am.

lib-sieve: vacation extension: Fixed determination of From: address for when sieve_vacation_dont_check_recipient is active.

lib-sieve: vnd.dovecot.duplicate extension: Only track duplicate when Sieve script executes successfully.

Forgot to initialize variable, causing a segfault at runtime.

It is now possible to track duplicates based on arbitrary headers or even arbitrary string values using the new :header and :value arguments respectively.
The experation time can be configured using the new :seconds argument.
This change is backwards compatible as long as the name argument wasn't used. This is now a :handle <handle> argument.

Forgot to skip line upon error.

Added means to log user errors/warnings as info in master log.
Previous change was inadequate because an error was still logged.

Prevents administrator frustration about useless log messages caused by users with too much mail.

Change backported from Pigeonhole v0.4 tree, which means that this change will duplicate there upon merge.

This was broken by the ld.gold fix.