You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2015/06/10 00:24:01 UTC
[jira] [Commented] (CB-9147) Adding a platform with a caret version
adds the very latest rather than the latest matching version
[ https://issues.apache.org/jira/browse/CB-9147?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14579665#comment-14579665 ]
ASF GitHub Bot commented on CB-9147:
------------------------------------
GitHub user TimBarham opened a pull request:
https://github.com/apache/cordova-lib/pull/244
CB-9147 Adding a platform via caret version adds wrong version
It adds the very latest version rather than the latest matching version.
Internally we are using a version of `npm` that doesn't understand caret versions, so it just gets the latest. I missed this previously because when I tried this, the latest version **was also** the latest matching version.
The same problem also applies to plugins.
I've made a slightly different fix in each case:
For platforms, we already had the latest matching version as a result of our call to `getLatestMatchingNpmVersion()`, so I just use that.
For plugins, in this specific scenario (a caret version) I turn it into a standard range using a version of `semver` that understands caret ranges (so `^1.2.3` becomes `>=1.2.3 <2.0.0`) and use that instead.
Note that this is currently an issue with platforms and plugin versions saved to `config.xml`.
Also, I noticed that we failed to find the version in `config.xml` if you added it using the new form (`cordova-plugin-camera`, for example), but it was stored in `config.xml` using the id, so fixed that.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/MSOpenTech/cordova-lib CB-9147
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cordova-lib/pull/244.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #244
----
commit 025127916530de530dccaeb98b2b9a9bc8df8e2a
Author: Tim Barham <ti...@microsoft.com>
Date: 2015-06-09T22:20:13Z
CB-9147 Adding a platform via caret version adds latest rather than the latest matching.
Internally we are using a version of npm that doesn't understand caret versions, so it just gets the latest. Missed this previously because when I tried this, the latest version *was* the latest matching version.
The same problem also applies to plugins.
Slightly different fix in each case:
For platforms, we already had the latest matching version as a result of our call to getLatestMatchingNpmVersion(), so we just use that.
For plugins, in this specific scenario (a caret version) we turn it into a standard range using a version of semver that understands caret ranges (so '^1.2.3' becomes '>=1.2.3 <2.0.0') and use that instead.
Also, I noticed that we failed to find the version in config.xml if you added it using the new form (cordova-plugin-camera, for example), but it was stored in config.xml using the id, so fixed that.
----
> 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-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org