[PATCHv3 1/4] config2header: Replace invalid characters

2012-05-21 Thread Philipp Hahn
The sieve extension 'vacation-seconds' contains a dash, which is not an allowed character for C identifiers [A-Z_a-z][0-9A-Z_a-z]*. Replace all other characters by _ when generating IMAP_ENUMs. Signed-off-by: Philipp Hahn --- tools/config2header |7 ++- 1 files changed, 6 insertions(+),

[PATCHv3 0/4] Add VACATION :seconds support

2012-05-21 Thread Philipp Hahn
Hello, some time ago I implemented RFC 6131 , which adds a variante of the VACATION extension supporting seconds instead of days. The original patch was for version 2.2.x and implemented a draft version. I introduced a new OP-code, which uses seconds instead of

[PATCHv3 2/4] Change vacation day to seconds

2012-05-21 Thread Philipp Hahn
In preparation for RFC6131 (draft-ietf-sieve-vacation-seconds) change the vacation time from days to seconds. Signed-off-by: Philipp Hahn --- imap/lmtp_sieve.c |6 +++--- lib/times.h |3 +++ sieve/bc_dump.c |4 +++- sieve/bc_emit.c |4 ++-- sieve

[PATCHv3 3/4] Add VACATION :seconds support

2012-05-21 Thread Philipp Hahn
Add support for specifying the vacation time in seconds in addition to days. Signed-off-by: Philipp Hahn --- doc/specs.html |2 ++ lib/imapoptions|2 +- sieve/README |3 +++

[PATCHv3 4/4] Make min/max VACACTION response configurable

2012-05-21 Thread Philipp Hahn
Add imapoptions to change the minimum and maximum allowed time interval between consecutive vacation responses. The defaults are still 3 and 90 days, which are too high for the vacation-seconds extension. Blindly lowering the limit might be problematic for high traffic sites. Signed-off-by: Philip

Re: Add VACATION :seconds support

2012-05-21 Thread Дилян Палаузов
Hello Philipp, it is great, that you bring this further. I have done the following changes to your code and uploaded it at http://git.cyrusimap.org/cyrus-imapd, branch dev/sieve/vacation-seconds. If there are no objections within two weeks, I will merge it in master. If there are no object