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

ASF GitHub Bot commented on CB-9147:
------------------------------------

Github user omefire commented on a diff in the pull request:

    https://github.com/apache/cordova-lib/pull/244#discussion_r32089468
  
    --- Diff: cordova-lib/src/plugman/registry/registry.js ---
    @@ -364,6 +364,26 @@ function fetchPlugin(plugin, client, useNpmRegistry) {
         });
     }
     
    +function processPluginVersion(plugin) {
    +    // If plugin includes a version that is a caret range, the ancient 
version of npm we're using won't now how to
    --- End diff --
    
    :nit 'now' -> 'know'


> Adding a platform with a caret version adds the very latest rather than the 
> latest matching version
> ---------------------------------------------------------------------------------------------------
>
>                 Key: CB-9147
>                 URL: https://issues.apache.org/jira/browse/CB-9147
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: CordovaLib
>    Affects Versions: 5.0.0
>            Reporter: Tim Barham
>            Assignee: Tim Barham
>
> If you add, say, windows 3.5.0 platform to a project with the save flag:
> {noformat}
> cordova platform add windows@3.5.0 --save
> {noformat}
> We write out {{^3.5.0}} to {{config.xml}}.
> If the platform is removed then re-added, we install {{4.0.0}} instead of the 
> expected {{3.8.2}}.
> This is because internally we are using a version of {{npm}} that doesn't 
> understand caret versions.



--
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