From f006d98e8d89e8ecefce70ed25ea5ee37d8fc1c2 Mon Sep 17 00:00:00 2001
From: Ben Newman <ben@meteor.com>
Date: Wed, 17 Aug 2016 10:55:52 -0400
Subject: [PATCH] Filter --all-packages package names through
 isIsobuildFeaturePackage.

---
 tools/cli/commands-packages.js | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/tools/cli/commands-packages.js b/tools/cli/commands-packages.js
index a1eb7abce8..31d0f54c9b 100644
--- a/tools/cli/commands-packages.js
+++ b/tools/cli/commands-packages.js
@@ -1612,21 +1612,26 @@ main.registerCommand({
   // args), take patches to indirect dependencies.
   var upgradeIndirectDepPatchVersions = false;
   if (options.args.length === 0) {
-    var allDependencies = _.keys(projectContext.packageMapFile._versions);
-
     // "all-packages" means update every package we depend on. The default
     // is to tend to leave indirect dependencies (i.e. things not listed in
     // `.meteor/packages`) alone.
-    if (options["all-packages"] && allDependencies.length > 0) {
-      upgradePackageNames = allDependencies;
-    } else {
+    if (options["all-packages"]) {
+      upgradePackageNames = _.filter(
+        _.keys(projectContext.packageMapFile.getCachedVersions()),
+        packageName => ! compiler.isIsobuildFeaturePackage(packageName)
+      );
+    }
+
+    if (upgradePackageNames.length === 0) {
       projectContext.projectConstraintsFile.eachConstraint(function (constraint) {
         if (! compiler.isIsobuildFeaturePackage(constraint.package)) {
           upgradePackageNames.push(constraint.package);
         }
       });
     }
+
     upgradeIndirectDepPatchVersions = true;
+
   } else {
     if (options["all-packages"]) {
       Console.error("You cannot both specify a list of packages to"