From bd9b8d57318420db58d9042e88d08a24a8b7691b Mon Sep 17 00:00:00 2001
From: Silvio Rhatto <rhatto@riseup.net>
Date: Mon, 30 Aug 2010 15:09:23 -0300
Subject: Fixing mydestination and adding sympa disable classes

---
 manifests/sympa.pp  | 29 +++++++++++++++++++++++++++++
 manifests/system.pp | 17 ++++++++++-------
 manifests/web.pp    |  6 ++++++
 3 files changed, 45 insertions(+), 7 deletions(-)

diff --git a/manifests/sympa.pp b/manifests/sympa.pp
index e526f7a..a9c45d3 100644
--- a/manifests/sympa.pp
+++ b/manifests/sympa.pp
@@ -103,3 +103,32 @@ class mail::sympa inherits mail::regexps {
     notify    => Exec["newaliases"],
   }
 }
+
+class mail::sympa::disabled inherits mail::sympa {
+  include mail::web::sympa::disabled
+
+  Service["sympa"] {
+    ensure    => stopped,
+    enable    => false,
+  }
+
+  File["/etc/postfix/transport_regexp"] {
+    content => undef,
+  }
+
+  File["/etc/postfix/virtual_regexp"] {
+    content => undef,
+  }
+
+  Mailalias["sympa"] {
+    ensure => absent,
+  }
+
+  Mailalias["sympa-request"] {
+    ensure => absent,
+  }
+
+  Mailalias["sympa-owner"] {
+    ensure => absent,
+  }
+}
diff --git a/manifests/system.pp b/manifests/system.pp
index 5207f4d..35cb2b9 100644
--- a/manifests/system.pp
+++ b/manifests/system.pp
@@ -10,6 +10,10 @@ class mail::system {
     '': { $postfix_mynetworks = "127.0.0.0/8" }
   }
 
+  case $sympa_subdomain {
+    '': { $sympa_subdomain = "lists" }
+  }
+
   # Module requirements
   include postfix
   include ssl::mail
@@ -34,13 +38,12 @@ class mail::system {
   # Sympa mailing list manager
   case $mail_sympa {
     true: {
-      case $sympa_subdomain {
-        '': { $sympa_subdomain = "lists" }
-      }
-
-      $sympa_mydestination = "${sympa_subdomain}.${domain}"
+      $sympa_mydestination = ", ${sympa_subdomain}.${domain}"
       include mail::sympa
     }
+    'disabled': {
+      include mail::sympa::disabled
+    }
     default: {
       include mail::regexps
     }
@@ -53,13 +56,13 @@ class mail::system {
         '': { $schleuder_subdomain = "encrypted" }
       }
 
-      $schleuder_mydestination = "${schleuder_subdomain}.${domain}"
+      $schleuder_mydestination = ", ${schleuder_subdomain}.${domain}"
       include mail::schleuder
     }
   }
 
   case $postfix_mydestination {
-    '': { $postfix_mydestination = "${postfix_my_defaultdestination} ${sympa_mydestination} ${schleuder_mydestination}" }
+    '': { $postfix_mydestination = "${postfix_my_defaultdestination}${sympa_mydestination}${schleuder_mydestination}" }
   }
 
   # Include base configuration
diff --git a/manifests/web.pp b/manifests/web.pp
index 73db06c..208381a 100644
--- a/manifests/web.pp
+++ b/manifests/web.pp
@@ -52,3 +52,9 @@ class mail::web::sympa inherits websites::setup {
     require => File["${apache_www_folder}/${sympa_subdomain}"],
   }
 }
+
+class mail::web::sympa::disabled inherits mail::web::sympa {
+  Apache::site["$sympa_subdomain"] {
+    ensure => absent,
+  }
+}
-- 
cgit v1.2.3