[GitHub] ant-ivy issue #34: Fix IVY-1528

2017-05-30 Thread janmaterne
Github user janmaterne commented on the issue: https://github.com/apache/ant-ivy/pull/34 It's ok. What's not ok is my own environmen: I can see your changes on PomModuleDescriptorWriter.java, but I am missing all new files. I used git fetch github pull/34/head:PR-34 git ch

[GitHub] ant-ivy issue #34: Fix IVY-1528

2017-05-30 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant-ivy/pull/34 I used >> git fetch github pull/34/head:PR-34 >> git checkout PR-34 >> Some ideas? What does the output of: ```git status``` and then ```git show``

[GitHub] ant-ivy issue #36: Replace emma with jacoco

2017-05-30 Thread janmaterne
Github user janmaterne commented on the issue: https://github.com/apache/ant-ivy/pull/36 The JaCoCo 'logfile' seems to be a binary one. So it should not be named '*.log'. Maybe 'jacoco-log.bin'? In 'test-report' the should be the last statement, so we get the report created.

[GitHub] ant-ivy issue #34: Fix IVY-1528

2017-05-30 Thread janmaterne
Github user janmaterne commented on the issue: https://github.com/apache/ant-ivy/pull/34 Thanks. It was the most common error: the user error :) Used the GUI "git history" not on the top folder ... --- If your project is set up for it, you can reply to this email and have your rep

[GitHub] ant-ivy pull request #34: Fix IVY-1528

2017-05-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/ant-ivy/pull/34 --- 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

[GitHub] ant-ivy issue #36: Replace emma with jacoco

2017-05-30 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant-ivy/pull/36 By default, jacoco "log" gets extension ".exec"; I'd rather call it ".data". I can amend the PR later tonight. Could you please check your classpath for multiple instances of asm? jacoco need

[VOTE] Release Compress Antlib 1.5 based on RC1

2017-05-30 Thread Stefan Bodewig
Hi all I've created a release candidate for Compress Antlib 1.5. This catches up with Commons Compress 1.14 and thus adds write support for LZMA and Snappy, full support for LZ4 and read-only support for Brotli. git tag: 1_5_RC1 on commit: 84e2f9b tarballs: https://dist.apache.org/repos/

Problems Releaseing Antlibs

2017-05-30 Thread Stefan Bodewig
Hi all its kind of obvious we haven't cut any releases of Antlibs for a long time. About three years ago the build process has been changed to use Ivy to retrieve dependencies and ever since then the release process has been broken. I've fixed it well enough to be able to cut a release candidate

Re: Problems Releaseing Antlibs

2017-05-30 Thread Gintautas Grigelionis
Hi, ivy:makepom doesn't do its job? > missing artifact Apache Ant#ant-compress;1.5!ant-compress.pom Gintas 2017-05-30 20:36 GMT+02:00 Stefan Bodewig : > Hi all > > its kind of obvious we haven't cut any releases of Antlibs for a long > time. About three years ago the build process has been cha

[GitHub] ant-ivy issue #36: Replace emma with jacoco

2017-05-30 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant-ivy/pull/36 JaCoCo "log" gets an extension ".data"; JaCoCo reports are generated when JUnit fails; Checkstyle rules sync'ed with Ant. Anything else? I found out that JUnit tests polute run.clas

Re: Problems Releaseing Antlibs

2017-05-30 Thread J Pai
I had a brief look at this project’s repo and triggered these tasks locally. Of course, I won’t be able to test the whole upload process, but I was able to see what’s wrong. Essentially, the build targets are all fine and they are generating the necessary files, including the pom file. It’s jus

Re: Problems Releaseing Antlibs

2017-05-30 Thread J Pai
By the way, a bit unrelated to the issue at hand - currently the Ivy version being used is 2.4.0-rc1 [1]. Using 2.4.0 instead might be a good idea since it’s been stable for more than a couple of years now. [1] https://github.com/apache/ant-antlibs-common/blob/master/build.properties#L20 -Jaik