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/10/02 20:18:00 UTC
[26/29] git commit: CB-7512 Use aligned apk rather than unaligned apk
when sorting
CB-7512 Use aligned apk rather than unaligned apk when sorting
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/82fb93d6
Tree: http://git-wip-us.apache.org/repos/asf/cordova-amazon-fireos/tree/82fb93d6
Diff: http://git-wip-us.apache.org/repos/asf/cordova-amazon-fireos/diff/82fb93d6
Branch: refs/heads/master
Commit: 82fb93d64866e0a4987ed2766eff9c1a429adf27
Parents: 3e18394
Author: Andrew Grieve <ag...@chromium.org>
Authored: Tue Sep 16 14:59:43 2014 -0400
Committer: Prabhjot Singh <pr...@amazon.com>
Committed: Wed Sep 17 22:53:54 2014 +0530
----------------------------------------------------------------------
bin/templates/cordova/lib/build.js | 20 +++++++++++++-------
1 file changed, 13 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cordova-amazon-fireos/blob/82fb93d6/bin/templates/cordova/lib/build.js
----------------------------------------------------------------------
diff --git a/bin/templates/cordova/lib/build.js b/bin/templates/cordova/lib/build.js
index e63e342..3224758 100644
--- a/bin/templates/cordova/lib/build.js
+++ b/bin/templates/cordova/lib/build.js
@@ -38,8 +38,11 @@ function find_files(directory, predicate) {
p = path.join(directory, p);
return { p: p, t: fs.statSync(p).mtime };
}).sort(function(a,b) {
- return a.t > b.t ? -1 :
- a.t < b.t ? 1 : 0;
+ var timeDiff = b.t - a.t;
+ if (timeDiff === 0) {
+ return a.p.length - b.p.length;
+ }
+ return timeDiff;
}).map(function(p) { return p.p; });
return candidates;
} else {
@@ -151,8 +154,9 @@ var builders = {
console.error('ERROR : No .apk found in ' + binDir + ' directory');
process.exit(2);
}
- console.log('Using apk: ' + candidates[0]);
- return [candidates[0]];
+ var ret = candidates[0];
+ console.log('Using apk: ' + ret);
+ return [ret];
}
},
gradle: {
@@ -245,14 +249,16 @@ var builders = {
var candidates = find_files(binDir, function(candidate) {
// Need to choose between release and debug .apk.
if (build_type === 'debug') {
- return (path.extname(candidate) == '.apk' && candidate.indexOf('-debug-') >= 0);
+ return (path.extname(candidate) == '.apk' && candidate.indexOf('-debug') >= 0);
}
if (build_type === 'release') {
- return (path.extname(candidate) == '.apk' && candidate.indexOf('-release-') >= 0);
+ return (path.extname(candidate) == '.apk' && candidate.indexOf('-release') >= 0);
}
return path.extname(candidate) == '.apk';
});
- return candidates;
+ var ret = candidates[0];
+ console.log('Using apk: ' + ret);
+ return [ret];
}
},
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org