[ 
https://issues.apache.org/jira/browse/CB-8484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14387741#comment-14387741
 ] 

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_r27446047
  
    --- 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, '\\\\');
    --- End diff --
    
    This escaping will need to be done for all of the values I think. Would be 
more appropriate to do it within the toProperties function


> 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