Right, I realize that it's not as easy as "remove" either, because otherwise you just break the ability to use libraries compiled with older versions of Groovy (which would be a pity). So if we go with indy "by default", we have to think what it means in terms of deliverables (a `-legacy` jar?) and what it means for consumers of those artifacts (imagine the number of projects which depend on `groovy-all` today).
2017-05-29 14:56 GMT+02:00 Paul King <pa...@asert.com.au>: > Master is already at Java 8 and will be the basis for Groovy 3 (or > possibly 4 if we bump the 2_6 branch to be Groovy 3). So I think it's safe > to target master for indy only. But agree we need to assess performance > regressions. > > > On 29 May 2017 9:21 am, "Cédric Champeau" <cedric.champ...@gmail.com> > wrote: > >> "Indy by default" doesn't mean anything. It's either "get rid of old call >> site caching/only keep indy" or, "keep as is". >> >> Last time we tentatively started a discussion on what should be in Groovy >> 3 or not, we talked about upgrading to Java 8 minimally. If this happens, >> then we can use indy only and get rid of the "old" call site caching. But I >> also mentioned back then that we MUST perform benchmarks to make sure the >> performance of this version is as good as the previous one. Especially, I >> recall there were performance regressions in some areas (primitive >> handling, for example). >> >> Also, bumping to Java 8 only may be an issue for some. At least, it would >> prevent Gradle from upgrading to Groovy 3. Which may, or may not be an >> issue. >> >> >> 2017-05-29 8:20 GMT+02:00 Mario Garcia <mario.g...@gmail.com>: >> >>> +1 Indy by default >>> >>> Cheers >>> Mario >>> >>> 2017-05-28 14:43 GMT+02:00 Jennifer Strater <jenn.stra...@gmail.com>: >>> >>>> +1 >>>> >>>> Regards, >>>> Jenn >>>> >>>> On Sun, May 28, 2017 at 2:22 PM, Daniel Sun <realblue...@hotmail.com> >>>> wrote: >>>> >>>>> +1 for setting indy by default. >>>>> >>>>> Cheers, >>>>> Daniel.Sun >>>>> >>>>> >>>>> >>>>> -- >>>>> View this message in context: http://groovy.329449.n5.nabble >>>>> .com/Indy-vs-default-build-tp5741393p5741394.html >>>>> Sent from the Groovy Dev mailing list archive at Nabble.com. >>>>> >>>> >>>> >>> >>
