You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cordova.apache.org by TimBarham <gi...@git.apache.org> on 2015/04/06 16:24:30 UTC
[GitHub] cordova-lib pull request: CB-8799 Save plugin/platform src and ver...
GitHub user TimBarham opened a pull request:
https://github.com/apache/cordova-lib/pull/202
CB-8799 Save plugin/platform src and version to 'spec' attribute.
When `--save` option is used, plugin and platform source location or version value is saved to `spec` attribute in `config.xml` (source location takes priority over version). This replaces `version` attribute for platforms, and `src` and `version` attributes for plugins.
Also:
* Updated semver dependency to 2.1.0 to support tilde range.
* Ensure when save platform version (to platforms.json and config.xml) we save the version that was actually installed.
* Ensure when save platform version to config.xml we save it as a tilde range.
Note that if it is decided we prefer to use `src` or `version` attributes instead of `spec`, a branch with the equivalent change but using the `src` attribute can be found [here](https://github.com/apache/cordova-lib/compare/master...MSOpenTech:CB-8799-src), and for the `version` attribute [here](https://github.com/apache/cordova-lib/compare/master...MSOpenTech:CB-8799-version).
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/MSOpenTech/cordova-lib CB-8799-spec
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cordova-lib/pull/202.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 #202
----
commit 2cae6de925ef1bde5d7ed580f3dd14043b4659a5
Author: Tim Barham <ti...@microsoft.com>
Date: 2015-04-06T13:42:29Z
CB-8799 Save plugin/platform src and version to 'spec' attribute.
When --save option is used, plugin and platform source location or version value is saved to 'spec' attribute in config.xml (source location takes priority over version). This replaces 'version' attribute for platforms, and 'src' and 'version' attributes for plugins.
Also:
* Updated semver dependency to 2.1.0 to support tilde range.
* Ensure when save platform version (to platforms.json and config.xml) we save the version that was actually installed.
* Ensure when save platform version to config.xml we save it as a tilde range.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org
[GitHub] cordova-lib pull request: CB-8799 Save plugin/platform src and ver...
Posted by nikhilkh <gi...@git.apache.org>.
Github user nikhilkh commented on the pull request:
https://github.com/apache/cordova-lib/pull/202#issuecomment-91326331
@gorkem Any chance you can review this today? Ideally, this can be part of the next TOOLS release.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org
[GitHub] cordova-lib pull request: CB-8799 Save plugin/platform src and ver...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-lib/pull/202
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org