From 85cc9f240749575fa2315df8009f796263db7043 Mon Sep 17 00:00:00 2001 From: Ben Newman Date: Mon, 12 Jun 2017 14:44:51 -0400 Subject: [PATCH] Don't skip reinstalling unchanged Npm.depends-style dependencies. 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 e3045a2bc5..03b6c97c23 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