From 4034168e71770448356e77d12586a1170e48573e Mon Sep 17 00:00:00 2001 From: Clark Boylan Date: Mon, 13 Feb 2017 17:44:43 -0800 Subject: [PATCH] Don't use npm list --parseable Turns out we lose version info with --parseable (so not sure in what world this is parseable output) so stop using the --parseable flag. This corrects behavior where we reinstall ethercalc every puppet run which can upgrade unpinned dep libs in ways that break them on older node (we are using the ethercalc prescribed node but its a little old). Change-Id: Ib9aebecf3a0c36c50e5069b1a871886e4ddb1a42 --- manifests/init.pp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/manifests/init.pp b/manifests/init.pp index 8241a48..b72fb72 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -69,7 +69,7 @@ class ethercalc ( exec { 'install-ethercalc': command => "npm install ethercalc@${ethercalc_version}", - unless => "npm ls --parseable | grep ethercalc@${ethercalc_version}", + unless => "npm ls | grep ethercalc@${ethercalc_version}", path => $path, cwd => $base_install_dir, require => Anchor['nodejs-anchor'],