You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by ag...@apache.org on 2013/12/10 02:54:05 UTC

git commit: CB-5619 Avoid Error: Error ...

Updated Branches:
  refs/heads/master ec88d6af8 -> b365138c2


CB-5619 Avoid Error: Error ...

Prevent errors like the following:
[Error: Error fetching plugin: Error: error: Could not fetch package information for imaginary.plugin]

https://github.com/apache/cordova-cli/pull/109


Project: http://git-wip-us.apache.org/repos/asf/cordova-cli/repo
Commit: http://git-wip-us.apache.org/repos/asf/cordova-cli/commit/b365138c
Tree: http://git-wip-us.apache.org/repos/asf/cordova-cli/tree/b365138c
Diff: http://git-wip-us.apache.org/repos/asf/cordova-cli/diff/b365138c

Branch: refs/heads/master
Commit: b365138c2a097da0793e7aae4d46ff8c596ef763
Parents: ec88d6a
Author: Josh Soref <js...@blackberry.com>
Authored: Mon Dec 9 17:00:13 2013 -0500
Committer: Andrew Grieve <ag...@chromium.org>
Committed: Mon Dec 9 20:53:46 2013 -0500

----------------------------------------------------------------------
 src/metadata/amazon_fireos_parser.js | 2 +-
 src/metadata/blackberry10_parser.js  | 2 +-
 src/metadata/windows8_parser.js      | 2 +-
 src/metadata/wp7_parser.js           | 2 +-
 src/metadata/wp8_parser.js           | 2 +-
 src/platform.js                      | 2 +-
 src/plugin.js                        | 2 +-
 7 files changed, 7 insertions(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cordova-cli/blob/b365138c/src/metadata/amazon_fireos_parser.js
----------------------------------------------------------------------
diff --git a/src/metadata/amazon_fireos_parser.js b/src/metadata/amazon_fireos_parser.js
index 4a6291f..2f1e2ab 100644
--- a/src/metadata/amazon_fireos_parser.js
+++ b/src/metadata/amazon_fireos_parser.js
@@ -71,7 +71,7 @@ module.exports.check_requirements = function(project_root) {
                 child_process.exec(cmd, function(err, output, stderr) {
                     events.emit('verbose', output + stderr);
                     if (err) {
-                        d2.reject(new Error('Error updating the Cordova library to work with your Android environment. Command run: "' + cmd + '", output: ' + output));
+                        d2.reject(new Error('Updating the Cordova library to work with your Android environment failed when running: "' + cmd + '", output: ' + output));
                     } else {
                         events.emit('log', 'Checking if ' + awv_interface + ' exists... in framework/libs folder');
                         var awv_interface_expected_path=path.join(framework_path,'libs');

http://git-wip-us.apache.org/repos/asf/cordova-cli/blob/b365138c/src/metadata/blackberry10_parser.js
----------------------------------------------------------------------
diff --git a/src/metadata/blackberry10_parser.js b/src/metadata/blackberry10_parser.js
index b5b8731..4ed280d 100644
--- a/src/metadata/blackberry10_parser.js
+++ b/src/metadata/blackberry10_parser.js
@@ -47,7 +47,7 @@ module.exports.check_requirements = function(project_root) {
     var d = Q.defer();
     child_process.exec("\"" + path.join(lib_path, 'bin', 'check_reqs') + "\"", function(err, output, stderr) {
         if (err) {
-            d.reject(new Error('Error while checking requirements: ' + output + stderr));
+            d.reject(new Error('Requirements check failed: ' + output + stderr));
         } else {
             d.resolve();
         }

http://git-wip-us.apache.org/repos/asf/cordova-cli/blob/b365138c/src/metadata/windows8_parser.js
----------------------------------------------------------------------
diff --git a/src/metadata/windows8_parser.js b/src/metadata/windows8_parser.js
index 194a654..e833d16 100644
--- a/src/metadata/windows8_parser.js
+++ b/src/metadata/windows8_parser.js
@@ -63,7 +63,7 @@ module.exports.check_requirements = function(project_root) {
     child_process.exec(command, function(err, output, stderr) {
         events.emit('verbose', output);
         if (err) {
-            d.reject(new Error('Error while checking requirements: ' + output + stderr));
+            d.reject(new Error('Requirements check failed: ' + output + stderr));
         } else {
             d.resolve();
         }

http://git-wip-us.apache.org/repos/asf/cordova-cli/blob/b365138c/src/metadata/wp7_parser.js
----------------------------------------------------------------------
diff --git a/src/metadata/wp7_parser.js b/src/metadata/wp7_parser.js
index 11a0016..2b441ed 100644
--- a/src/metadata/wp7_parser.js
+++ b/src/metadata/wp7_parser.js
@@ -59,7 +59,7 @@ module.exports.check_requirements = function(project_root) {
     child_process.exec(command, function(err, output, stderr) {
         events.emit('verbose', output+stderr);
         if (err) {
-            d.reject(new Error('Error while checking requirements: ' + output + stderr));
+            d.reject(new Error('Requirements check failed: ' + output + stderr));
         } else {
             d.resolve();
         }

http://git-wip-us.apache.org/repos/asf/cordova-cli/blob/b365138c/src/metadata/wp8_parser.js
----------------------------------------------------------------------
diff --git a/src/metadata/wp8_parser.js b/src/metadata/wp8_parser.js
index 6393759..163d56b 100644
--- a/src/metadata/wp8_parser.js
+++ b/src/metadata/wp8_parser.js
@@ -59,7 +59,7 @@ module.exports.check_requirements = function(project_root) {
     child_process.exec(command, function(err, output, stderr) {
         events.emit('verbose', output);
         if (err) {
-            d.reject(new Error('Error while checking requirements: ' + output + stderr));
+            d.reject(new Error('Requirements check failed: ' + output + stderr));
         } else {
             d.resolve();
         }

http://git-wip-us.apache.org/repos/asf/cordova-cli/blob/b365138c/src/platform.js
----------------------------------------------------------------------
diff --git a/src/platform.js b/src/platform.js
index 730e411..21a61b7 100644
--- a/src/platform.js
+++ b/src/platform.js
@@ -120,7 +120,7 @@ module.exports = function platform(command, targets) {
                     var d = Q.defer();
                     child_process.exec(script + ' "' + path.join(projectRoot, 'platforms', plat) + '"', function(err, stdout, stderr) {
                         if (err) {
-                            d.reject(new Error('Error running update script: ' + err + stderr));
+                            d.reject(new Error('Update script failed: ' + err + stderr));
                         } else {
                             events.emit('log', plat + ' updated to ' + platforms[plat].version);
                             d.resolve();

http://git-wip-us.apache.org/repos/asf/cordova-cli/blob/b365138c/src/plugin.js
----------------------------------------------------------------------
diff --git a/src/plugin.js b/src/plugin.js
index a0294d7..b1c0e4f 100644
--- a/src/plugin.js
+++ b/src/plugin.js
@@ -87,7 +87,7 @@ module.exports = function plugin(command, targets) {
                         return plugman.raw.fetch(target, pluginsDir, {})
                     })
                     .fail(function(err) {
-                        return Q.reject(new Error('Error fetching plugin: ' + err));
+                        return Q.reject(new Error('Fetching plugin failed: ' + err));
                     })
                     .then(function(dir) {
                         // Iterate (in serial!) over all platforms in the project and install the plugin.