[GitHub] ant issue #51: [master] Changes for BZ-19516

2017-12-12 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/ant/pull/51 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Ant%20Github-PR-Windows/8/ --- -

[GitHub] ant issue #51: [master] Changes for BZ-19516

2017-12-12 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant/pull/51 >> an alternative would be to only wrap the stream inside of zipFile if mark wasn't supported. @bodewig, I actually liked your idea. So I did an additional change in this commit (PR has been up

[GitHub] ant issue #51: [master] Changes for BZ-19516

2017-12-12 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/ant/pull/51 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Ant%20Github-PR-Linux/1/ --- ---

Re: Jenkins build became unstable: Ant-Build-Matrix-master-Linux » JDK 1.8 (latest) #977

2017-12-12 Thread Jaikiran Pai
Do let me know if there's anything I can do to help investigate this. I'm curious myself to see what's causing this. -Jaikiran On 13/12/17 2:56 AM, Stefan Bodewig wrote: On 2017-12-12, Stefan Bodewig wrote: On 2017-12-12, Jaikiran Pai wrote: So the one final test that's still failing is th

Re: Jenkins build became unstable: Ant-Build-Matrix-master-Linux » JDK 1.8 (latest) #977

2017-12-12 Thread Stefan Bodewig
On 2017-12-12, Stefan Bodewig wrote: > On 2017-12-12, Jaikiran Pai wrote: >> So the one final test that's still failing is this one[1]. It's >> failing consistently only on Jenkins, that too with only Java 8 >> (jdk1.8.0_131). I have tried reproducing that one locally with that >> exact Java vers

Re: Please ignore the github emails for the next hour or so

2017-12-12 Thread Jaikiran Pai
My experiments have completed [1] and any new mails from github/jenkins jobs are genuine mails. Hopefully, the next time I try something around this, it won't take these many spurious mails. [1] https://www.mail-archive.com/dev@ant.apache.org/msg46284.html -Jaikiran On 12/12/17 7:05 PM, Jaik

Github ant repo now integrated with Jenkins for pull request testing

2017-12-12 Thread Jaikiran Pai
We now have integrated our Jenkins build environment hosted by apache.org with our Ant github repo[1], such that whenever a pull request is submitted, it now triggers a build and runs tests against the submitted pull request and then comments/adds a status to the PR on how the build went. This

[GitHub] ant issue #51: [master] Changes for BZ-19516

2017-12-12 Thread bodewig
Github user bodewig commented on the issue: https://github.com/apache/ant/pull/51 an alternative would be to only wrap the stream inside of `zipFile` if `mark` wasn't supported. But I'm fine with the change as it is. Thank you for dusting of really old issues. --- -

[GitHub] ant pull request #38: Do not merge

2017-12-12 Thread jaikiran
Github user jaikiran closed the pull request at: https://github.com/apache/ant/pull/38 --- - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.org

[GitHub] ant issue #51: [master] Changes for BZ-19516

2017-12-12 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/ant/pull/51 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Ant-Master%20Github-PR-Windows/7/ --- --

Re: Jenkins build became unstable: Ant-Build-Matrix-master-Linux » JDK 1.8 (latest) #977

2017-12-12 Thread Stefan Bodewig
On 2017-12-12, Jaikiran Pai wrote: > So the one final test that's still failing is this one[1]. It's > failing consistently only on Jenkins, that too with only Java 8 > (jdk1.8.0_131). I have tried reproducing that one locally with that > exact Java version and even the latest Java 8 and haven't b

[GitHub] ant issue #38: Do not merge

2017-12-12 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/ant/pull/38 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Ant-Master%20Github-PR-Windows/6/ --- --

[GitHub] ant pull request #51: [master] Changes for BZ-19516

2017-12-12 Thread jaikiran
Github user jaikiran closed the pull request at: https://github.com/apache/ant/pull/51 --- - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.org

[GitHub] ant pull request #51: [master] Changes for BZ-19516

2017-12-12 Thread jaikiran
GitHub user jaikiran reopened a pull request: https://github.com/apache/ant/pull/51 [master] Changes for BZ-19516 As suggested in https://bz.apache.org/bugzilla/show_bug.cgi?id=19516, the change in this PR uses `java.io.BufferedInputStream` which can take an underlying `InputStrea

[GitHub] ant issue #38: Do not merge

2017-12-12 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant/pull/38 retest this please --- - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.

Re: Build Script Templates

2017-12-12 Thread Martin Gainty
MG>quick comments below From: Jochen Wiedmann Sent: Tuesday, December 12, 2017 7:54 AM To: dev@ant.apache.org Subject: Build Script Templates Hi, over the last months, I was working in a larger project in a Java environment, but not directly a Java project. Suff

