You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by na...@apache.org on 2014/08/28 21:03:43 UTC
[29/50] git commit: CB-3445 Speed up gradle building (incremental
builds go from 10s -> 1.5s for me)
CB-3445 Speed up gradle building (incremental builds go from 10s -> 1.5s for me)
Biggest win is disabling the linter.
Project: http://git-wip-us.apache.org/repos/asf/cordova-amazon-fireos/repo
Commit: http://git-wip-us.apache.org/repos/asf/cordova-amazon-fireos/commit/c3eda01f
Tree: http://git-wip-us.apache.org/repos/asf/cordova-amazon-fireos/tree/c3eda01f
Diff: http://git-wip-us.apache.org/repos/asf/cordova-amazon-fireos/diff/c3eda01f
Branch: refs/heads/master
Commit: c3eda01f56f58a8a567057ad9175496657d413a1
Parents: db0847e
Author: Andrew Grieve <ag...@chromium.org>
Authored: Mon Aug 18 14:16:27 2014 -0400
Committer: Archana Naik <na...@lab126.com>
Committed: Mon Aug 18 12:40:58 2014 -0700
----------------------------------------------------------------------
bin/templates/cordova/lib/build.js | 15 +++++++++++++++
1 file changed, 15 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cordova-amazon-fireos/blob/c3eda01f/bin/templates/cordova/lib/build.js
----------------------------------------------------------------------
diff --git a/bin/templates/cordova/lib/build.js b/bin/templates/cordova/lib/build.js
index 3d0ae87..1f0cd23 100644
--- a/bin/templates/cordova/lib/build.js
+++ b/bin/templates/cordova/lib/build.js
@@ -102,7 +102,22 @@ module.exports.builders = {
},
gradle: {
getArgs: function(cmd) {
+ var lintSteps = [
+ 'lint',
+ 'lintVitalRelease',
+ 'compileLint',
+ 'copyReleaseLint',
+ 'copyDebugLint'
+ ];
var args = [cmd, '-b', path.join(ROOT, 'build.gradle')];
+ // 10 seconds -> 6 seconds
+ args.push('-Dorg.gradle.daemon=true');
+ // Excluding lint: 6s-> 1.6s
+ for (var i = 0; i < lintSteps.length; ++i) {
+ args.push('-x', lintSteps[i]);
+ }
+ // Shaves another 100ms, but produces a "try at own risk" warning. Not worth it (yet):
+ // args.push('-Dorg.gradle.parallel=true');
return args;
},