You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by barthel <gi...@git.apache.org> on 2015/07/09 22:51:44 UTC

[GitHub] maven pull request: MNG-3092: strategy based version resolver

GitHub user barthel opened a pull request:

    https://github.com/apache/maven/pull/56

    MNG-3092: strategy based version resolver

    These changes replaces the default version range resolver with a strategy based version range resolver.

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

    $ git pull https://github.com/barthel/maven MNG-3092

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

    https://github.com/apache/maven/pull/56.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 #56
    
----
commit fa8daaea046ff3acc7d78ad4bb92d4ada49c3eaa
Author: barthel <ba...@users.noreply.github.com>
Date:   2015-06-23T18:04:41Z

    Merge pull request #1 from apache/master
    
    Merge all commits from apache/master into barthel/master.

commit 136073cca113b13aa3b5ca09d35c37b99b47a6ce
Author: barthel <ba...@users.noreply.github.com>
Date:   2015-04-30T12:37:52Z

    MNG-3092: Add basal tests for DefaultVersionRangeResolver.

commit 9a5635e05611b5910b490eccce8f8c58fdef7607
Author: barthel <ba...@users.noreply.github.com>
Date:   2015-07-02T22:06:00Z

    MNG-3092: Add copy of Maven default version range behaviour.
    
    Copy the DefaultVersionRangeResolver and add with the new name
    MavenDefaultVersionRangeResolver.
    First step to insert the strategy pattern for different version range
    resolver.

commit da16089672eaa1f1ee9c6d0fe3349f0aff4cce5f
Author: barthel <ba...@users.noreply.github.com>
Date:   2015-07-02T22:18:18Z

    MNG-3092: Add release only version range resolver strategy.
    
    This strategy resolves only released versions within the version range.

commit f084f6fdd27b805b54c47a821f34221b29179966
Author: barthel <ba...@users.noreply.github.com>
Date:   2015-07-02T22:23:19Z

    MNG-3092: Add new only released version on boundaries strategy.
    
    This strategy resolves all versions in version range but only released
    version at the lowest and highest boundary.

commit 7986e21a2f730ad3af431aa213ff3c280f4c853d
Author: barthel <ba...@users.noreply.github.com>
Date:   2015-07-03T23:28:36Z

    MNG-3092: Use strategy version range resolver as default.
    
    Add a new strategy based version range resolver and use
    as default resolver.
    Leave DefaultVersionRangeResolver for backward compatibility
    only.

commit 62bee372fac0b3bc2a81a2d88971361bba817199
Author: barthel <ba...@users.noreply.github.com>
Date:   2015-07-04T00:06:38Z

    MNG-3092: Make AbstractVersionRangeTestCase abstract.

commit 984435cdcbf56447d33f0db2cc00038a422a0e37
Author: barthel <ba...@users.noreply.github.com>
Date:   2015-07-04T01:32:36Z

    MNG-3092: Configure strategy version range resolver in Guice module
    
    Replace the DefaultVersionRangeResolver with
    StrategyVersionRangeResolver in MavenAetherModule as default.
    Lazy initialize the version range resolver strategy instance.

commit 08f6d223d5df64c8c21a522484ad9d5c7969d6fe
Author: barthel <ba...@users.noreply.github.com>
Date:   2015-07-05T20:17:23Z

    MNG-3092: Add final's and extends JavaDoc.
    
    Add some final at variables and parameters.
    Add or extend JavaDoc on modified/added classes.

commit 9010f79b76e1ca97742d1cf095e7aa5fd309d657
Author: barthel <ba...@users.noreply.github.com>
Date:   2015-07-05T20:33:26Z

    Merge pull request #2 from apache/master
    
    [MNG-5813] pass debug-opts from mvnDebug to mvn script in the additio…

commit e3ff2d62d3927ee1098dfeee12aa7a0a0f81cd99
Author: barthel <ba...@users.noreply.github.com>
Date:   2015-07-06T19:44:57Z

    Merge branch 'MNG-3092'

commit 967ff00b32403b25c5eb16c41d4d91b5fb53f93b
Author: barthel <ba...@users.noreply.github.com>
Date:   2015-07-06T19:45:08Z

    Merge branch 'master' into MNG-3092

commit e0ff7b6c23a71f261760d56e2c392158c5f0b575
Author: barthel <ba...@users.noreply.github.com>
Date:   2015-07-06T22:18:34Z

    Use RepositorySystemSession#getSystemProperties instead of System#getProperties

----


---
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@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[GitHub] maven pull request: MNG-3092: strategy based version resolver

Posted by barthel <gi...@git.apache.org>.
Github user barthel closed the pull request at:

    https://github.com/apache/maven/pull/56


---
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@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[GitHub] maven pull request: MNG-3092: strategy based version resolver

Posted by jvanzyl <gi...@git.apache.org>.
Github user jvanzyl commented on the pull request:

    https://github.com/apache/maven/pull/56#issuecomment-120199784
  
    I'm generally against changing anything to do with resolving strategies and it has the potential for breaking everything because it's not just what you might use, but what you might publish using your differences and others being exposed to what your assumptions are. So you'll have provide some reasoning. Also note most of this is controlled in Aether. Also, please squash your changes into a single commit.


---
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@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[GitHub] maven pull request: MNG-3092: strategy based version resolver

Posted by barthel <gi...@git.apache.org>.
Github user barthel commented on the pull request:

    https://github.com/apache/maven/pull/56#issuecomment-120301859
  
    I'll open a new pull request with a stashed commit and more explanation.


---
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@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org