AW: Ivy - Move to symlink creation standard Java API?

2017-07-27 Thread jhm
If the new Java version supports the symlink creation and it's an implementation detail of Ivy (so not part of an API other could use) I am in favour of getting rid of the old implementation. Getting rid of that means being more standard compliant and having less to maintain. ;) Jan >

Re: Ivy - Move to symlink creation standard Java API?

2017-07-27 Thread Gintautas Grigelionis
There might be some issues with Cygwin, see Eclipse/JGit discussion [1] Gintas [1] https://bugs.eclipse.org/bugs/show_bug.cgi?id=354367 2017-07-27 10:44 GMT+02:00 Jan Matèrne (jhm) : > If the new Java version supports the symlink creation and it's an > implementation detail > of Ivy (so not par

Re: Ivy - Move to symlink creation standard Java API?

2017-07-27 Thread Gintautas Grigelionis
I did a few simple tests on Windows 7 + Cygwin and NIO.2 seems to work just fine (if privilege to create symlinks is granted). +1 Gintas 2017-07-27 11:40 GMT+02:00 Gintautas Grigelionis : > There might be some issues with Cygwin, see Eclipse/JGit discussion [1] > > Gintas > > [1] https://bugs.e

[GitHub] ant-ivy pull request #55: use vectorised logo

2017-07-27 Thread twogee
GitHub user twogee opened a pull request: https://github.com/apache/ant-ivy/pull/55 use vectorised logo I'd like to remove downloaded.gif, error.gif, evicted.gif and searched.gif as well. Maybe we could vote on colours for downloaded and evicted 😉 if green is too dark or fuc

Re: Ivy - Move to symlink creation standard Java API?

2017-07-27 Thread Jaikiran Pai
Thank you for testing that kind of setup. I have now pushed a commit which does the necessary changes to use Java standard API and also deprecates that use of symlinkmass option (which is no longer relevant) of the retrieve task. The testcases have been updated to make sure the tests are run o

Re: AW: Ivy - Move to symlink creation standard Java API?

2017-07-27 Thread Jaikiran Pai
On 27/07/17 2:14 PM, Jan Matèrne (jhm) wrote: Getting rid of that means being more standard compliant and having less to maintain. ;) Agreed. Plus the fact that our previous implementation of this feature relied on the presence of the "ln" command on the systems, which effectively ruled out t

[GitHub] ant-ivy pull request #:

2017-07-27 Thread twogee
Github user twogee commented on the pull request: https://github.com/apache/ant-ivy/commit/366fb741f0772c8309f9dfe963f6032b091d7b89#commitcomment-23334637 Perhaps `UnsupportedOperationException` would be a useful heuristic? --- If your project is set up for it, you can reply to this

[GitHub] ant-ivy pull request #:

2017-07-27 Thread jaikiran
Github user jaikiran commented on the pull request: https://github.com/apache/ant-ivy/commit/366fb741f0772c8309f9dfe963f6032b091d7b89#commitcomment-23334838 Do you mean for the `symlinkmass` deprecation? In this release, I just wanted it to be deprecated with a log message showing up

[GitHub] ant-ivy issue #53: Suppressed warnings

2017-07-27 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant-ivy/pull/53 Commit https://github.com/apache/ant-ivy/pull/53/commits/414651c8841a09a9d9d4b0365ee8e7227fba8429 in this PR has now been merged to upstream https://github.com/apache/ant-ivy/commit/1a283ab1be5b495

Is CacheResolver relevant?

2017-07-27 Thread Jaikiran Pai
There's this JIRA issue https://issues.apache.org/jira/browse/IVY-804 where the user reports a genuine bug in our typedef.properties file where we list the "cache" property twice with different values. Once to consider the element as a resolver and the other to consider the element as the cac

Re: Is CacheResolver relevant?

2017-07-27 Thread Jaikiran Pai
On 27/07/17 9:34 PM, Jaikiran Pai wrote: There's a JIRA[2] asking for the docs to be updated to explain this resolver. ... [2] https://issues.apache.org/jira/browse/IVY-804 I meant this JIRA which asks for it to be documented https://issues.apache.org/jira/browse/IVY-612 -Jaikiran

[GitHub] ant-ivy pull request #:

2017-07-27 Thread twogee
Github user twogee commented on the pull request: https://github.com/apache/ant-ivy/commit/6607bdcb3ba69d2ab4f41be6292cc5a6df6365ba#commitcomment-23339716 In URLHandlerDispatcher, ``` @Override public void download(final URL src, final File dest, final CopyProgress

[GitHub] ant-ivy issue #53: Suppressed warnings

2017-07-27 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant-ivy/pull/53 With all due respect, there are some refactorings worth attention IMHO. Besides, some deprecations need to be followed through 😉 --- If your project is set up for it, you can reply to this email

[GitHub] ant-ivy pull request #56: use “Maven 2” consistently

2017-07-27 Thread twogee
GitHub user twogee opened a pull request: https://github.com/apache/ant-ivy/pull/56 use “Maven 2” consistently You can merge this pull request into a Git repository by running: $ git pull https://github.com/twogee/ant-ivy maven-2-in-asciidoc Alternatively you can review a

[GitHub] ant-ivy pull request #57: fix last inconsistencies in generics

2017-07-27 Thread twogee
GitHub user twogee opened a pull request: https://github.com/apache/ant-ivy/pull/57 fix last inconsistencies in generics ... use collections, not arrays You can merge this pull request into a Git repository by running: $ git pull https://github.com/twogee/ant-ivy consistent-gen

[GitHub] ant-ivy pull request #:

2017-07-27 Thread twogee
Github user twogee commented on the pull request: https://github.com/apache/ant-ivy/commit/366fb741f0772c8309f9dfe963f6032b091d7b89#commitcomment-23340291 Sorry for a terse suggestion, I meant we could provide more information to the end user about the symlink/symlinkmass now that the

Re: Ivy logo as SVG

2017-07-27 Thread Gintautas Grigelionis
In case you missed among all GitHub chat: the SVG (and Draw 2D :-) logo has arrived as PR-55. A shoutout to Flamingo project ( https://github.com/kirill-grouchnikov/flamingo) and its SVG transcoder (http://ebourg.github.io/flamingo-svg-transcoder/). Gintas 2017-06-08 16:46 GMT+02:00 Dominique Dev

[GitHub] ant-ivy pull request #56: use “Maven 2” consistently

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

Re: Is CacheResolver relevant?

2017-07-27 Thread Gintautas Grigelionis
Perhaps another JIRA issue [1] would cast some light? [1] https://issues.apache.org/jira/browse/IVY-1227 Gintas 2017-07-27 18:10 GMT+02:00 Jaikiran Pai : > > On 27/07/17 9:34 PM, Jaikiran Pai wrote: > >> There's a JIRA[2] asking for the docs to be updated to explain this >> resolver. >> >> ...

Re: Is CacheResolver relevant?

2017-07-27 Thread Gintautas Grigelionis
Here's a relevant discussion [1] with Xavier himself explaining the reason for cache resolver being there. [1] http://apache-ivy.996301.n3.nabble.com/Does-Ivy-really-have-a-cache-resolver-td3648.html Gintas P.S. It's unrelated, but, while researching, I came across this issue: [1] https://githu