From 36b00e6332d075c0e298542daed4418c7555b112 Mon Sep 17 00:00:00 2001 From: Ahmad Sherif Date: Fri, 30 May 2014 19:24:47 +0300 Subject: [PATCH 1/3] Remove useless if statement in source.sls --- nginx/source.sls | 4 ---- 1 file changed, 4 deletions(-) diff --git a/nginx/source.sls b/nginx/source.sls index 9428ad7..bacba90 100644 --- a/nginx/source.sls +++ b/nginx/source.sls @@ -79,10 +79,6 @@ get-nginx-{{name}}: - cmd: make-nginx {% endfor -%} -{% if install_luajit -%} - -{% endif -%} - get-ngx_devel_kit: file.managed: - name: {{ source }}/ngx_devel_kit.tar.gz From c28cac34839819b467779737e22cb37282bfd322 Mon Sep 17 00:00:00 2001 From: Ahmad Sherif Date: Fri, 30 May 2014 19:28:59 +0300 Subject: [PATCH 2/3] Group source.sls include declarations into one --- nginx/source.sls | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/nginx/source.sls b/nginx/source.sls index bacba90..17b030a 100644 --- a/nginx/source.sls +++ b/nginx/source.sls @@ -1,6 +1,3 @@ -include: - - nginx.common - {% set nginx = pillar.get('nginx', {}) -%} {% set version = nginx.get('version', '1.5.2') -%} {% set checksum = nginx.get('checksum', 'sha1=3546be28a72251f8823ab6be6a1180d300d06f76') -%} @@ -11,13 +8,12 @@ include: {% set nginx_home = home + "/nginx-" + version -%} {% set nginx_modules_dir = source + "/nginx-modules" -%} -{% if nginx['with_luajit'] -%} include: + - nginx.common +{% if nginx['with_luajit'] %} - nginx.luajit2 {% endif -%} - -{% if nginx['with_openresty'] -%} -include: +{% if nginx['with_openresty'] %} - nginx.openresty {% endif -%} From aac18d2505be295ce46b7bb02e7a104fb51b011f Mon Sep 17 00:00:00 2001 From: Ahmad Sherif Date: Fri, 30 May 2014 19:31:00 +0300 Subject: [PATCH 3/3] Correct requisite ID in source.sls --- nginx/source.sls | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/nginx/source.sls b/nginx/source.sls index 17b030a..8995ebc 100644 --- a/nginx/source.sls +++ b/nginx/source.sls @@ -72,7 +72,7 @@ get-nginx-{{name}}: - watch: - file: get-nginx - require_in: - - cmd: make-nginx + - cmd: nginx {% endfor -%} get-ngx_devel_kit: