From e4bd9b7a07147bb090004e5bb8418adda4e33fcf Mon Sep 17 00:00:00 2001 From: ekatek Date: Tue, 13 Jan 2015 13:52:22 -0800 Subject: [PATCH] merging 'meteor show' changes This is mostly cleaning up some syntax errrors that have resulted from merging the previous commits. --- tools/commands-packages-query.js | 2 +- tools/package-client.js | 11 +++-------- tools/tests/package-tests.js | 5 +++-- 3 files changed, 7 insertions(+), 11 deletions(-) diff --git a/tools/commands-packages-query.js b/tools/commands-packages-query.js index 3eaca68113..6bf4734cd2 100644 --- a/tools/commands-packages-query.js +++ b/tools/commands-packages-query.js @@ -809,7 +809,7 @@ _.extend(PackageQuery.prototype, { catalog.official.getVersion(data.name, data.version)) { Console.info(); Console.info( - "This package version is build locally from source.", + "This package version is built locally from source.", "The same version of this package also exists on the package server.", "To view its metadata, run", Console.command("'meteor show " + data.name + "@" + data.version + "'"), diff --git a/tools/package-client.js b/tools/package-client.js index fee635beb9..228afce8be 100644 --- a/tools/package-client.js +++ b/tools/package-client.js @@ -721,6 +721,7 @@ exports.publishPackage = function (options) { }); if (buildmessage.jobHasMessages()) return; +console.log(uploadInfo); // XXX If package version already exists, print a nice error message // telling them to try 'meteor publish-for-arch' if they want to @@ -740,13 +741,6 @@ exports.publishPackage = function (options) { if (buildmessage.jobHasMessages()) return; - buildmessage.enterJob("publishing package " + name, function () { - callPackageServerBM(conn, 'publishPackageVersion', - uploadInfo.uploadToken, - { tarballHash: sourceBundleResult.tarballHash, - treeHash: sourceBundleResult.treeHash }); - }); - var hashes = { tarballHash: sourceBundleResult.tarballHash, treeHash: sourceBundleResult.treeHash, @@ -766,7 +760,8 @@ exports.publishPackage = function (options) { } return; - }; + } +}; // Call the server to ask if we are authorized to update this release or // package. This is a way to save time before sending data to the server. It diff --git a/tools/tests/package-tests.js b/tools/tests/package-tests.js index 578ffe3673..ef322f72dd 100644 --- a/tools/tests/package-tests.js +++ b/tools/tests/package-tests.js @@ -1335,8 +1335,9 @@ selftest.define("show and search local overrides server", // When we ask for version 1.0.0, we get the local version. var addendum = - "The same version of this package also exists on the package server. " + - "To view its\nmetadata, run 'meteor show " + fullPackageName + + "This package version is built locally from source. " + + "The same version of this\npackage also exists on the package server. " + + "To view its metadata, run\n'meteor show " + fullPackageName + "@1.0.0' from outside the project."; testShowPackageVersion(s, { packageName: fullPackageName,