From 1ff2c3bae988efe78ce0ac10693399ca731fc77a Mon Sep 17 00:00:00 2001
From: David Glasser <glasser@meteor.com>
Date: Fri, 22 Aug 2014 12:38:30 -0700
Subject: [PATCH] Bundle build before creating time-sensitive URL

---
 tools/package-client.js | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/tools/package-client.js b/tools/package-client.js
index 58c8c82ca2..995c65e21b 100644
--- a/tools/package-client.js
+++ b/tools/package-client.js
@@ -427,6 +427,13 @@ exports.bundleBuild = bundleBuild;
 var createAndPublishBuiltPackage = function (conn, unipackage) {
   buildmessage.assertInJob();
 
+  // Note: we really want to do this before createPackageBuild, because the URL
+  // we get from createPackageBuild will expire!
+  process.stdout.write('Bundling build...\n');
+  var bundleResult = bundleBuild(unipackage);
+  if (buildmessage.jobHasMessages())
+    return;
+
   process.stdout.write('Creating package build...\n');
   var uploadInfo = conn.call('createPackageBuild', {
     packageName: unipackage.name,
@@ -434,10 +441,6 @@ var createAndPublishBuiltPackage = function (conn, unipackage) {
     buildArchitectures: unipackage.buildArchitectures()
   });
 
-  var bundleResult = bundleBuild(unipackage);
-  if (buildmessage.jobHasMessages())
-    return;
-
   process.stdout.write('Uploading build...\n');
   uploadTarball(uploadInfo.uploadUrl,
                 bundleResult.buildTarball);