twogee
twogee
The methods in `AbstractResolver` must be defined the other way around: the default implementation of the method with new signature should call the method with the old signature. May I...
The same applies to `ChainResolver` (which should check resolver type as suggested by @jaikiran); actually, all custom resolvers that do not extend `AbstractResolver` could be wrapped in the chain resolver...
Is this sufficient, or should I add an extra check in `ChainResolver` ``` if (resolver instanceof AbstractResolver || hasGenericMethod(resolver)) { ... } ... private boolean hasGenericMethod(DependencyResolver resolver) { try {...
Thanks for clarifying. My point was that the use of the method is limited to two classes: `SearchEngine` and `ChainResolver` (because it is essentially a proxy). A quick search seems...
@nlalevee, we already broke a few things by going Java 7 (and gained native symlinks 😉). For more examples, look at commons-lang3 breaking off commons-text and introducing escapeHtml3 and escapeHtml4...
Could you please provide a unit test to demonstrate what is wrong with [IVY-1576 fix](https://github.com/apache/ant-ivy/commit/3b17df0ab67a1f14794bc6c60b735f3354caec9e)?
I suggest closing this PR. The main problem here is that programmatic use should not be adressed by adding CLI options which make no sense. Also, if resolve() ends with...
Issue tracker for the project is [JIRA](https://issues.apache.org/jira). Apache has own Git infrastructure, GitHub is used to increase exposure and as easy means to suggest code changes (rather than uploading patches)....
@supertick FYI, Ivy 2.5.0 RC is on the way, @nlalevee has kindly taken the role of release manager. Your latest commits are not merged to master, though. If you are...
Check out the [nightlies](https://ant.apache.org/nightlies.html) I was commenting here because @supertick is apparently pushing changes to 2.4.x release branch, but the RC is from the master.