From 7e1d4ba6a17961124efe4e28fc104f05b7f87241 Mon Sep 17 00:00:00 2001 From: Ben Newman Date: Mon, 12 Jun 2017 16:59:34 -0400 Subject: [PATCH] Don't skip reinstalling unchanged Npm.depends-style dependencies. (#8787) Fixes #8732. Should also fix #8747. --- tools/isobuild/meteor-npm.js | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/tools/isobuild/meteor-npm.js b/tools/isobuild/meteor-npm.js index 2b47f44bcb..4ec6c3086c 100644 --- a/tools/isobuild/meteor-npm.js +++ b/tools/isobuild/meteor-npm.js @@ -646,10 +646,7 @@ var updateExistingNpmDirectory = function (packageName, newPackageNpmDir, // Otherwise install npmTree.dependencies as if we were creating a new // .npm/package directory, and leave preservedShrinkwrap empty. _.each(npmTree.dependencies, (info, name) => { - const installed = minInstalledTree.dependencies[name]; - if (! installed || installed.version !== info.version) { - installNpmModule(name, info.version, newPackageNpmDir); - } + installNpmModule(name, info.version, newPackageNpmDir); }); // Note: as of npm@4.0.0, npm-shrinkwrap.json files are regarded as