From e72351a00f68846685b4a502bdd67092359aa3d7 Mon Sep 17 00:00:00 2001
From: Niels Abspoel <aboe76@gmail.com>
Date: Fri, 1 Sep 2017 23:59:43 +0200
Subject: [PATCH] add sury repo for debian machines

---
 php/ng/cli/install.sls | 15 +++++++++++++++
 php/ng/installed.jinja | 36 +++++++++++++++++++++++++-----------
 2 files changed, 40 insertions(+), 11 deletions(-)

diff --git a/php/ng/cli/install.sls b/php/ng/cli/install.sls
index c52a6ef..ee4d463 100644
--- a/php/ng/cli/install.sls
+++ b/php/ng/cli/install.sls
@@ -1,2 +1,17 @@
 {% set state = 'cli' %}
 {% include "php/ng/installed.jinja" %}
+
+{%- if salt['grains.get']('os_family') == "Debian" %}
+{% set current_php = salt['alternatives.show_current']('php') %}
+{% set phpng_version = salt['pillar.get']('php:ng:version', '7.0')|string %}
+
+php_{{ phpng_version }}_link:
+  alternatives.set:
+    - name: php
+    - path: /usr/bin/php{{ phpng_version }}
+    - require_in:
+      - pkg: php_install_{{ state }}
+    - onlyif:
+      - which php
+      - test {{ current_php }} != $(which php{{ phpng_version }})
+{% endif %}
diff --git a/php/ng/installed.jinja b/php/ng/installed.jinja
index 853e6b6..5088605 100644
--- a/php/ng/installed.jinja
+++ b/php/ng/installed.jinja
@@ -6,7 +6,6 @@
 
 
 {% set pkginfo = php.lookup.pkgs.get(state) %}
-{% set phpng_version = salt['pillar.get']('php:ng:version', '7.0')|string %}
 
 {% set pkgs = [] %}
 {% set specials = [] %}
@@ -27,9 +26,8 @@
   {% set use_external_repo = salt['pillar.get']('php:use_external_repo', False) %}
 
   {% if use_external_repo %}
-    {% set external_repo_name = salt['pillar.get']('php:external_repo_name', 'ondrej/php') %}
-    {% set current_php = salt['alternatives.show_current']('php') %}
-
+    {% if grains['os'] == 'Ubuntu' %}
+      {% set external_repo_name = salt['pillar.get']('php:external_repo_name', 'ondrej/php') %}
 php_ppa_{{ state }}:
   pkgrepo.managed:
     - ppa: {{ external_repo_name }}
@@ -47,16 +45,32 @@ php_ppa_{{ state }}:
     - onchanges:
       - pkgrepo: php_ppa_{{ state }}
 
-php_{{ phpng_version }}_link:
-  alternatives.set:
-    - name: php
-    - path: /usr/bin/php{{ phpng_version }}
+    {% else %}
+      {% set external_repo_name = salt['pillar.get']('php:external_repo_name', 'packages.sury.org/php' ) %}
+php_repo_{{ state }}:
+  pkg.installed:
+    - name: apt-transport-https
+
+php_ppa_{{ state }}:
+  pkgrepo.managed:
+    - humanname: {{ external_repo_name }}
+    - name: deb https://packages.sury.org/php/ {{ grains['oscodename'] }} main
+    - file: /etc/apt/sources.list.d/ondrej-php.list
+    - key_url: https://packages.sury.org/php/apt.gpg
+    - env:
+      - LC_ALL: C.UTF-8
+    - onlyif:
+      - test ! -e /etc/apt/sources.list.d/ondrej-php.list 
     - require_in:
       - pkg: php_install_{{ state }}
-    - onlyif:
-      - which php
-      - test {{ current_php }} != $(which php{{ phpng_version }})
+  pkg.latest:
+    - name: {{ state }}
+    - pkgs: {{ pkgs|json() }}
+    - refresh: True
+    - onchanges:
+      - pkgrepo: php_ppa_{{ state }}
 
+    {% endif %}
   {% endif %}
 {% endif %}