[ 
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

Reply via email to