Update main.yml

roles/nodejs/tasks/main.yml

@@ -1,6 +1,8 @@
-# 2019-01-16: duplicate Node.js code unified thanks to @jvonau. It's used by:
+# 2019-01-16/29: duplicate Node.js code unified thanks to @jvonau. It's used by:
-#   roles/sugarizer/tasks/main.yml with roles/sugarizer/meta/main.yml
+#   roles/pbx/tasks/main.yml       with roles/pbx/meta/main.yml (forthcoming for Asterisk with FreePBX)
 #   roles/nodered/tasks/main.yml   with roles/nodered/meta/main.yml
+#   roles/sugarizer/tasks/main.yml with roles/sugarizer/meta/main.yml
+
 
 # 2019-01-16: fyi Node.js 10.x became "LTS" on 2018-10-30 but distros are
 # holding back for now: certainly Ubuntu 18.04 and even Debian 10/Buster
@@ -8,23 +10,26 @@
 # more bleeding-edge Debian Sid ("unstable" branch) does install Node.js 10.x
 #
 # This May Change: thanks all for running "apt -a list nodejs" on Buster's
-# daily builds @ www.debian.org/devel/debian-installer/ to keep us informed!
+# daily builds @ www.debian.org/devel/debian-installer/ and Disco Dingo (Ubuntu
+# 19.04) https://launchpad.net/ubuntu/+source/nodejs to keep us informed!
 
-- name: Set up Node.js 8.x apt sources (debuntu distros UP TO 2017)
+- name: Set up Node.js {{ nodejs_version }} apt sources (debuntu distros UP TO 2017)
-  shell: curl -sL https://deb.nodesource.com/setup_8.x | bash -
+  shell: curl -sL https://deb.nodesource.com/setup_{{ nodejs_version }} | bash -
   args:
     warn: no
-  when: internet_available and (is_debian_8 or is_debian_9 or is_ubuntu_16 or is_ubuntu_17)
+  when: internet_available
+  #when: internet_available and (is_debian_8 or is_debian_9 or is_ubuntu_16 or is_ubuntu_17)
   # NOT NEC TO TEST FOR is_raspbian_8 OR is_raspbian_9 AS /opt/iiab/iiab/vars/<OS>.yml
   # DEFINES THESE AS SUBSETS OF is_debian_8 OR is_debian_9 (FOR NOW!)
 
-- name: Install latest Node.js which includes /usr/bin/npm (debuntu distros UP TO 2017)
+- name: Install Node.js {{ nodejs_version }} which includes /usr/bin/npm (debuntu distros UP TO 2017)
   package:
-    name: nodejs
+    name: nodejs={{ nodejs_version }}    # Nec to change above from 'package:' to 'apt:' ?
-    # name: nodejs=8.x
+    # name: nodejs
     state: latest
     # state: present
-  when: internet_available and (is_debian_8 or is_debian_9 or is_ubuntu_16 or is_ubuntu_17)
+  when: internet_available
+  #when: internet_available and (is_debian_8 or is_debian_9 or is_ubuntu_16 or is_ubuntu_17)
 
 # 2018-07-14: BOTH STEPS ABOVE TAKE TIME, but Raspbian (apt offers npm
 # 1.4.21) & Debian 9 (apt offers no npm!) STILL NEED the above
@@ -45,10 +50,10 @@
 # where it upgrades /usr/bin/npm in place:
 # https://askubuntu.com/questions/1036278/npm-is-incorrect-version-on-latest-ubuntu-18-04-installation
 
-- name: Install latest packages nodejs and npm (debuntu distros AFTER 2017, or other distros)
+# - name: Install packages nodejs {{ nodejs_version }} and npm (debuntu distros AFTER 2017, or other distros)
-  package:
+#   package:
-    name:
+#     name:
-      - nodejs
+#       - nodejs={{ nodejs_version }}    # Nec to change above from 'package:' to 'apt:' ?
-      - npm
+#       - npm
-    state: latest
+#     state: latest
-  when: internet_available and not (is_debian_8 or is_debian_9 or is_ubuntu_16 or is_ubuntu_17)
+#   when: internet_available and not (is_debian_8 or is_debian_9 or is_ubuntu_16 or is_ubuntu_17)