Switch to jinja template, fixes issue #2

syslog_ng/config.sls

@@ -7,7 +7,7 @@ syslog_ng.conf:
   file.managed:
     - name: {{ syslog_ng.syslog_ng_config }}
     - source: {{ syslog_ng.syslog_ng_config_src }}
-    - template: mako
+    - template: jinja
     - user: root
     - group: root
     - mode: 644

syslog_ng/files/syslog-ng.conf

@@ -1,56 +1,62 @@
-<% syslog_config = pillar.get('syslog_ng', {}) %>\
-<%!
-def rule_builder(rule):
-  if rule is None:
-    return ''
-  elif type(rule) is bool:
-    return {True: 'yes', False: 'no'}[rule]
-  elif type(rule) is int:
-    return rule
-  elif type(rule) is list:
-    return ' '.join([rule_builder(v) for v in rule])
-  elif type(rule) is dict:
-    return ' '.join([ '%s(%s)' % (k,rule_builder(v)) for k, v in rule.items()])
-  else:
-    if rule[0] == '=':
-      return rule[1:]
-    return '"%s"' % rule
-%>\
-
-@version: ${ syslog_config.get('version', '3.3') }
-
-% for module in syslog_config.get('module', []):
-@module ${ rule_builder(module) }
-% endfor
-
-% for inc in syslog_config.get('include', []):
-@include ${ rule_builder(inc) }
-% endfor
-
+{%- set syslog_config = pillar.get('syslog_ng', {}) -%}
+
+{%- macro rule_builder(rule) -%}
+    {%- if rule is none -%}
+    {%- elif rule is sameas true -%}
+yes
+    {%- elif rule is sameas false -%}
+no
+    {%- elif rule is number -%}
+{{ rule }}
+    {%- elif rule is mapping -%}
+        {%- for k, v in rule.items() -%}
+{{ k }}({{ rule_builder(v) }})
+        {%- endfor -%}
+    {%- elif rule is iterable and rule is not string -%}
+        {%- for v in rule -%}
+{{ rule_builder(v) }}
+        {%- endfor -%}
+    {%- else -%}
+        {%- if rule[0] == '=' -%}
+{{ rule[1:] }}
+        {%- else -%}
+"{{ rule }}"
+        {%- endif -%}
+    {%- endif -%}
+{%- endmacro -%}
+
+@version: {{ syslog_config.get('version', '3.3') }}
+
+{%- for module in syslog_config.get('module', []) %}
+@module {{ rule_builder(module) }}
+{%- endfor %}
+
+{%- for inc in syslog_config.get('include', []) %}
+@include {{ rule_builder(inc) }}
+{%- endfor %}
 
 options {
-% for rule in syslog_config.get('options', []):
-        ${ rule_builder(rule) };
-% endfor
+{%- for rule in syslog_config.get('options', []) %}
+        {{ rule_builder(rule) }};
+{%- endfor %}
 };
 
-% for obj in ('source', 'destination', 'filter', 'parser', 'rewrite', 'template'): 
-  % for params in syslog_config.get(obj, []):
-    % for name, rules in params.items():
-${ obj } ${ name } {
-      % for rule in rules:
-        ${ rule_builder(rule) };
-      % endfor
+{%- for obj in ('source', 'destination', 'filter', 'parser', 'rewrite', 'template') %}
+{%- for params in syslog_config.get(obj, []) %}
+{% for name, rules in params.items() %}
+{{ obj }} {{ name }} {
+      {%- for rule in rules %}
+        {{ rule_builder(rule) }};
+      {%- endfor %}
 };
+{%- endfor %}
+{%- endfor %}
+{%- endfor %}
 
-    % endfor
-  % endfor
-
-% endfor
-% for loggers in syslog_config.get('log', []):
+{% for loggers in syslog_config.get('log', []) -%}
 log {
-  % for rule in loggers:
-        ${ rule_builder(rule) };
-  % endfor
+  {%- for rule in loggers %}
+        {{ rule_builder(rule) }};
+  {%- endfor %}
 };
-% endfor
+{%- endfor %}