You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by sg...@apache.org on 2014/06/27 10:45:55 UTC

git commit: CB-6976 fixes deploy error when --nobuild option specified

Repository: cordova-windows
Updated Branches:
  refs/heads/master 69a2c2f23 -> b3a9d0824


CB-6976 fixes deploy error when --nobuild option specified


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

Branch: refs/heads/master
Commit: b3a9d0824a7a7c018ffe60b2192ff090a74e5f17
Parents: 69a2c2f
Author: Vladimir Kotikov <v-...@microsoft.com>
Authored: Thu Jun 26 13:31:16 2014 +0400
Committer: sgrebnov <v-...@microsoft.com>
Committed: Fri Jun 27 12:43:19 2014 +0400

----------------------------------------------------------------------
 windows/template/cordova/lib/deploy.js | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cordova-windows/blob/b3a9d082/windows/template/cordova/lib/deploy.js
----------------------------------------------------------------------
diff --git a/windows/template/cordova/lib/deploy.js b/windows/template/cordova/lib/deploy.js
index d80730d..e99bd2d 100644
--- a/windows/template/cordova/lib/deploy.js
+++ b/windows/template/cordova/lib/deploy.js
@@ -364,10 +364,6 @@ function parseArgs () {
         else if (arg == "--release" || arg == "-r") {
             return "release";
         }
-        else if (arg == "--nobuild") {
-            noBuild = true;
-            return true;
-        }
         return null;
     }
 
@@ -420,6 +416,14 @@ function parseArgs () {
         return null;
     }
 
+    // returns true if nobuild option is specified else false
+    function getNoBuildOption (arg) {
+        if (arg.toLowerCase() == "--nobuild") {
+            return true;
+        }
+        return false;
+    }
+
     for (var i = 0; i < args.Length; i++) {
         if (getBuildType(args(i))) {
             buildType = getBuildType(args(i));
@@ -429,6 +433,8 @@ function parseArgs () {
             buildTarget = getBuildTarget(args(i));
         } else if (getProjectType(args(i))){
             projectType = getProjectType(args(i));
+        } else if (getNoBuildOption(args(i))) {
+            noBuild = getNoBuildOption(args(i));
         } else {
             Log("Error: \"" + args(i) + "\" is not recognized as a build/deploy option", true);
             Usage();