[GitHub] ant issue #38: Do not merge

2017-12-12 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant/pull/38 retest this please --- - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.

[GitHub] ant issue #38: Do not merge

2017-12-12 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/ant/pull/38 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Ant-Master%20Github-PR-Windows/5/ --- --

[GitHub] ant issue #38: Do not merge

2017-12-12 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant/pull/38 retest this please --- - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.

Please ignore the github emails for the next hour or so

2017-12-12 Thread Jaikiran Pai
Sorry about the continuous mails from github/ant repo. I'm (yet again) trying to get the Jenkins job integrated with Ant github repo. I don't have a way to disable these mails while I experiment with it to get it right. Hopefully, it should be sorted out in the next hour or so. -Jaikiran ---

[GitHub] ant issue #51: [master] Changes for BZ-19516

2017-12-12 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/ant/pull/51 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Ant-1.9.x%20Github-PR-Windows/8/ --- ---

[GitHub] ant issue #51: [master] Changes for BZ-19516

2017-12-12 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/ant/pull/51 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Ant-Master%20Github-PR-Windows/4/ --- --

[GitHub] ant issue #38: Do not merge

2017-12-12 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/ant/pull/38 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Ant-Master%20Github-PR-Windows/3/ --- --

[GitHub] ant issue #51: [master] Changes for BZ-19516

2017-12-12 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/ant/pull/51 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Ant-1.9.x%20Github-PR-Windows/7/ --- ---

[GitHub] ant issue #51: [master] Changes for BZ-19516

2017-12-12 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant/pull/51 retest this please --- - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.

[GitHub] ant issue #38: Do not merge

2017-12-12 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/ant/pull/38 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Ant-1.9.x%20Github-PR-Windows/6/ --- ---

[GitHub] ant issue #51: [master] Changes for BZ-19516

2017-12-12 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/ant/pull/51 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Ant-Master%20Github-PR-Windows/2/ --- --

[GitHub] ant issue #51: [master] Changes for BZ-19516

2017-12-12 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant/pull/51 retest this please --- - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.

[GitHub] ant issue #38: Do not merge

2017-12-12 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant/pull/38 retest this please --- - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.

[GitHub] ant issue #38: Do not merge

2017-12-12 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant/pull/38 this is ok to test --- - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.

[GitHub] ant issue #38: Do not merge

2017-12-12 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/ant/pull/38 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail:

[GitHub] ant issue #38: Do not merge

2017-12-12 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant/pull/38 retest this please --- - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.

Build Script Templates

2017-12-12 Thread Jochen Wiedmann
Hi, over the last months, I was working in a larger project in a Java environment, but not directly a Java project. Suffice it to say, that we have a number of Java subprojects, but generally aren't so much interested in Java builds, etc. There's no rule without exceptions, so I've been tasked wi

[GitHub] ant pull request #51: [master] Changes for BZ-19516

2017-12-12 Thread jaikiran
Github user jaikiran closed the pull request at: https://github.com/apache/ant/pull/51 --- - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.org

[GitHub] ant pull request #51: [master] Changes for BZ-19516

2017-12-12 Thread jaikiran
GitHub user jaikiran reopened a pull request: https://github.com/apache/ant/pull/51 [master] Changes for BZ-19516 As suggested in https://bz.apache.org/bugzilla/show_bug.cgi?id=19516, the change in this PR uses `java.io.BufferedInputStream` which can take an underlying `InputStrea

[GitHub] ant pull request #38: Do not merge

2017-12-12 Thread jaikiran
GitHub user jaikiran reopened a pull request: https://github.com/apache/ant/pull/38 Do not merge Testing Jenkins integration. Do not merge this PR You can merge this pull request into a Git repository by running: $ git pull https://github.com/jaikiran/ant dummy Alternatively

[GitHub] ant pull request #51: [master] Changes for BZ-19516

2017-12-12 Thread jaikiran
GitHub user jaikiran opened a pull request: https://github.com/apache/ant/pull/51 [master] Changes for BZ-19516 As suggested in https://bz.apache.org/bugzilla/show_bug.cgi?id=19516, the change in this PR uses `java.io.BufferedInputStream` which can take an underlying `InputStream`

Re: Jenkins build became unstable: Ant-Build-Matrix-master-Linux » JDK 1.8 (latest) #977

2017-12-12 Thread Jaikiran Pai
So the one final test that's still failing is this one[1]. It's failing consistently only on Jenkins, that too with only Java 8 (jdk1.8.0_131). I have tried reproducing that one locally with that exact Java version and even the latest Java 8 and haven't been able to reproduce it. Before I try a