Re: [PATCH 0/4] Add VACATION :seconds support

2012-05-09 Thread Bron Gondwana
On Wed, May 09, 2012 at 12:08:53PM +1000, Greg Banks wrote: On Tue, May 8, 2012, at 01:52 PM, Bron Gondwana wrote: On Tue, May 8, 2012, at 01:50 PM, Philipp Hahn wrote: Hello, Am Dienstag 08 Mai 2012 13:07:04 schrieb Bron Gondwana: On Tue, May 8, 2012, at 11:54 AM, Philipp

Re: Add VACATION :seconds support

2012-05-09 Thread Philipp Hahn
Hello, Am Dienstag 08 Mai 2012 13:47:46 schrieb Дилян Палаузов: Extending sieve_listextensions() probably requires adding vacation-seconds in sieve_extensions in lib/imapoptions. There is one small problem: vacation-seconds contains a dash, which config2header converts verbatim into the

[PATCHv2 0/4] Add VACATION :seconds support

2012-05-09 Thread Philipp Hahn
is 7 days, so the limit is implicitly applied. Please have a look at man/imapd.conf.5, if 3,90 * DAY2SEC is acceptable. Philipp Hahn (4): config2header: Replace invalid characters Change vacation day to seconds Add VACATION :seconds support Make min/max VACACTION response configurable imap

[PATCH 0/4] Add VACATION :seconds support

2012-05-08 Thread Philipp Hahn
for defining this as a macro? Philipp Hahn (4): autobuild: Fix directory handling Change vacation day to seconds Add VACATION :seconds support Support VACACTION :seconds 0 autobuild.sh | 18 ++- imap/lmtp_sieve.c

[PATCH 3/4] Add VACATION :seconds support

2012-05-08 Thread Philipp Hahn
Add support for specifying the vacation time in seconds in addition to days. Signed-off-by: Philipp Hahn h...@univention.de --- sieve/README |3 +++ sieve/script.c | 11 +++ sieve/script.h

Re: [PATCH 0/4] Add VACATION :seconds support

2012-05-08 Thread Bron Gondwana
to seconds Add VACATION :seconds support Support VACACTION :seconds 0 Looks like the output of git. I don't suppose you have a repository online somewhere that you could point to? Cherry-picking from an upstream is even easier than applying patches from emails :) Bron. -- Bron Gondwana br

Re: Add VACATION :seconds support

2012-05-08 Thread Дилян Палаузов
vacation day to seconds Add VACATION :seconds support Support VACACTION :seconds 0 autobuild.sh | 18 ++- imap/lmtp_sieve.c |6 ++-- sieve/README |3 ++ sieve

Re: Add VACATION :seconds support

2012-05-08 Thread Дилян Палаузов
Post scriptum: Also sieve/interp.c:sieve_listextensions(sieve_interp_t *i) has to be fixed, so that it includes vacation-seconds in its return value. The result of sieve_listextensions, including vacation-seconds, is printed by timsieved's CAPABILITY command. Extending

Re: [PATCH 0/4] Add VACATION :seconds support

2012-05-08 Thread Philipp Hahn
. Would the following be fine: #define DAY2SEC(x) ((x) * 24 * 60 * 60) Or is there a more preferrable name? Philipp Hahn (4): autobuild: Fix directory handling Change vacation day to seconds Add VACATION :seconds support Support VACACTION :seconds 0 Looks like the output of git

Re: Add VACATION :seconds support

2012-05-08 Thread Philipp Hahn
Hello, Am Dienstag 08 Mai 2012 13:32:56 schrieb Дилян Палаузов: Hello, To patch 1: autobuild: Fix directory handling Why do you use autobuild.sh instead of autoreconf -vi? autobuild had just the right name and did what I was looking for - until it renamed $PWD to $PWD.orig :-( To patch

Re: [PATCH 0/4] Add VACATION :seconds support

2012-05-08 Thread Bron Gondwana
to seconds Add VACATION :seconds support Support VACACTION :seconds 0 Looks like the output of git. I don't suppose you have a repository online somewhere that you could point to? Cherry-picking from an upstream is even easier than applying patches from emails :) git://github.com

Re: Add VACATION :seconds support

2012-05-08 Thread Bron Gondwana
On Tue, May 8, 2012, at 01:52 PM, Philipp Hahn wrote: Hello, Am Dienstag 08 Mai 2012 13:32:56 schrieb Дилян Палаузов: Hello, To patch 1: autobuild: Fix directory handling Why do you use autobuild.sh instead of autoreconf -vi? autobuild had just the right name and did what I was

Re: Add VACATION :seconds support

2012-05-08 Thread Greg Banks
On Tue, May 8, 2012, at 01:55 PM, Bron Gondwana wrote: On Tue, May 8, 2012, at 01:52 PM, Philipp Hahn wrote: autobuild had just the right name and did what I was looking for - until it renamed $PWD to $PWD.orig :-( gnb - looks like an attractive nuisance. Yeah - naming is hard :(

Re: [PATCH 0/4] Add VACATION :seconds support

2012-05-08 Thread Greg Banks
On Tue, May 8, 2012, at 01:52 PM, Bron Gondwana wrote: On Tue, May 8, 2012, at 01:50 PM, Philipp Hahn wrote: Hello, Am Dienstag 08 Mai 2012 13:07:04 schrieb Bron Gondwana: On Tue, May 8, 2012, at 11:54 AM, Philipp Hahn wrote: For converting days to seconds and the inverse I