[GitHub] ant-ivy issue #22: Fix transient test failures

2017-05-19 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant-ivy/pull/22 Looks like this didn't help. It failed again (passes locally): >> [junit] Running org.apache.ivy.plugins.circular.WarnCircularDependencyStrategyTest [junit] Tests run: 3, Failures: 1

[GitHub] ant-ivy pull request #22: Fix transient test failures

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

[GitHub] ant-ivy issue #21: Fix IVY-1495

2017-05-19 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant-ivy/pull/21 >> That WarnCircularDependencyStrategyTest testcase though probably needs to be looked into separately, since it keeps failing once in a while on Jenkins. I think this https://github.com/apa

[GitHub] ant-ivy pull request #22: Fix transient test failures

2017-05-19 Thread jaikiran
GitHub user jaikiran opened a pull request: https://github.com/apache/ant-ivy/pull/22 Fix transient test failures The `WarnCircularDependencyStrategyTest` fails once in a while on Jenkins. Looking at the testcase, it resets/updates a shared JVM level logger instance (via `Message.s

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

2017-05-19 Thread J Pai
Not fully familiar with the voting process and don’t know if IvyDE will require a voting thread of its own, but either way +1 for Java 7 for IvyDE too. -Jaikiran On 19-May-2017, at 12:50 PM, Gintautas Grigelionis wrote: Java 7 implements TLS 1.2; Ivy depends a lot more on network communication

Re: Ivy - Buildjobs/PreCommit

2017-05-19 Thread J Pai
This pre-commit job is now almost functional. It gets triggered whenever there’s a PR and runs the build and tests and reports back if there are any failures. However, the only thing that’s pending or seems to be an issue is it sometimes doesn’t report back with the status. In these 2 PRs http

[GitHub] ant-ivy issue #21: Fix IVY-1495

2017-05-19 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant-ivy/pull/21 FWIW - the test failures don't look related. That specific testcase though probably needs to be looked into separately though, since it keeps failing once in a while on Jenkins. --- If your projec

Re: Ivy - BasicURLHandler ignoring timeout during connection?

2017-05-19 Thread Gintautas Grigelionis
My 2¢: timeout has only effect on doHead, in all other cases Ivy is trying the hardest it can. Just search for setTimeout :-) 2017-05-19 16:10 GMT+02:00 J Pai : > I was looking at some timing issues in test cases and noticed that the > BasicURLHandler.getURLInfo with a timeout[1] seems to be ign

Ivy - BasicURLHandler ignoring timeout during connection?

2017-05-19 Thread J Pai
I was looking at some timing issues in test cases and noticed that the BasicURLHandler.getURLInfo with a timeout[1] seems to be ignoring that timeout completely. Am I missing something or is it just a oversight/bug? Furthermore, the javadoc of URLHandler.getURLInfo doesn’t tell much about what t

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

2017-05-19 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant-ivy/pull/20 I am afraid that Ivy allows for a misfeature here. Note that you must test this on Windows because colon is not allowed in NTFS filenames: https://kb.acronis.com/content/39790 --- If your project is

[GitHub] ant-ivy pull request #21: Fix IVY-1495

2017-05-19 Thread jaikiran
GitHub user jaikiran opened a pull request: https://github.com/apache/ant-ivy/pull/21 Fix IVY-1495 The commit here is a potential fix for the issue reported in https://issues.apache.org/jira/browse/IVY-1495. The issue happens when the `ivysettings.xml` is being parsed and the `IvyS

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

2017-05-19 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant-ivy/pull/20 The latest build completed successfully https://builds.apache.org/job/Ivy-GithubPR/15/console. Coming to the question: > Why not sticking only to well-known versioning schemes: OSGi

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

2017-05-19 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant-ivy/pull/20 Looking at the logs, the BasicURLHandler seems to have crashed because the Jenkins process was killed due to: ``` [junit] Running org.apache.ivy.util.url.BasicURLHandlerTest **Bu

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

2017-05-19 Thread nlalevee
Github user nlalevee commented on the issue: https://github.com/apache/ant-ivy/pull/20 I have increased the timeout to 20m --- 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

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

2017-05-19 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant-ivy/pull/20 BasicURLHandler crashed... ":" need to be URL encoded? --- 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 thi

JDK 9 EA Build 170 is available on jdk.java.net

2017-05-19 Thread Rory O'Donnell
Hi Stefan, * JDK 9 Early Access* build 170 is available at the new location : - jdk.java.net/9/ A summary of all the changes in this build are listed here . Changes which were introduced since the last availability email that may

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

2017-05-19 Thread Dominique Devienne
On Fri, May 19, 2017 at 7:25 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). > +1. --DD PS: Ivy already has trouble attracting dev attention, so I feel we should let those maki

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

2017-05-19 Thread Nicolas Lalevée
+1 Be able to test on an old jvm will be near impossible for me. Nicolas > Le 19 mai 2017 à 07:25, Jan Matèrne a écrit : > > 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 incr

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

2017-05-19 Thread J Pai
One of the reasons, other than the fact that I haven’t personally seen anyone using Java 5 in development projects or production for many years now, has to do with the development and testing of the project itself. It’s hard to find a Java 5 runtime to actually test the product against (almost

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

2017-05-19 Thread Maarten Coene
Yes, but it's not because Ivy is compiled with Java 5 that you cannot run it with Java 7 to have TLS1.2 support. Maarten Van: Gintautas Grigelionis Aan: Ant Developers List ; Maarten Coene Verzonden: vrijdag 19 mei 9:20 2017 Onderwerp: Re: [VOTE] Increase minimum Java version for Ivy

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

2017-05-19 Thread Gintautas Grigelionis
Java 7 implements TLS 1.2; Ivy depends a lot more on network communications than Ant does. +1, but please extend the vote to the baseline of IvyDE as well. 2017-05-19 9:03 GMT+02:00 Maarten Coene : > I think we should stick to Java5 as long as Ant 1.9.x is actively > maintained.This keeps Ivy usa

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

2017-05-19 Thread Maarten Coene
I think we should stick to Java5 as long as Ant 1.9.x is actively maintained.This keeps Ivy usable for both Ant 1.9.x and Ant 1.10.x Or are there good reasons to increase the minimum Java version, like features that cannot be implemented with Java 5? But since I don't have the time at the moment