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

2017-05-20 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant-ivy/pull/20 I think both me and Nicolas agree that this "fix" isn't going to solve the case where this and other similar characters can't be fully supported as long as we are using the module descriptor

[GitHub] ant-ivy pull request #23: Use the specified timeout for connections in Basic...

2017-05-20 Thread jaikiran
GitHub user jaikiran opened a pull request: https://github.com/apache/ant-ivy/pull/23 Use the specified timeout for connections in BasicURLHandler The commit here fixes the `BasicURLHandler` to use the specified timeout for connection timeout. It also updates the

Ivy - What roles do some of these (internal) resolution files play?

2017-05-20 Thread J Pai
I’ve been going through some of the Ivy code recently and have a few related questions. Some of my basic testing shows that a “resolve” of a module produces the following (internal) files: 1. A resolution report, in the form of XML, per configuration of the module, in the form of --.xml (ex:

Re: Ivy - BasicURLHandler ignoring timeout during connection?

2017-05-20 Thread J Pai
Having been involved in a HttpClient 3 to 4 migration in an unrelated project, I think at this point, it’s too big a change to consider. A few stable Ivy releases down the line, maybe we can re-evaluate this and consider the HttpClient libraries and their state at that time. -Jaikiran On

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

2017-05-20 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant-ivy/pull/20 I can assure you that colon in version has NEVER have worked on Windows. Test case ``` import java.io.*; public class TestColonNTFS { public static void main(String[]

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

2017-05-20 Thread nlalevee
Github user nlalevee commented on the issue: https://github.com/apache/ant-ivy/pull/20 I agree with you both. Ivy has never specified a version scheme, and it is kind of its strength, it can be compatible with a lot of use case. We should continue to do so. But we shouldn't

Re: Ivy - BasicURLHandler ignoring timeout during connection?

2017-05-20 Thread Gintautas Grigelionis
Looks like HttpClientHandler was added as an alternative to handle proxies with authentication by wrapping HttpClient from Commons. There is a maintenance problem because HttpClient has a major API change between version 3 (that Ivy uses currently) and 4. Perhaps it makes sense to look at

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

2017-05-20 Thread Gintautas Grigelionis
Should we start talking about using HTTP/2? Then we're firmly in Java 8 territory :-) Gintas 2017-05-19 9:23 GMT+02:00 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

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

2017-05-20 Thread Gintautas Grigelionis
Sorry for sloppy suggestion, I did not mean changing the subject. My intention was to remind about another component which has Ivy as a direct dependency and that it makes sense to align their baselines. Gintas 2017-05-20 11:25 GMT+02:00 Nicolas Lalevée : > Ant

Re: Ivy - BasicURLHandler ignoring timeout during connection?

2017-05-20 Thread Nicolas Lalevée
I don’t know the history of that code, it probably is older that its incubation into Apache. But from what I can read, I think that timeout was introduced but just supported by one implementation of URLHandler: the HttpClient one, HttpClientHandler. Proper support in the BasicURLHandler will

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

2017-05-20 Thread Nicolas Lalevée
Ant Bylaws is a good read to understand the process https://ant.apache.org/bylaws.html Since the initial question is about Ivy only, another vote will be needed for IvyDE. Nicolas > Le 20 mai 2017 à 04:47, J Pai a écrit : > > Not

Re: Ivy - Buildjobs/PreCommit

2017-05-20 Thread Nicolas Lalevée
> Le 20 mai 2017 à 04:45, J Pai a écrit : > > 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