You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@groovy.apache.org by daniel_sun <re...@hotmail.com> on 2016/10/06 10:44:52 UTC
答复: 答复: [PROGRESS REPORT 20161004]groovy-parser
Hi Graeme,
Thanks for your help! I'll give it a try :)
PS: The new parser requires Java8 because streams and lambda expression are used.
Cheers,
Daniel.Sun
________________________________
发件人: Graeme Rocher-2 [via Groovy] <ml...@n5.nabble.com>
发送时间: 2016年10月6日 18:27
收件人: daniel_sun
主题: Re: 答复: [PROGRESS REPORT 20161004]groovy-parser
Hi Daniel,
Yes I have seen that before, it happens when Travis runs out of memory
since each Travis build is limited by memory. Try split up the steps
that compile the code with the steps that run the tests and also avoid
using the Gradle daemon:
./gradlew compileGroovy --no-daemon
./gradlew compileTestGroovy --no-daemon
./gradlew test --no-daemon
Also in my experience Java 8 is more memory intensive than Java 7, so
if you can use Java 7 to build that often helps.
Cheers
On Thu, Oct 6, 2016 at 12:16 PM, daniel_sun <[hidden email]</user/SendEmail.jtp?type=node&node=5735833&i=0>> wrote:
> Hi Graeme,
>
>
> The last test result is shown as following:
>
> https://travis-ci.org/danielsun1106/groovy/builds/165263810
>
> 13395 tests completed, 3 failed, 9 skipped
>
> Currently I'm trying to solve the following issue, which blocks me to
> verify issues. It's pity that I'm not very familiar with gradle and travis
> CI, so I've asked Cédric and Paul for help. BTW, I can run test cases
> without any problem on my local machine.
>
> [org.gradle.api.tasks.TaskExecutionException: Execution failed for task
> ':groovy-ant:compileTestGroovy']
> org.gradle.process.internal.worker.WorkerProcessException: Failed to run
> Gradle Compiler Daemon
> https://github.com/danielsun1106/groovy/issues/4
>
>
> Graeme, did you encounter some similiar problem like the above ever?
>
> Thanks,
> Daniel.Sun
> ________________________________
> 发件人: Graeme Rocher-2 [via Groovy] <ml-node+[hidden email]>
> 发送时间: 2016年10月6日 17:29
> 收件人: daniel_sun
> 主题: Re: [PROGRESS REPORT 20161004]groovy-parser
>
> That is awesome, keep up the good work.
>
> Cheers
>
> On Wed, Oct 5, 2016 at 5:57 AM, daniel_sun <[hidden email]> wrote:
>> Thanks :)
>>
>>
>>
>> --
>> View this message in context:
>> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735778.html
>> Sent from the Groovy Dev mailing list archive at Nabble.com.
>
>
>
> --
> Graeme Rocher
>
>
> ________________________________
> If you reply to this email, your message will be added to the discussion
> below:
> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735829.html
> To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here.
> NAML
>
> ________________________________
> View this message in context: 答复: [PROGRESS REPORT 20161004]groovy-parser
>
> Sent from the Groovy Dev mailing list archive at Nabble.com.
--
Graeme Rocher
________________________________
If you reply to this email, your message will be added to the discussion below:
http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735833.html
To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=5735725&code=cmVhbGJsdWVzdW5AaG90bWFpbC5jb218NTczNTcyNXwxMTQ2MjE4MjI1>.
NAML<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml>
--
View this message in context: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735834.html
Sent from the Groovy Dev mailing list archive at Nabble.com.
Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser
Posted by daniel_sun <re...@hotmail.com>.
IMO, when the new parser passes all test cases of groovy project, that says it conforms to GLS. All groovy program should conform to GLS, shouldn't them? :-) But we can modify GLS if Groovy core team approves ;-)
在 "Thibault Kruse [via Groovy]" <ml...@n5.nabble.com>，2016年10月9日 13:55写道：
Should the new parser also pass tests for the most-used frameworks
before being merged? Grails, Gradle, Spock and Geb come to my mind.
On Fri, Oct 7, 2016 at 8:40 PM, Jochen Theodorou <[hidden email]> wrote:
>
>
> On 07.10.2016 13:35, daniel_sun wrote:
>>
>> I see.
>>
>> Feel free to merge the new parser back when you think it is mature and
>> ready to go home ;) Currently The parser is lack of some syntax
>> checks(e.g. continue should only be used in loop) and friendly messages
>> for developers.
>
>
> Before that I would like the parser to be able to parse the same files and
> have no test cases failing. Because as part of the merge we need to
> integrate the parser in the test cycle to avoid any problems if we change
> the syntax after all. I know we said we would not, but it happened at least
> 3(?) this year.
>
> bye Jochen
________________________________
If you reply to this email, your message will be added to the discussion below:
http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735940.html
To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=5735725&code=cmVhbGJsdWVzdW5AaG90bWFpbC5jb218NTczNTcyNXwxMTQ2MjE4MjI1>.
NAML<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml>
--
View this message in context: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735941.html
Sent from the Groovy Dev mailing list archive at Nabble.com.
Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser
Posted by daniel_sun <re...@hotmail.com>.
NO... I did not see them in the failing test list when using travis ci. It is lucky that Sergei had locate the cause of the failing tests and made a PR as follows:
https://github.com/apache/groovy/pull/443
在 "Jochen Theodorou [via Groovy]" <ml...@n5.nabble.com>，2016年10月9日 15:03写道：
On 07.10.2016 18:09, daniel_sun wrote:
> Jochen, you can see the test result at:
> http://ci.groovy-lang.org/viewType.html?buildTypeId=Groovy_Antlr4grammar
thanks... but there are 3 tests failing. Are those the test you have a
local fix for?
bye Jochen
________________________________
If you reply to this email, your message will be added to the discussion below:
http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735942.html
To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=5735725&code=cmVhbGJsdWVzdW5AaG90bWFpbC5jb218NTczNTcyNXwxMTQ2MjE4MjI1>.
NAML<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml>
--
View this message in context: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735943.html
Sent from the Groovy Dev mailing list archive at Nabble.com.
Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser
Posted by Jochen Theodorou <bl...@gmx.org>.
On 07.10.2016 18:09, daniel_sun wrote:
> Jochen, you can see the test result at:
> http://ci.groovy-lang.org/viewType.html?buildTypeId=Groovy_Antlr4grammar
thanks... but there are 3 tests failing. Are those the test you have a
local fix for?
bye Jochen
Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser
Posted by daniel_sun <re...@hotmail.com>.
Jochen, you can see the test result at: http://ci.groovy-lang.org/viewType.html?buildTypeId=Groovy_Antlr4grammar
在 "Jochen Theodorou [via Groovy]" <ml...@n5.nabble.com>，2016年10月7日 下午7:40写道：
On 07.10.2016 13:35, daniel_sun wrote:
> I see.
>
> Feel free to merge the new parser back when you think it is mature and
> ready to go home ;) Currently The parser is lack of some syntax
> checks(e.g. continue should only be used in loop) and friendly messages
> for developers.
Before that I would like the parser to be able to parse the same files
and have no test cases failing. Because as part of the merge we need to
integrate the parser in the test cycle to avoid any problems if we
change the syntax after all. I know we said we would not, but it
happened at least 3(?) this year.
bye Jochen
________________________________
If you reply to this email, your message will be added to the discussion below:
http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735891.html
To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=5735725&code=cmVhbGJsdWVzdW5AaG90bWFpbC5jb218NTczNTcyNXwxMTQ2MjE4MjI1>.
NAML<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml>
--
View this message in context: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735907.html
Sent from the Groovy Dev mailing list archive at Nabble.com.
Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser
Posted by daniel_sun <re...@hotmail.com>.
OK. The 3 remaining failed test cases of last CI have been fixed on my local machine. It's pity that travis ci fails to run test cases recently. So I'm applying a new CI instance :)
Cheers,
Daniel.Sun
在 "Jochen Theodorou [via Groovy]" <ml...@n5.nabble.com>，2016年10月7日 19:40写道：
On 07.10.2016 13:35, daniel_sun wrote:
> I see.
>
> Feel free to merge the new parser back when you think it is mature and
> ready to go home ;) Currently The parser is lack of some syntax
> checks(e.g. continue should only be used in loop) and friendly messages
> for developers.
Before that I would like the parser to be able to parse the same files
and have no test cases failing. Because as part of the merge we need to
integrate the parser in the test cycle to avoid any problems if we
change the syntax after all. I know we said we would not, but it
happened at least 3(?) this year.
bye Jochen
________________________________
If you reply to this email, your message will be added to the discussion below:
http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735891.html
To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=5735725&code=cmVhbGJsdWVzdW5AaG90bWFpbC5jb218NTczNTcyNXwxMTQ2MjE4MjI1>.
NAML<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml>
--
View this message in context: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735894.html
Sent from the Groovy Dev mailing list archive at Nabble.com.
Re: [PROGRESS REPORT 20161004]groovy-parser
Posted by Guillaume Laforge <gl...@gmail.com>.
I agree we should also try the new parser on a handful of other popular
projects, as it'd ensure the parser can parse even more of Groovy that is
in the wild.
Le dimanche 9 octobre 2016, Thibault Kruse <ti...@googlemail.com> a
écrit :
> Should the new parser also pass tests for the most-used frameworks
> before being merged? Grails, Gradle, Spock and Geb come to my mind.
>
> On Fri, Oct 7, 2016 at 8:40 PM, Jochen Theodorou <blackdrag@gmx.org
> <javascript:;>> wrote:
> >
> >
> > On 07.10.2016 13:35, daniel_sun wrote:
> >>
> >> I see.
> >>
> >> Feel free to merge the new parser back when you think it is mature and
> >> ready to go home ;) Currently The parser is lack of some syntax
> >> checks(e.g. continue should only be used in loop) and friendly messages
> >> for developers.
> >
> >
> > Before that I would like the parser to be able to parse the same files
> and
> > have no test cases failing. Because as part of the merge we need to
> > integrate the parser in the test cycle to avoid any problems if we change
> > the syntax after all. I know we said we would not, but it happened at
> least
> > 3(?) this year.
> >
> > bye Jochen
>
--
Guillaume Laforge
Apache Groovy committer & PMC Vice-President
Developer Advocate @ Google Cloud Platform
Blog: http://glaforge.appspot.com/
Social: @glaforge <http://twitter.com/glaforge> / Google+
<https://plus.google.com/u/0/114130972232398734985/posts>
Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser
Posted by Thibault Kruse <ti...@googlemail.com>.
Should the new parser also pass tests for the most-used frameworks
before being merged? Grails, Gradle, Spock and Geb come to my mind.
On Fri, Oct 7, 2016 at 8:40 PM, Jochen Theodorou <bl...@gmx.org> wrote:
>
>
> On 07.10.2016 13:35, daniel_sun wrote:
>>
>> I see.
>>
>> Feel free to merge the new parser back when you think it is mature and
>> ready to go home ;) Currently The parser is lack of some syntax
>> checks(e.g. continue should only be used in loop) and friendly messages
>> for developers.
>
>
> Before that I would like the parser to be able to parse the same files and
> have no test cases failing. Because as part of the merge we need to
> integrate the parser in the test cycle to avoid any problems if we change
> the syntax after all. I know we said we would not, but it happened at least
> 3(?) this year.
>
> bye Jochen
Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser
Posted by Jochen Theodorou <bl...@gmx.org>.
On 07.10.2016 13:35, daniel_sun wrote:
> I see.
>
> Feel free to merge the new parser back when you think it is mature and
> ready to go home ;) Currently The parser is lack of some syntax
> checks(e.g. continue should only be used in loop) and friendly messages
> for developers.
Before that I would like the parser to be able to parse the same files
and have no test cases failing. Because as part of the merge we need to
integrate the parser in the test cycle to avoid any problems if we
change the syntax after all. I know we said we would not, but it
happened at least 3(?) this year.
bye Jochen
Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser
Posted by daniel_sun <re...@hotmail.com>.
I see.
Feel free to merge the new parser back when you think it is mature and ready to go home ;) Currently The parser is lack of some syntax checks(e.g. continue should only be used in loop) and friendly messages for developers.
Cheers,
Daniel.Sun
在 "Jochen Theodorou [via Groovy]" <ml...@n5.nabble.com>，2016年10月7日 19:03写道：
On 07.10.2016 08:52, daniel_sun wrote:
> How about providing a jar file containing the new parser, which is
> compiled by Java8. When developers use Java8, they can try the new
> parser by turning on the switch.
I think for the time being we could make the new parser just a
subproject that depends on main. Then this subproject can be java8,
while the remainder of the project is java7. We need to
ressurect/repair/activate the antlrparser plugin mechanism, but
otherwise there should be no big problem.... Of course java7 would then
not be able to use the new parser. WE can still think about porting or
not, once we have it merged back.
bye Jochen
________________________________
If you reply to this email, your message will be added to the discussion below:
http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735888.html
To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=5735725&code=cmVhbGJsdWVzdW5AaG90bWFpbC5jb218NTczNTcyNXwxMTQ2MjE4MjI1>.
NAML<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml>
--
View this message in context: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735890.html
Sent from the Groovy Dev mailing list archive at Nabble.com.
Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser
Posted by Jochen Theodorou <bl...@gmx.org>.
On 07.10.2016 08:52, daniel_sun wrote:
> How about providing a jar file containing the new parser, which is
> compiled by Java8. When developers use Java8, they can try the new
> parser by turning on the switch.
I think for the time being we could make the new parser just a
subproject that depends on main. Then this subproject can be java8,
while the remainder of the project is java7. We need to
ressurect/repair/activate the antlrparser plugin mechanism, but
otherwise there should be no big problem.... Of course java7 would then
not be able to use the new parser. WE can still think about porting or
not, once we have it merged back.
bye Jochen
Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser
Posted by daniel_sun <re...@hotmail.com>.
How about providing a jar file containing the new parser, which is compiled by Java8. When developers use Java8, they can try the new parser by turning on the switch.
Cheers,
Daniel.Sun
在 "paulk_asert [via Groovy]" <ml...@n5.nabble.com>，2016年10月7日 上午4:55写道：
I didn't realise Java 8 was a requirement. I was hoping we could maybe
even sneak the new parser into Groovy 2.5 (but not turned on by
default) since we have been delayed so long and the parser is coming
along nicely.
Given the Java 8 requirement, that might be a bit more work than we
originally planned - perhaps a plugin-style mechanism would be
required?
I think 3.0 was the target for Java8+. We would certainly have the new
parser there and probably switched on by default - with a switch to
fall back (only if we absolutely need it). I don't know what others
think.
Paul.
On Thu, Oct 6, 2016 at 10:39 PM, daniel_sun <[hidden email]> wrote:
> As far as I know, Groovy will migrate to Java8 as soon as possible, so it
> will be OK in the near future ;)
>
>
>
> 在 "Graeme Rocher-2 [via Groovy]" <ml-node+[hidden email]>，2016年10月6日
> 下午8:19写道：
>
> Oh, it could be a problem dropping Java 7 support if this were ever
> contributed to Groovy core
>
> Cheers
>
> On Thu, Oct 6, 2016 at 12:44 PM, daniel_sun <[hidden email]> wrote:
>
>> Hi Graeme,
>>
>>
>> Thanks for your help! I'll give it a try :)
>>
>>
>> PS: The new parser requires Java8 because streams and lambda expression
>> are
>> used.
>>
>>
>> Cheers,
>> Daniel.Sun
>>
>> ________________________________
>> 发件人: Graeme Rocher-2 [via Groovy] <ml-node+[hidden email]>
>> 发送时间: 2016年10月6日 18:27
>> 收件人: daniel_sun
>> 主题: Re: 答复: [PROGRESS REPORT 20161004]groovy-parser
>>
>> Hi Daniel,
>>
>> Yes I have seen that before, it happens when Travis runs out of memory
>> since each Travis build is limited by memory. Try split up the steps
>> that compile the code with the steps that run the tests and also avoid
>> using the Gradle daemon:
>>
>> ./gradlew compileGroovy --no-daemon
>> ./gradlew compileTestGroovy --no-daemon
>> ./gradlew test --no-daemon
>>
>> Also in my experience Java 8 is more memory intensive than Java 7, so
>> if you can use Java 7 to build that often helps.
>>
>> Cheers
>>
>> On Thu, Oct 6, 2016 at 12:16 PM, daniel_sun <[hidden email]> wrote:
>>
>>> Hi Graeme,
>>>
>>>
>>> The last test result is shown as following:
>>>
>>> https://travis-ci.org/danielsun1106/groovy/builds/165263810
>>>
>>> 13395 tests completed, 3 failed, 9 skipped
>>>
>>> Currently I'm trying to solve the following issue, which blocks me
>>> to
>>> verify issues. It's pity that I'm not very familiar with gradle and
>>> travis
>>> CI, so I've asked Cédric and Paul for help. BTW, I can run test cases
>>> without any problem on my local machine.
>>>
>>> [org.gradle.api.tasks.TaskExecutionException: Execution failed for task
>>> ':groovy-ant:compileTestGroovy']
>>> org.gradle.process.internal.worker.WorkerProcessException: Failed to run
>>> Gradle Compiler Daemon
>>> https://github.com/danielsun1106/groovy/issues/4
>>>
>>>
>>> Graeme, did you encounter some similiar problem like the above
>>> ever?
>>>
>>> Thanks,
>>> Daniel.Sun
>>> ________________________________
>>> 发件人: Graeme Rocher-2 [via Groovy] <ml-node+[hidden email]>
>>> 发送时间: 2016年10月6日 17:29
>>> 收件人: daniel_sun
>>> 主题: Re: [PROGRESS REPORT 20161004]groovy-parser
>>>
>>> That is awesome, keep up the good work.
>>>
>>> Cheers
>>>
>>> On Wed, Oct 5, 2016 at 5:57 AM, daniel_sun <[hidden email]> wrote:
>>>> Thanks :)
>>>>
>>>>
>>>>
>>>> --
>>>> View this message in context:
>>>>
>>>>
>>>> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735778.html
>>>> Sent from the Groovy Dev mailing list archive at Nabble.com.
>>>
>>>
>>>
>>> --
>>> Graeme Rocher
>>>
>>>
>>> ________________________________
>>> If you reply to this email, your message will be added to the discussion
>>> below:
>>>
>>>
>>> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735829.html
>>> To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here.
>>> NAML
>>>
>>> ________________________________
>>> View this message in context: 答复: [PROGRESS REPORT 20161004]groovy-parser
>>>
>>> Sent from the Groovy Dev mailing list archive at Nabble.com.
>>
>>
>>
>> --
>> Graeme Rocher
>>
>>
>> ________________________________
>> If you reply to this email, your message will be added to the discussion
>> below:
>>
>> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735833.html
>> To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here.
>> NAML
>>
>> ________________________________
>> View this message in context: 答复: 答复: [PROGRESS REPORT
>> 20161004]groovy-parser
>>
>> Sent from the Groovy Dev mailing list archive at Nabble.com.
>
>
>
> --
> Graeme Rocher
>
>
> ________________________________
> If you reply to this email, your message will be added to the discussion
> below:
> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735842.html
> To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here.
> NAML
>
>
> ________________________________
> View this message in context: Re: 答复: 答复: [PROGRESS REPORT
> 20161004]groovy-parser
>
> Sent from the Groovy Dev mailing list archive at Nabble.com.
________________________________
If you reply to this email, your message will be added to the discussion below:
http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735850.html
To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=5735725&code=cmVhbGJsdWVzdW5AaG90bWFpbC5jb218NTczNTcyNXwxMTQ2MjE4MjI1>.
NAML<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml>
--
View this message in context: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735879.html
Sent from the Groovy Dev mailing list archive at Nabble.com.
Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser
Posted by Paul King <pa...@asert.com.au>.
I didn't realise Java 8 was a requirement. I was hoping we could maybe
even sneak the new parser into Groovy 2.5 (but not turned on by
default) since we have been delayed so long and the parser is coming
along nicely.
Given the Java 8 requirement, that might be a bit more work than we
originally planned - perhaps a plugin-style mechanism would be
required?
I think 3.0 was the target for Java8+. We would certainly have the new
parser there and probably switched on by default - with a switch to
fall back (only if we absolutely need it). I don't know what others
think.
Paul.
On Thu, Oct 6, 2016 at 10:39 PM, daniel_sun <re...@hotmail.com> wrote:
> As far as I know, Groovy will migrate to Java8 as soon as possible, so it
> will be OK in the near future ;)
>
>
>
> 在 "Graeme Rocher-2 [via Groovy]" <ml-node+[hidden email]>，2016年10月6日
> 下午8:19写道：
>
> Oh, it could be a problem dropping Java 7 support if this were ever
> contributed to Groovy core
>
> Cheers
>
> On Thu, Oct 6, 2016 at 12:44 PM, daniel_sun <[hidden email]> wrote:
>
>> Hi Graeme,
>>
>>
>> Thanks for your help! I'll give it a try :)
>>
>>
>> PS: The new parser requires Java8 because streams and lambda expression
>> are
>> used.
>>
>>
>> Cheers,
>> Daniel.Sun
>>
>> ________________________________
>> 发件人: Graeme Rocher-2 [via Groovy] <ml-node+[hidden email]>
>> 发送时间: 2016年10月6日 18:27
>> 收件人: daniel_sun
>> 主题: Re: 答复: [PROGRESS REPORT 20161004]groovy-parser
>>
>> Hi Daniel,
>>
>> Yes I have seen that before, it happens when Travis runs out of memory
>> since each Travis build is limited by memory. Try split up the steps
>> that compile the code with the steps that run the tests and also avoid
>> using the Gradle daemon:
>>
>> ./gradlew compileGroovy --no-daemon
>> ./gradlew compileTestGroovy --no-daemon
>> ./gradlew test --no-daemon
>>
>> Also in my experience Java 8 is more memory intensive than Java 7, so
>> if you can use Java 7 to build that often helps.
>>
>> Cheers
>>
>> On Thu, Oct 6, 2016 at 12:16 PM, daniel_sun <[hidden email]> wrote:
>>
>>> Hi Graeme,
>>>
>>>
>>> The last test result is shown as following:
>>>
>>> https://travis-ci.org/danielsun1106/groovy/builds/165263810
>>>
>>> 13395 tests completed, 3 failed, 9 skipped
>>>
>>> Currently I'm trying to solve the following issue, which blocks me
>>> to
>>> verify issues. It's pity that I'm not very familiar with gradle and
>>> travis
>>> CI, so I've asked Cédric and Paul for help. BTW, I can run test cases
>>> without any problem on my local machine.
>>>
>>> [org.gradle.api.tasks.TaskExecutionException: Execution failed for task
>>> ':groovy-ant:compileTestGroovy']
>>> org.gradle.process.internal.worker.WorkerProcessException: Failed to run
>>> Gradle Compiler Daemon
>>> https://github.com/danielsun1106/groovy/issues/4
>>>
>>>
>>> Graeme, did you encounter some similiar problem like the above
>>> ever?
>>>
>>> Thanks,
>>> Daniel.Sun
>>> ________________________________
>>> 发件人: Graeme Rocher-2 [via Groovy] <ml-node+[hidden email]>
>>> 发送时间: 2016年10月6日 17:29
>>> 收件人: daniel_sun
>>> 主题: Re: [PROGRESS REPORT 20161004]groovy-parser
>>>
>>> That is awesome, keep up the good work.
>>>
>>> Cheers
>>>
>>> On Wed, Oct 5, 2016 at 5:57 AM, daniel_sun <[hidden email]> wrote:
>>>> Thanks :)
>>>>
>>>>
>>>>
>>>> --
>>>> View this message in context:
>>>>
>>>>
>>>> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735778.html
>>>> Sent from the Groovy Dev mailing list archive at Nabble.com.
>>>
>>>
>>>
>>> --
>>> Graeme Rocher
>>>
>>>
>>> ________________________________
>>> If you reply to this email, your message will be added to the discussion
>>> below:
>>>
>>>
>>> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735829.html
>>> To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here.
>>> NAML
>>>
>>> ________________________________
>>> View this message in context: 答复: [PROGRESS REPORT 20161004]groovy-parser
>>>
>>> Sent from the Groovy Dev mailing list archive at Nabble.com.
>>
>>
>>
>> --
>> Graeme Rocher
>>
>>
>> ________________________________
>> If you reply to this email, your message will be added to the discussion
>> below:
>>
>> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735833.html
>> To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here.
>> NAML
>>
>> ________________________________
>> View this message in context: 答复: 答复: [PROGRESS REPORT
>> 20161004]groovy-parser
>>
>> Sent from the Groovy Dev mailing list archive at Nabble.com.
>
>
>
> --
> Graeme Rocher
>
>
> ________________________________
> If you reply to this email, your message will be added to the discussion
> below:
> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735842.html
> To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here.
> NAML
>
>
> ________________________________
> View this message in context: Re: 答复: 答复: [PROGRESS REPORT
> 20161004]groovy-parser
>
> Sent from the Groovy Dev mailing list archive at Nabble.com.
Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser
Posted by daniel_sun <re...@hotmail.com>.
As far as I know, Groovy will migrate to Java8 as soon as possible, so it will be OK in the near future ;)
在 "Graeme Rocher-2 [via Groovy]" <ml...@n5.nabble.com>，2016年10月6日 下午8:19写道：
Oh, it could be a problem dropping Java 7 support if this were ever
contributed to Groovy core
Cheers
On Thu, Oct 6, 2016 at 12:44 PM, daniel_sun <[hidden email]> wrote:
> Hi Graeme,
>
>
> Thanks for your help! I'll give it a try :)
>
>
> PS: The new parser requires Java8 because streams and lambda expression are
> used.
>
>
> Cheers,
> Daniel.Sun
>
> ________________________________
> 发件人: Graeme Rocher-2 [via Groovy] <ml-node+[hidden email]>
> 发送时间: 2016年10月6日 18:27
> 收件人: daniel_sun
> 主题: Re: 答复: [PROGRESS REPORT 20161004]groovy-parser
>
> Hi Daniel,
>
> Yes I have seen that before, it happens when Travis runs out of memory
> since each Travis build is limited by memory. Try split up the steps
> that compile the code with the steps that run the tests and also avoid
> using the Gradle daemon:
>
> ./gradlew compileGroovy --no-daemon
> ./gradlew compileTestGroovy --no-daemon
> ./gradlew test --no-daemon
>
> Also in my experience Java 8 is more memory intensive than Java 7, so
> if you can use Java 7 to build that often helps.
>
> Cheers
>
> On Thu, Oct 6, 2016 at 12:16 PM, daniel_sun <[hidden email]> wrote:
>
>> Hi Graeme,
>>
>>
>> The last test result is shown as following:
>>
>> https://travis-ci.org/danielsun1106/groovy/builds/165263810
>>
>> 13395 tests completed, 3 failed, 9 skipped
>>
>> Currently I'm trying to solve the following issue, which blocks me
>> to
>> verify issues. It's pity that I'm not very familiar with gradle and travis
>> CI, so I've asked Cédric and Paul for help. BTW, I can run test cases
>> without any problem on my local machine.
>>
>> [org.gradle.api.tasks.TaskExecutionException: Execution failed for task
>> ':groovy-ant:compileTestGroovy']
>> org.gradle.process.internal.worker.WorkerProcessException: Failed to run
>> Gradle Compiler Daemon
>> https://github.com/danielsun1106/groovy/issues/4
>>
>>
>> Graeme, did you encounter some similiar problem like the above
>> ever?
>>
>> Thanks,
>> Daniel.Sun
>> ________________________________
>> 发件人: Graeme Rocher-2 [via Groovy] <ml-node+[hidden email]>
>> 发送时间: 2016年10月6日 17:29
>> 收件人: daniel_sun
>> 主题: Re: [PROGRESS REPORT 20161004]groovy-parser
>>
>> That is awesome, keep up the good work.
>>
>> Cheers
>>
>> On Wed, Oct 5, 2016 at 5:57 AM, daniel_sun <[hidden email]> wrote:
>>> Thanks :)
>>>
>>>
>>>
>>> --
>>> View this message in context:
>>>
>>> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735778.html
>>> Sent from the Groovy Dev mailing list archive at Nabble.com.
>>
>>
>>
>> --
>> Graeme Rocher
>>
>>
>> ________________________________
>> If you reply to this email, your message will be added to the discussion
>> below:
>>
>> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735829.html
>> To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here.
>> NAML
>>
>> ________________________________
>> View this message in context: 答复: [PROGRESS REPORT 20161004]groovy-parser
>>
>> Sent from the Groovy Dev mailing list archive at Nabble.com.
>
>
>
> --
> Graeme Rocher
>
>
> ________________________________
> If you reply to this email, your message will be added to the discussion
> below:
> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735833.html
> To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here.
> NAML
>
> ________________________________
> View this message in context: 答复: 答复: [PROGRESS REPORT
> 20161004]groovy-parser
>
> Sent from the Groovy Dev mailing list archive at Nabble.com.
--
Graeme Rocher
________________________________
If you reply to this email, your message will be added to the discussion below:
http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735842.html
To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=5735725&code=cmVhbGJsdWVzdW5AaG90bWFpbC5jb218NTczNTcyNXwxMTQ2MjE4MjI1>.
NAML<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml>
--
View this message in context: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735843.html
Sent from the Groovy Dev mailing list archive at Nabble.com.
Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser
Posted by Graeme Rocher <gr...@gmail.com>.
Oh, it could be a problem dropping Java 7 support if this were ever
contributed to Groovy core
Cheers
On Thu, Oct 6, 2016 at 12:44 PM, daniel_sun <re...@hotmail.com> wrote:
> Hi Graeme,
>
>
> Thanks for your help! I'll give it a try :)
>
>
> PS: The new parser requires Java8 because streams and lambda expression are
> used.
>
>
> Cheers,
> Daniel.Sun
>
> ________________________________
> 发件人: Graeme Rocher-2 [via Groovy] <ml-node+[hidden email]>
> 发送时间: 2016年10月6日 18:27
> 收件人: daniel_sun
> 主题: Re: 答复: [PROGRESS REPORT 20161004]groovy-parser
>
> Hi Daniel,
>
> Yes I have seen that before, it happens when Travis runs out of memory
> since each Travis build is limited by memory. Try split up the steps
> that compile the code with the steps that run the tests and also avoid
> using the Gradle daemon:
>
> ./gradlew compileGroovy --no-daemon
> ./gradlew compileTestGroovy --no-daemon
> ./gradlew test --no-daemon
>
> Also in my experience Java 8 is more memory intensive than Java 7, so
> if you can use Java 7 to build that often helps.
>
> Cheers
>
> On Thu, Oct 6, 2016 at 12:16 PM, daniel_sun <[hidden email]> wrote:
>
>> Hi Graeme,
>>
>>
>> The last test result is shown as following:
>>
>> https://travis-ci.org/danielsun1106/groovy/builds/165263810
>>
>> 13395 tests completed, 3 failed, 9 skipped
>>
>> Currently I'm trying to solve the following issue, which blocks me
>> to
>> verify issues. It's pity that I'm not very familiar with gradle and travis
>> CI, so I've asked Cédric and Paul for help. BTW, I can run test cases
>> without any problem on my local machine.
>>
>> [org.gradle.api.tasks.TaskExecutionException: Execution failed for task
>> ':groovy-ant:compileTestGroovy']
>> org.gradle.process.internal.worker.WorkerProcessException: Failed to run
>> Gradle Compiler Daemon
>> https://github.com/danielsun1106/groovy/issues/4
>>
>>
>> Graeme, did you encounter some similiar problem like the above
>> ever?
>>
>> Thanks,
>> Daniel.Sun
>> ________________________________
>> 发件人: Graeme Rocher-2 [via Groovy] <ml-node+[hidden email]>
>> 发送时间: 2016年10月6日 17:29
>> 收件人: daniel_sun
>> 主题: Re: [PROGRESS REPORT 20161004]groovy-parser
>>
>> That is awesome, keep up the good work.
>>
>> Cheers
>>
>> On Wed, Oct 5, 2016 at 5:57 AM, daniel_sun <[hidden email]> wrote:
>>> Thanks :)
>>>
>>>
>>>
>>> --
>>> View this message in context:
>>>
>>> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735778.html
>>> Sent from the Groovy Dev mailing list archive at Nabble.com.
>>
>>
>>
>> --
>> Graeme Rocher
>>
>>
>> ________________________________
>> If you reply to this email, your message will be added to the discussion
>> below:
>>
>> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735829.html
>> To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here.
>> NAML
>>
>> ________________________________
>> View this message in context: 答复: [PROGRESS REPORT 20161004]groovy-parser
>>
>> Sent from the Groovy Dev mailing list archive at Nabble.com.
>
>
>
> --
> Graeme Rocher
>
>
> ________________________________
> If you reply to this email, your message will be added to the discussion
> below:
> http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735833.html
> To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here.
> NAML
>
> ________________________________
> View this message in context: 答复: 答复: [PROGRESS REPORT
> 20161004]groovy-parser
>
> Sent from the Groovy Dev mailing list archive at Nabble.com.
--
Graeme Rocher