Re: [VOTE] Increase minimum Java version for Ivy to Java7

2017-05-18 Thread J Pai
+1 -Jaikiran On 19-May-2017, at 10:55 AM, Jan Matèrne wrote: As discussed on this mailing list [1] we should increase the minimum Java version [2] from current (Ivy-2.4.0) Java5 to Java 7 (Ivy-2.5.0). Decisions whether to increase to Java8 after that release should be make

[VOTE] Increase minimum Java version for Ivy to Java7

2017-05-18 Thread Jan Matèrne
As discussed on this mailing list [1] we should increase the minimum Java version [2] from current (Ivy-2.4.0) Java5 to Java 7 (Ivy-2.5.0). Decisions whether to increase to Java8 after that release should be make after that release. According to the bylaws [3] this vote is open for one week (until

Re: Minimum Java runtime version for proposed upcoming Ivy release

2017-05-18 Thread J Pai
So far the majority seems to be to require a minimum of Java 7. If there are no concerns or objections to this by the end of this week, then on Monday, I’ll raise a PR to mandate Java 7 for Ivy. -Jaikiran On 18-May-2017, at 11:48 PM, Gintautas Grigelionis wrote:

[GitHub] ant-ivy pull request #20: Fix IVY-1522

2017-05-18 Thread jaikiran
GitHub user jaikiran opened a pull request: https://github.com/apache/ant-ivy/pull/20 Fix IVY-1522 The commit here contains a potential fix for the issue reported in https://issues.apache.org/jira/browse/IVY-1522. This PR also contains a test case to verify the fix.

Re: Minimum Java runtime version for proposed upcoming Ivy release

2017-05-18 Thread Gintautas Grigelionis
Gradle does have an Ivy plugin which wraps Ivy 2.2.0 2017-05-18 11:26 GMT+02:00 Paul King : > The current version of Groovy has 1.6 as the minimum but is our maintenance > stream. > The upcoming next version will require 1.7 and versions with 1.8 as the > minimum are not too

[GitHub] ant-ivy pull request #18: Fix IVY-1448

2017-05-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ant-ivy/pull/18 --- 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

Re: Minimum Java runtime version for proposed upcoming Ivy release

2017-05-18 Thread Gintautas Grigelionis
That's fine, IvyDE is already at Java 7/Eclipse 3.7.1; then IvyDE baseline should be bumped to Java 8/Eclipse 4.4 with the next Ivy release. Hopefully updatesite resolver could be used then. 2017-05-18 17:12 GMT+02:00 Matt Sicker : > Using 1.7 for the next release and then 1.8

[GitHub] ant-ivy pull request #17: Fix IVY-1478

2017-05-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ant-ivy/pull/17 --- 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

Re: Minimum Java runtime version for proposed upcoming Ivy release

2017-05-18 Thread Matt Sicker
Using 1.7 for the next release and then 1.8 for the following release makes sense to me. On 18 May 2017 at 05:58, J Pai wrote: > +1 > > -Jaikiran > On 18-May-2017, at 4:26 PM, Jan Matèrne (jhm) wrote: > > I would favour 1.7 as it's the newest before

Re: Minimum Java runtime version for proposed upcoming Ivy release

2017-05-18 Thread J Pai
+1 -Jaikiran On 18-May-2017, at 4:26 PM, Jan Matèrne (jhm) wrote: I would favour 1.7 as it's the newest before the major update to Java8. Having a 1.7 in the target environment should not been so restrictive ... Jan > -Ursprüngliche Nachricht- > Von: Paul King

AW: Minimum Java runtime version for proposed upcoming Ivy release

2017-05-18 Thread jhm
I would favour 1.7 as it's the newest before the major update to Java8. Having a 1.7 in the target environment should not been so restrictive ... Jan > -Ursprüngliche Nachricht- > Von: Paul King [mailto:pa...@asert.com.au] > Gesendet: Donnerstag, 18. Mai 2017 11:27 > An: Ant Developers

[GitHub] ant-ivy pull request #19: IVY-1554 - Add (conditional) support for SHA-256 S...

2017-05-18 Thread jaikiran
GitHub user jaikiran opened a pull request: https://github.com/apache/ant-ivy/pull/19 IVY-1554 - Add (conditional) support for SHA-256 SHA-512 and SHA-384 checksum algorithms The commit here adds support for SHA-256, SHA-512 and SHA-384 algorithms as requested in

Re: Ivy - Buildjobs/PreCommit

2017-05-18 Thread Nicolas Lalevée
I have changed the config of the build so it does a build, then checkstyle, then findbugs, and then the unit tests. Nicolas > Le 18 mai 2017 à 08:21, J Pai a écrit : > > Now that the findbugs URL issue has been resolved with this PR[1] being > merged, maybe we

Re: Minimum Java runtime version for proposed upcoming Ivy release

2017-05-18 Thread Paul King
The current version of Groovy has 1.6 as the minimum but is our maintenance stream. The upcoming next version will require 1.7 and versions with 1.8 as the minimum are not too far away. Ant 1.9.x is still on Java5 but Ant 1.10.x requires Java 8. I don't think Gradle uses any Ivy classes any

Re: Minimum Java runtime version for proposed upcoming Ivy release

2017-05-18 Thread Nicolas Lalevée
I think that upgrading the requirement on the JDK is a good idea, because at least us, the maintainers, need at some point to be able to test it if there is an issue with that minimum JDK. One thing to consider is which JDK is being required in the environment Ivy is being used: Ant, Gradle,

Minimum Java runtime version for proposed upcoming Ivy release

2017-05-18 Thread J Pai
Now that the plan seems to be to release 2.5.x of Ivy, would it be fine if we mandate the _minimum_ Java runtime version to be something higher than Java 5 that’s currently supported for 2.4.x http://ant.apache.org/ivy/history/latest-milestone/compatibility.html. Given that Java 6 itself has

[GitHub] ant-ivy pull request #18: Fix IVY-1448

2017-05-18 Thread jaikiran
GitHub user jaikiran opened a pull request: https://github.com/apache/ant-ivy/pull/18 Fix IVY-1448 The commit here fixes the issue reported in https://issues.apache.org/jira/browse/IVY-1448 and also includes a test case to verify the fix. **Note that this issue isn't

[GitHub] ant-ivy pull request #13: Dummy PR - Do not merge

2017-05-18 Thread jaikiran
Github user jaikiran closed the pull request at: https://github.com/apache/ant-ivy/pull/13 --- 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

[GitHub] ant-ivy pull request #15: Create just-test-github-integration

2017-05-18 Thread janmaterne
Github user janmaterne closed the pull request at: https://github.com/apache/ant-ivy/pull/15 --- 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

[GitHub] ant-ivy issue #15: Create just-test-github-integration

2017-05-18 Thread janmaterne
Github user janmaterne commented on the issue: https://github.com/apache/ant-ivy/pull/15 close test-PR --- 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

AW: [jira] [Resolved] (IVY-1520) Have makepom task take description from ivy-module > info > description element

2017-05-18 Thread jhm
Ok, thanks. Jan > -Ursprüngliche Nachricht- > Von: Nicolas Lalevée [mailto:nicolas.lale...@hibnet.org] > Gesendet: Mittwoch, 17. Mai 2017 15:37 > An: dev@ant.apache.org > Betreff: Re: [jira] [Resolved] (IVY-1520) Have makepom task take > description from ivy-module > info > description

Re: Ivy - Buildjobs/PreCommit

2017-05-18 Thread J Pai
Now that the findbugs URL issue has been resolved with this PR[1] being merged, maybe we should change the targets that get run via this pre commit job to be just: ant clean test and as a start, see how it goes with the PR integration process? [1] https://github.com/apache/ant-ivy/pull/16

[GitHub] ant-ivy pull request #17: Fix IVY-1478

2017-05-18 Thread jaikiran
GitHub user jaikiran opened a pull request: https://github.com/apache/ant-ivy/pull/17 Fix IVY-1478 The commit here fixes the issue reported in https://issues.apache.org/jira/browse/IVY-1478 and also includes a test case to verify the changes. The fix involves setting the