[GitHub] ant issue #54: Let’s use Ivy (properly!) and drop Maven Ant tasks, Commons...

2017-12-27 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant/pull/54 Re password input: isn't there a [task](https://ant.apache.org/manual/Tasks/input.html) with SecureInputHandler for that? --- - To

[GitHub] ant issue #54: Let’s use Ivy (properly!) and drop Maven Ant tasks, Commons...

2017-12-27 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant/pull/54 `buildosxpackage` was peanuts compared to getting rid of `root` system property kludge for surefire in tests. BTW, Ivy `signer` saves you work with 1. signing in advance 2. having to

[GitHub] ant issue #54: Let’s use Ivy (properly!) and drop Maven Ant tasks, Commons...

2017-12-27 Thread bodewig
Github user bodewig commented on the issue: https://github.com/apache/ant/pull/54 I didn't suggest to remove the `buildosxpackage` feature, just that you are putting quite a bit of effort into an area that is probably never used - at least not much. As I said, I don't use

[GitHub] ant pull request #:

2017-12-27 Thread bodewig
Github user bodewig commented on the pull request: https://github.com/apache/ant/commit/d43d83e0198f86c9233631bcb443da132e394d65#commitcomment-26506626 You're welcome. That's why I try to make changes that are target for both branches just to the 1.9.x branch and merge it to

[GitHub] ant pull request #:

2017-12-27 Thread twogee
Github user twogee commented on the pull request: https://github.com/apache/ant/commit/d43d83e0198f86c9233631bcb443da132e394d65#commitcomment-26504265 Thanks for catching this one up, a slip of attention while backporting changes in master. ---

[GitHub] ant issue #54: Let’s use Ivy (properly!) and drop Maven Ant tasks, Commons...

2017-12-27 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant/pull/54 I would like to point out one distinction: I would like to replace the stuff that works (sort of) with **less** stuff that does the same job, and hopefully more. Besides, my beef with Maven Ant tasks was

[GitHub] ant issue #53: Optional libraries for Ant 1.10

2017-12-27 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant/pull/53 I see `dest` as a useful functionality of fetch.xml (why else should it be a part of the distribution?); and that is basically the crux of the matter. I will update the documentation, thanks for

Re: ant git commit: refactored build.xml - made dump-info a dependency of test-init; - removed extraneous dependencies of distribution targets; - turned camelcase and snakecase into kebabcase for cons

2017-12-27 Thread Stefan Bodewig
Please, please, please, please don't mix formatting changes with real changes. This commit contains more that 300 changed lines and it is extremely difficult to see the real changes. Stefan - To unsubscribe, e-mail:

[GitHub] ant issue #54: Let’s use Ivy (properly!) and drop Maven Ant tasks, Commons...

2017-12-27 Thread bodewig
Github user bodewig commented on the issue: https://github.com/apache/ant/pull/54 I don't have any problem with you replacing stuff that works with other stuff that works as well. But before you put too much effort into it, maybe you should look through

[GitHub] ant issue #53: Optional libraries for Ant 1.10

2017-12-27 Thread bodewig
Github user bodewig commented on the issue: https://github.com/apache/ant/pull/53 I see you've already merged this PR, I'll look st it later. I hope you didn't wait for me to respond during Christmas. I'm not sure why `dest` is part of `fetch.xml`, we've always seen at as