Re: 3rc2 regression?

2019-12-15 Thread Romain Manni-Bucau
case it tends to show that the bug is likely on groovy side would misload some class. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rm

Re: 3rc2 regression?

2019-12-14 Thread Romain Manni-Bucau
Have to admit I'd still fix it in groovy since it "feels" like a regression. What about dropping ASM_API_VERSION in favor of "4" in org.codehaus.groovy.runtime.callsite.GroovySunClassLoader#loadAbstract ? Romain Manni-Bucau @rmannibucau <https://twitter.com

Re: 3rc2 regression?

2019-12-13 Thread Romain Manni-Bucau
I managed to disable the verify:none option (it seems no more needed on the build now jruby fixed some issues and got upgraded) but can be neat to fix this issue for the rc3 or GA. Thanks for having a look so fast! Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog

Re: 3rc2 regression?

2019-12-13 Thread Romain Manni-Bucau
I'm not yet sure it is a JVM bug or a groovy generated bytecode bug. Concretely rc1 was working fine with verify:none so some change in rc2 triggered that probably. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/>

Re: 3rc2 regression?

2019-12-13 Thread Romain Manni-Bucau
Can you try on java 8 (Azul 1.8 202 in my case, it is available at https://cdn.azul.com/zulu/bin/) please? FYI using azul 11 works for me too so it is likely linked to the JVM version Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.met

Re: 3rc2 regression?

2019-12-13 Thread Romain Manni-Bucau
exactly, simple reproducer: java -Xverify:none -jar ~/.m2/repository/org/codehaus/groovy/groovy/3.0.0-rc-2/groovy-3.0.0-rc-2.jar -e="new File('.').toPath()" Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.me

Re: 3rc2 regression?

2019-12-13 Thread Romain Manni-Bucau
...just need to send a mail to find the cause the jvm uses -Xverify:none (for jruby/asciidoctor), this lead to the issue Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com&g

Re: 3rc2 regression?

2019-12-13 Thread Romain Manni-Bucau
(org.codehaus.groovy.runtime.callsite.CallSite,groovy.lang.MetaClassImpl,groovy.lang.MetaMethod,java.lang.Class[],java.lang.reflect.Constructor) Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpres

3rc2 regression?

2019-12-12 Thread Romain Manni-Bucau
Hi all Upgrading to rc2, one of my build script does not run anymore: https://github.com/Talend/component-runtime/blob/c41da9f67f9fdfa7839c48f58f35fe8c2b678853/sample-parent/documentation-sample/_src/build/Bundles.groovy Here is the stack: https://gist.github.com/rmannibucau/e5b1d485d7e01be128f87

Re: 2.5.7 regression with references?

2019-05-11 Thread Romain Manni-Bucau
Done at https://issues.apache.org/jira/browse/GROOVY-9120 Le sam. 11 mai 2019 à 18:28, Romain Manni-Bucau a écrit : > > > Le sam. 11 mai 2019 à 18:02, Daniel.Sun a écrit : > >> I'm a bit confused because I see the build result of upgrading groovy to >> 2.5.7 is g

Re: 2.5.7 regression with references?

2019-05-11 Thread Romain Manni-Bucau
Le sam. 11 mai 2019 à 18:02, Daniel.Sun a écrit : > I'm a bit confused because I see the build result of upgrading groovy to > 2.5.7 is green: > > > https://travis-ci.org/Talend/component-runtime/builds/530836768?utm_source=github_status&utm_medium=notification I dropped the anonymous class in

Re: 2.5.7 regression with references?

2019-05-10 Thread Romain Manni-Bucau
Sure, the class is there: https://github.com/Talend/component-runtime/blob/component-runtime-1.1.9/sample-parent/documentation-sample/_src/build/Bundles.groovy#L71 and the reference issue is at the line before. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog

2.5.7 regression with references?

2019-05-10 Thread Romain Manni-Bucau
defined before the class (~final in java). Is it known? Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <

get bytecode from groovyclassloader?

2018-10-28 Thread Romain Manni-Bucau
ere else). wdyt? Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://www.linkedin.com/in/rmann

Re: Java 9 support?

2018-09-10 Thread Romain Manni-Bucau
bit more on the steps to get the error you are seeing? > We've mainly been using gradle on JDK11. > > On Tue, Sep 11, 2018 at 6:05 AM Romain Manni-Bucau > wrote: > >> Hi guys, >> >> I'm using gplus maven plugin and I get issues running on java 9: >> >

Java 9 support?

