Use RUNONCHANGE mechanism to restart or reload imapd:

cyrus.conf.template: openpkg rc imapd reload (see man cyrmaster)
imapd.conf.template: openpkg rc imapd restart (reload and reconnect was not
 enough when I tested changing unixhierarchysep and annotation_definitions)
imapd.annotation_definitions.template: openpkg rc imapd restart (reload and
 reconnect was not enough)
imapd.group.template: nothing (a simple reconnect was enough, this is the old
 behaviour and changing this to restart to force new permissions would be
 overkill since then all users would be disconnected)

1 files changed, 0 insertions, 8 deletions

diff --git a/lib/Kolab/Conf.pm b/lib/Kolab/Conf.pm
index 7d902fe..3491795 100644
--- a/lib/Kolab/Conf.pm
+++ b/lib/Kolab/Conf.pm
@@ -288,8 +288,6 @@ sub build {
         if ($rc) {
             if ($cfg =~ /openldap/) {
                 $haschanged{'slapd'} = 1;
-            } elsif ($cfg =~ /imapd/) {
-                $haschanged{'imapd'} = 1;
             }
             $templatehaschanged{$tmpl} = 1;
 
@@ -814,12 +812,6 @@ sub reload
         system("$Kolab::config{'KOLABRC'} rc openldap restart &");
     }
 
-    if ($haschanged{'imapd'}) {
-        &Kolab::log('K', 'Restarting imapd...');
-	# Would it be enough with a reload here? /steffen
-        system("$Kolab::config{'KOLABRC'} rc imapd restart");
-    }
-
     %Kolab::Conf::haschanged = ();
 
     &Kolab::log('K', 'Reload finished');