paulk-asert commented on PR #2451:
URL: https://github.com/apache/groovy/pull/2451#issuecomment-4220575281

   Your fix seems to also work. Without the fix I get:
   
   ```
     Caused by: java.lang.AssertionError
           at groovyjarjarasm.asm.Frame.putAbstractType(Frame.java:1482)
           at 
groovyjarjarasm.asm.MethodWriter.putAbstractTypes(MethodWriter.java:1961)
           at groovyjarjarasm.asm.MethodWriter.putFrame(MethodWriter.java:1920)
           at 
groovyjarjarasm.asm.MethodWriter.visitFrameEnd(MethodWriter.java:1852)
           at groovyjarjarasm.asm.Frame.accept(Frame.java:1405)
           at 
groovyjarjarasm.asm.MethodWriter.computeAllFrames(MethodWriter.java:1630)
           at groovyjarjarasm.asm.MethodWriter.visitMaxs(MethodWriter.java:1548)
           at 
org.codehaus.groovy.classgen.AsmClassGenerator.visitConstructorOrMethod(AsmClassGenerator.java:624)
           ... 44 more
   ```
           
   With the fix, the following command works (I didn't try the full build):
   
       $ ./gradlew :grails-geb:compileTestFixturesGroovy
   
   The test task also completes successfully on GROOVY_5_0_X with your fix in 
place. I am happy to close my proposed fix unless you think there is more to 
check.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to