[GitHub] ant-ivy pull request #65: IVY-1485 Ensure dependency is applicable to all co...

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] ant-ivy pull request #65: IVY-1485 Ensure dependency is applicable to all co...

twogee
GitHub user twogee opened a pull request:

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

    IVY-1485 Ensure dependency is applicable to all configurations

    I adopted a patch by @tbingaman

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

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

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

    https://github.com/apache/ant-ivy/pull/65.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 #65
   
----
commit fbac59fbbd6f7c7fa8aa99597f8804211d05bd26
Author: twogee <g.grigelionis@...>
Date:   2018-02-01T13:54:30Z

    IVY-1485 Ensure dependency is applicable to all configurations

----


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] ant-ivy issue #65: IVY-1485 Ensure dependency is applicable to all configura...

twogee
Github user twogee commented on the issue:

    https://github.com/apache/ant-ivy/pull/65
 
    @jaikiran please let me know if you did some addition coding, otherwise I hope you would rather focus on JUnit 5 😄


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] ant-ivy issue #65: IVY-1485 Ensure dependency is applicable to all configura...

twogee
In reply to this post by twogee
Github user jaikiran commented on the issue:

    https://github.com/apache/ant-ivy/pull/65
 
    These changes don't solve the real issue. The details of this issue are part of the discussion in ant dev mailing list, where I explained what the real issue is. I do have a completely different code with tests that's not yet ready to be merged.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] ant-ivy pull request #65: IVY-1485 Ensure dependency is applicable to all co...

twogee
In reply to this post by twogee
Github user twogee closed the pull request at:

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


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]