From 130bf7795719b20c5cf956a031b2de1920dfcacc Mon Sep 17 00:00:00 2001
From: David Glasser <glasser@meteor.com>
Date: Wed, 5 Nov 2014 18:02:37 -0800
Subject: [PATCH] add XXX comment about downloading package message

---
 tools/tropohouse.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/tools/tropohouse.js b/tools/tropohouse.js
index f9c6c1e402..d06a4ccbe2 100644
--- a/tools/tropohouse.js
+++ b/tools/tropohouse.js
@@ -313,6 +313,11 @@ _.extend(exports.Tropohouse.prototype, {
     options = options || {};
     var serverArch = options.serverArch || archinfo.host();
     var downloadedPackages = {};
+    // XXX We intentionally didn't make this message "downloading packages"
+    // because users were confused if this message showed up when there wasn't
+    // actually any packages being downloaded. But since we use forkJoin here,
+    // the nested downloading jobs don't print, so now there's NEVER a
+    // downloading status message.
     buildmessage.forkJoin({ title: 'Checking local package versions', parallel: true },
       versionMap, function (version, name) {
       try {