You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@groovy.apache.org by GitBox <gi...@apache.org> on 2020/01/31 22:44:53 UTC
[GitHub] [groovy] paulk-asert opened a new pull request #1154: GROOVY-4694: Move AstBuilderTransformation Global xForm to separate m…
paulk-asert opened a new pull request #1154: GROOVY-4694: Move AstBuilderTransformation Global xForm to separate m…
URL: https://github.com/apache/groovy/pull/1154
…odule
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [groovy] paulk-asert commented on issue #1154: GROOVY-4694: Move AstBuilderTransformation Global xForm to separate m…
Posted by GitBox <gi...@apache.org>.
paulk-asert commented on issue #1154: GROOVY-4694: Move AstBuilderTransformation Global xForm to separate m…
URL: https://github.com/apache/groovy/pull/1154#issuecomment-581203472
Strange, I can't reproduce the failure in any other environment.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [groovy] asfgit merged pull request #1154: GROOVY-4694: Move AstBuilderTransformation Global xForm to separate m…
Posted by GitBox <gi...@apache.org>.
asfgit merged pull request #1154: GROOVY-4694: Move AstBuilderTransformation Global xForm to separate m…
URL: https://github.com/apache/groovy/pull/1154
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [groovy] danielsun1106 commented on issue #1154: GROOVY-4694: Move AstBuilderTransformation Global xForm to separate m…
Posted by GitBox <gi...@apache.org>.
danielsun1106 commented on issue #1154: GROOVY-4694: Move AstBuilderTransformation Global xForm to separate m…
URL: https://github.com/apache/groovy/pull/1154#issuecomment-581144935
The following test fails on travis CI:
```
org.codehaus.groovy.ast.LineColumnCheckTest > testLineColumnInfo[Test oneLineDef: Source: Integer onLine = 42
def ii = 17 Expected: [ExpressionStatement,(1:1),(1:20)][ClassNode,(1:1),(1:8)][DeclarationExpression,(1:1),(1:20)];
[VariableExpression,(1:9),(1:15)][ConstantExpression,(1:18),(1:20)];
[ExpressionStatement,(2:1),(2:12)][ClassNode,(-1:-1),(-1:-1)][DeclarationExpression,(2:1),(2:12)];
[VariableExpression,(2:5),(2:7)][ConstantExpression,(2:10),(2:12)]] FAILED
junit.framework.AssertionFailedError: '
[ExpressionStatement,(2:1),(2:12)][ClassNode,(-1:-1),(-1:-1)][DeclarationExpression,(2:1),(2:12)]' not found in '[BlockStatement,(-1:-1),(-1:-1)][ExpressionStatement,(1:1),(1:20)][ClassNode,(1:1),(1:8)][DeclarationExpression,(1:1),(1:20)][VariableExpression,(1:9),(1:15)][ConstantExpression,(1:18),(1:20)][ExpressionStatement,(2:1),(2:12)][ClassNode,(2:17),(2:40)][DeclarationExpression,(2:1),(2:12)][VariableExpression,(2:5),(2:7)][ConstantExpression,(2:10),(2:12)]'
at junit.framework.Assert.fail(Assert.java:57)
at junit.framework.Assert.assertTrue(Assert.java:22)
at junit.framework.TestCase.assertTrue(TestCase.java:192)
at jdk.internal.reflect.GeneratedMethodAccessor90.invoke(Unknown Source)
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.base/java.lang.reflect.Method.invoke(Method.java:566)
at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:107)
at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:323)
at groovy.lang.MetaClassImpl.invokeStaticMethod(MetaClassImpl.java:1532)
at org.codehaus.groovy.runtime.callsite.StaticMetaClassSite.callStatic(StaticMetaClassSite.java:62)
at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callStatic(AbstractCallSite.java:216)
at org.codehaus.groovy.ast.LineColumnCheckTest.testLineColumnInfo(LineColumnCheckTest.groovy:141)
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [groovy] asfgit merged pull request #1154: GROOVY-4694: Move AstBuilderTransformation Global xForm to separate m…
Posted by GitBox <gi...@apache.org>.
asfgit merged pull request #1154: GROOVY-4694: Move AstBuilderTransformation Global xForm to separate m…
URL: https://github.com/apache/groovy/pull/1154
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services