You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by bh...@apache.org on 2013/08/02 17:29:44 UTC

webworks commit: [CB3439] Introduces --web-inspector flag to enable webinspector for signed builds

Updated Branches:
  refs/heads/master b6becd2a2 -> f3f5d3d62


[CB3439] Introduces --web-inspector flag to enable webinspector for signed builds

Reviewed by Bryan Higgins <bh...@blackberry.com>
Tested by Tracy Li <tl...@blackberry.com>


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

Branch: refs/heads/master
Commit: f3f5d3d62b62da9fd34803f05587222c26421e98
Parents: b6becd2
Author: jkeshavarzi <jk...@blackberry.com>
Authored: Wed Jul 31 16:29:12 2013 -0400
Committer: Bryan Higgins <br...@bryanhiggins.net>
Committed: Fri Aug 2 11:28:58 2013 -0400

----------------------------------------------------------------------
 blackberry10/bin/templates/project/cordova/lib/build | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cordova-blackberry/blob/f3f5d3d6/blackberry10/bin/templates/project/cordova/lib/build
----------------------------------------------------------------------
diff --git a/blackberry10/bin/templates/project/cordova/lib/build b/blackberry10/bin/templates/project/cordova/lib/build
index a4f7656..8b54709 100755
--- a/blackberry10/bin/templates/project/cordova/lib/build
+++ b/blackberry10/bin/templates/project/cordova/lib/build
@@ -39,13 +39,6 @@ function copyArgIfExists(arg) {
     }
 }
 
-function doDebugBuild() {
-    //build in debug mode by default
-    bbwpArgv.push("-d");
-
-    build();
-}
-
 function doReleaseBuild() {
     //Note: Packager refers to signing password as "password" not "keystorepass"
     if (command["keystorepass"]) {
@@ -66,6 +59,11 @@ function doReleaseBuild() {
 }
 
 function build() {
+    //enable webinspector in debug mode or if --webinspector was provided
+    if (!command.release || command.webInspector) {
+        bbwpArgv.push("-d");
+    }
+
     copyArgIfExists("params");
     copyArgIfExists("loglevel");
 
@@ -83,7 +81,7 @@ function postClean() {
     if (command.release) {
         doReleaseBuild();
     } else {
-        doDebugBuild();
+        build();
     }
 }
 
@@ -116,6 +114,7 @@ command
     .option('--release', 'build in release mode. This will sign the resulting bar.')
     .option('-k, --keystorepass <password>', 'signing key password')
     .option('-b, --buildId <num>', 'specifies the build number for signing (typically incremented from previous signing).')
+    .option('-d, --web-inspector', 'enables webinspector. Enabled by default in debug mode.).')
     .option('-p, --params <params JSON file>', 'specifies additional parameters to pass to downstream tools.')
     .option('-ll, --loglevel <loglevel>', 'set the logging level (error, warn, verbose)');