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

ASF GitHub Bot commented on IVY-1486:
-------------------------------------

GitHub user twogee opened a pull request:

    https://github.com/apache/ant-ivy/pull/63

    IVY-1486 respect exclude regardless of order

    

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

    $ git pull https://github.com/twogee/ant-ivy ivy-1486

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

    https://github.com/apache/ant-ivy/pull/63.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 #63
    
----
commit c2601118d9a41aac00a0c0eb89279095d9a83712
Author: twogee <g.grigelionis@...>
Date:   2018-01-09T07:04:42Z

    IVY-1486 respect exclude regardless of order

----


> Order-dependent resolution: exclude not respected
> -------------------------------------------------
>
>                 Key: IVY-1486
>                 URL: https://issues.apache.org/jira/browse/IVY-1486
>             Project: Ivy
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: master
>            Reporter: David Turner
>            Assignee: Antoine Levy-Lambert
>         Attachments: 0001-Fix-IVY-1486.patch, 
> 0001-Make-doesCallersExclude-check-only-callers-of-this-v.patch, ivy-4sq.xml, 
> ivy-exclude-finagle.xml, ivy-min2.xml
>
>
> To repro, create an  ivy.xml which looks like this (see the attached one for 
> an real-world example that you can test out):
> dependency A 1.1
> dependency X 1.2
>   exclude Y
> -------
> dependency A 1.1's depends on:
>    dependency X 1.1
> Dependency X 1.1's depends on:
>   dependency Y 1.1
> Dependency X 1.2's depends on:
>   dependency Y 1.2
> ------------------------------
> In this case, we will get Y 1.2.  But if we swap the order of the 
> dependencies in our ivy.xml, we'll get Y 1.1.  I think 1.1 is correct, 
> because we've excluded 1.2. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to