Merge pull request #42 from westurner/fix-nested-_roots

Fix for nested [file|pillar]_roots in minion tmpl (seeAlso: #37)

salt/files/master

 {%- endfor -%}
 {% elif 'pillar_roots' in salt -%}
 pillar_roots:
-{%- for name, roots in salt['pillar_roots'].items() -%}
+{%- for name, roots in salt['pillar_roots']|dictsort %}
   {{ name }}:
-{%- for dir in roots -%}
+{%- for dir in roots %}
     - {{ dir }}
 {%- endfor -%}
 {%- endfor -%}

salt/files/minion

 #
 {% if 'file_roots' in minion -%}
 file_roots:
-  {% for name, roots in minion['file_roots'].items() -%}
+{%- for name, roots in minion['file_roots']|dictsort %}
   {{ name }}:
-    {% for dir in roots -%}
-      - {{ dir }}
-    {% endfor -%}
-  {% endfor -%}
+{%- for dir in roots %}
+    - {{ dir }}
+{%- endfor -%}
+{%- endfor -%}
 {% elif 'file_roots' in salt -%}
 file_roots:
-  {% for name, roots in salt['file_roots'].items() -%}
+{%- for name, roots in salt['file_roots']|dictsort %}
   {{ name }}:
-    {% for dir in roots -%}
-      - {{ dir }}
-    {% endfor -%}
-  {% endfor -%}
+{%- for dir in roots %}
+    - {{ dir }}
+{%- endfor -%}
+{%- endfor -%}
 {% else -%}
 #file_roots:
 #  base:
 # also be configured on the minion:
 {% if 'pillar_roots' in minion -%}
 pillar_roots:
-  {% for name, roots in minion['pillar_roots'].items() -%}
+{%- for name, roots in minion['pillar_roots']|dictsort %}
   {{ name }}:
-    {% for dir in roots -%}
-      - {{ dir }}
-    {% endfor -%}
-  {% endfor -%}
+{%- for dir in roots %}
+    - {{ dir }}
+{%- endfor -%}
+{%- endfor -%}
 {% elif 'pillar_roots' in salt -%}
 pillar_roots:
-  {% for name, roots in salt['pillar_roots'].items() -%}
+{%- for name, roots in salt['pillar_roots']|dictsort %}
   {{ name }}:
-    {% for dir in roots -%}
-      - {{ dir }}
-    {% endfor -%}
-  {% endfor -%}
-{% else %}
+{%- for dir in roots %}
+    - {{ dir }}
+{%- endfor -%}
+{%- endfor -%}
+{% else -%}
 #pillar_roots:
 #  base:
-#    - /srv/pillar
+#    - /srv/salt
 {%- endif %}
 
 ######        Security settings       #####