2018-09-10 Thread Romain Manni-Bucau
n(e); } } Any plan to release a j9/10/11 groovy version? [1] https://github.com/apache/groovy/blob/ac0e1d232f0612518a87ce2aa5e443e0975cb3f4/src/main/java/org/codehaus/groovy/vmplugin/v7/Java7.java#L40 Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://

Re: Groovy 3 + JDK9 and problems with our Closure

2018-08-09 Thread Romain Manni-Bucau
Le jeu. 9 août 2018 à 10:49, a écrit : > Why would anyone subclass Closure? Isn't there enough trouble to be > found elsewhere? > It is common in java code integrating with groovy code. > > H2 > > > Den 2018-08-08 20:05, skrev Jochen Theodorou: > > Hi all, > > > > during JCrete I was playing a

Re: Commons cli contribution?

2018-05-28 Thread Romain Manni-Bucau
;> >> Picocli is a young project and it’s difficult to get traction against the >> incumbents that have a large installed base. >> >> It would be great if you could help picocli get visibility in the Java >> community. Your enthusiasm could make a massive differen

Re: Commons cli contribution?

2018-05-28 Thread Romain Manni-Bucau
8 at 5:29 AM, Romain Manni-Bucau > wrote: > >> Do you think groovy community could try to extract it and do a PR to cli >> project? I already did it like 5 times (+ tomitribe-cli work) and would be >> happy to see some convergence here. Cli is a strong base for the parsing &

Re: Commons cli contribution?

2018-05-28 Thread Romain Manni-Bucau
But I may be biased... :-) >> >> Remko >> >> On Tue, May 29, 2018 at 2:06 AM, Romain Manni-Bucau < >> rmannibu...@gmail.com> wrote: >> >>> Hi guys >>> >>> Saw you added some annotations to write a cli in groovy cli, did you try &

Commons cli contribution?

2018-05-28 Thread Romain Manni-Bucau
Hi guys Saw you added some annotations to write a cli in groovy cli, did you try (or do you plan) to propose it to commons cli? It would be an awesome addition which affects more than groovy users IMHO and could unify part of the specific api you created. Romain

Re: MavenGrapeEngine?

2017-01-10 Thread Romain Manni-Bucau
rn())) would make it working. Of course this is doable in groovy but should probably be reported to ivy. Wdyt? Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://blog-rmannibucau.rhcloud.com> | Old Blog <http://rmannibucau.wordpress.com> |

Re: MavenGrapeEngine?

2017-01-10 Thread Romain Manni-Bucau
s.changing : false to boolean changing = deps.containsKey('changing') ? deps.changing || version.endsWith('-SNAPSHOT') : false ? Not an issue if not, would just makes maven standard better integrated with the pitfall of having changing value forced (so less obviou

Re: rm servlet-api.jar?

2017-01-09 Thread Romain Manni-Bucau
Le 10 janv. 2017 01:44, "Jochen Theodorou" a écrit : On 06.01.2017 10:54, Romain Manni-Bucau wrote: > Hi guys, > > Using groovy bundle (export PATH=$GROOVY_HOME/bin:$PATH) I often hit a > blocker: servlet-api is there. It means that if you use tomcat you will > use the

Re: MavenGrapeEngine?

2017-01-06 Thread Romain Manni-Bucau
2017-01-06 15:33 GMT+01:00 Jochen Theodorou : > On 06.01.2017 15:00, Romain Manni-Bucau wrote: > [...] > >> Means SNAPSHOT patterns needs to be identified (likely a flag in the >> @Grab(snapshot=true)?) and groovy should clean up the artifacts before >> resolving to a

Re: MavenGrapeEngine?

2017-01-06 Thread Romain Manni-Bucau
2017-01-06 14:54 GMT+01:00 Jochen Theodorou : > On 06.01.2017 10:45, Romain Manni-Bucau wrote: > [...] > >> I opened https://issues.apache.org/jira/browse/GROOVY-8037 to try to >> enhance it but there are still a few questions to solve before coding: >> >> - does

rm servlet-api.jar?

2017-01-06 Thread Romain Manni-Bucau
/groovy --root-loader={groovy.home}/lib/*.jar myscript.groovy any thoughts on this? Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://blog-rmannibucau.rhcloud.com> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rm

MavenGrapeEngine?

2017-01-06 Thread Romain Manni-Bucau
m but don't think there would be a blocker). Any input welcomed! Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://blog-rmannibucau.rhcloud.com> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | Lin