Re: [VOTE] Release Groovy 2.4.9

2017-02-27 Thread Paul King
Hi Daniel, I won't include your vote in the official vote count since it was neither a veto nor a +1 on the proposed artifacts as per Apache voting: http://apache.org/foundation/voting.html But we'll no doubt do a 2.4.10 soon and we can merge the change for GROOVY-8085 into that release. Cheers, P

Re: [VOTE] Release Groovy 2.4.9

2017-02-26 Thread Daniel Sun
Hi Jochen, > I wanted to be sure because 4721 is pretty difficult to fix... I'll try to fix it. Cheers, Daniel.Sun -- View this message in context: http://groovy.329449.n5.nabble.com/VOTE-Release-Groovy-2-4-9-tp5738742p5738784.html Sent from the Groovy Dev mailing list archive at

Re: [VOTE] Release Groovy 2.4.9

2017-02-26 Thread Jim Jagielski
+1! > On Feb 25, 2017, at 7:08 PM, John Wagenleitner > wrote: > > +1 > > On Feb 24, 2017 1:49 AM, "Paul King" wrote: > Dear community, > > I am happy to start the VOTE thread for a Groovy 2.4.9 release! > > This release includes 12 bug fixes/improvements as outlined in the changelog: > https

Re: [VOTE] Release Groovy 2.4.9

2017-02-26 Thread Jochen Theodorou
On 26.02.2017 14:44, Daniel Sun wrote: Doesn't appear to address GROOVY-4721 from my initial check. Yeah, the old issue GROOVY-4721 still exists. I wanted to be sure because 4721 is pretty difficult to fix... well maybe somebody with a fresh mind will find a solution fast I might be too

Re: [VOTE] Release Groovy 2.4.9

2017-02-26 Thread Daniel Sun
> Doesn't appear to address GROOVY-4721 from my initial check. Yeah, the old issue GROOVY-4721 still exists. Cheers, Daniel.Sun -- View this message in context: http://groovy.329449.n5.nabble.com/VOTE-Release-Groovy-2-4-9-tp5738742p5738775.html Sent from the Groovy Dev mailing list archive at

Re: [VOTE] Release Groovy 2.4.9

2017-02-26 Thread Guillaume Laforge
+1 I had some issues with some dependencies, but seems to have been due to some connectivity issues, giving me a bogus dependency or something like that. Now running the build works fine for me. Other than that (and I haven't checked the signature), everything seems to be look alright. Guillaume

Re: [VOTE] Release Groovy 2.4.9

2017-02-26 Thread Paul King
Doesn't appear to address GROOVY-4721 from my initial check. On Sun, Feb 26, 2017 at 7:21 PM, Jochen Theodorou wrote: > On 24.02.2017 11:14, Daniel Sun wrote: >> >> Hi Paul, >> >>If the PR(https://github.com/apache/groovy/pull/504) for fixing >> GROOVY-8085 is merged, +1 from me. > > > Co

Re: [VOTE] Release Groovy 2.4.9

2017-02-26 Thread Daniel Sun
Hi Jochen, I'll check it later :) P.S. I am not beside my computer. Cheers, Daniel.Sun 在 2017年2月26日 下午5:21,"Jochen Theodorou [via Groovy]" 写道:https://github.com/apache/groovy/pull/504) for fixing > GROOVY-8085 is merged, +1 from me. Could you check if this fixes https://issues.apache

Re: [VOTE] Release Groovy 2.4.9

2017-02-26 Thread Jochen Theodorou
On 24.02.2017 11:14, Daniel Sun wrote: Hi Paul, If the PR(https://github.com/apache/groovy/pull/504) for fixing GROOVY-8085 is merged, +1 from me. Could you check if this fixes https://issues.apache.org/jira/browse/GROOVY-4721 ? bye Jochen

Re: [VOTE] Release Groovy 2.4.9

2017-02-25 Thread John Wagenleitner
+1 On Feb 24, 2017 1:49 AM, "Paul King" wrote: Dear community, I am happy to start the VOTE thread for a Groovy 2.4.9 release! This release includes 12 bug fixes/improvements as outlined in the changelog: https://issues.apache.org/jira/secure/ReleaseNote.jspa? projectId=12318123&version=123391

Re: [VOTE] Release Groovy 2.4.9

2017-02-24 Thread Daniel Sun
Hi Paul, If the PR(https://github.com/apache/groovy/pull/504) for fixing GROOVY-8085 is merged, +1 from me. Cheers, Daniel.Sun -- View this message in context: http://groovy.329449.n5.nabble.com/VOTE-Release-Groovy-2-4-9-tp5738742p5738747.html Sent from the Groovy Dev mailing list archi

Re: [VOTE] Release Groovy 2.4.9

2017-02-24 Thread Guillaume Laforge
I had a couple test failures in the JSON module: :groovy-json:test json.StreamingJsonBuilderTest > testStreamingJsonBuilder FAILED org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed: General error during conversion: Error grabbing Grapes -- [download failed: