[ https://issues.apache.org/jira/browse/CB-8484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14387740#comment-14387740 ]
ASF GitHub Bot commented on CB-8484: ------------------------------------ Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/164#discussion_r27446020 --- Diff: bin/templates/cordova/lib/build.js --- @@ -530,8 +609,54 @@ module.exports.findBestApkForArchitecture = function(buildResults, arch) { throw new Error('Could not find apk architecture: ' + arch + ' build-type: ' + buildResults.buildType); }; +function PackageInfo(keystore, alias, keystorePassword, password, storeType) { + keystore = keystore.replace(/\\/g, '\\\\'); + this.keystore = { + 'name': 'key.store', + 'value': keystore + }; + this.alias = { + 'name': 'key.alias', + 'value': alias + }; + if (keystorePassword) { + this.keystorePassword = { + 'name': 'key.store.password', + 'value': keystorePassword + }; + } + if (password) { + this.password = { + 'name': 'key.alias.password', + 'value': password + }; + } + if (storeType) { + this.storeType = { + 'name': 'key.store.type', + 'value': storeType + }; + } +} + +PackageInfo.prototype = { + toProperties: function() { + var self = this; + var result = ''; + Object.keys(self).forEach(function(key) { + if (self[key]) { --- End diff -- Probably should not require the value to be non-empty. Perfectly valid to have `key.alias.password=` in the file (although not recommended :P) > Cordova package support for Android > ----------------------------------- > > Key: CB-8484 > URL: https://issues.apache.org/jira/browse/CB-8484 > Project: Apache Cordova > Issue Type: Sub-task > Components: Android > Affects Versions: 4.0.0 > Reporter: Nikhil Khandelwal > Assignee: Nikhil Khandelwal > > Details of the design here: > https://docs.google.com/document/d/1tJQ9OoGrrMhZcLI3mg46rGzAfbiQu9PuNBL1auAMGFM/edit# -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org For additional commands, e-mail: issues-h...@cordova.apache.org