From 73b33ee4da5bf3d9509ed07f55a359c65774f01d Mon Sep 17 00:00:00 2001 From: David Glasser Date: Mon, 18 Aug 2014 00:28:23 -0700 Subject: [PATCH] note about why official.refresh uses buildmessage --- tools/catalog.js | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/tools/catalog.js b/tools/catalog.js index 8b4f01036d..f84f354f26 100644 --- a/tools/catalog.js +++ b/tools/catalog.js @@ -112,6 +112,11 @@ _.extend(OfficialCatalog.prototype, { // the in-progress refresh to finish. refresh: function () { var self = this; + // note: this only needs to be in a capture because it refreshes the + // complete catalog (which actually uses the build system). if + // catalog.complete was refactored to not require a rebuild whenever + // catalog.official changes, this function wouldn't need + // buildmessage.assertInCapture any more. buildmessage.assertInCapture(); self._requireInitialized();