cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From TimBarham <...@git.apache.org>
Subject [GitHub] cordova-lib pull request: CB-9108 Handle version ranges when add p...
Date Tue, 02 Jun 2015 21:17:43 GMT
GitHub user TimBarham opened a pull request:

    https://github.com/apache/cordova-lib/pull/232

    CB-9108 Handle version ranges when add platform with --usegit.

    Adding a platform using a version range didn't work with `--usegit`, since it would try
to use that version range as a git tag. This was a problem if you added the platform using
`platform add <platform>@<range>`, and also because our pinned platforms now use
tilde versions (which are shorthand for a version range).
    
    Fix is to use `npm view` to find the most recent version that matches the range, and use
that.
    
    We need to do the same when installing from npm, when we do our check to see if the requested
version has previously been downloaded using git.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/MSOpenTech/cordova-lib CB-9108

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cordova-lib/pull/232.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 #232
    
----
commit 884340739b9c289f1acf12bfd63696d45e03482e
Author: Tim Barham <tim.barham@microsoft.com>
Date:   2015-06-02T21:14:57Z

    CB-9108 Handle version ranges when add platform with --usegit.
    
    Adding a platform using a version range didn't work with --usegit, since it would try
to use that version range as a git tag. This was a problem if you added the platform using
'platform add <platform>@<range>', and also because our pinned platforms now use
tilde versions (which are shorthand for a version range).
    
    Fix is to use npm to find the most recent version that matches the range, and use that.
    
    We need to do the same when installing from npm, when we do our check to see if the requested
version has previously been downloaded using git.

----


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


Mime
View raw message