You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Christian Grobmeier (Issue Comment Edited) (JIRA)" <ji...@apache.org> on 2011/10/03 23:52:34 UTC
[jira] [Issue Comment Edited] (OGNL-8) use javacc to generate AST*
and OGNLParser Java sources
[ https://issues.apache.org/jira/browse/OGNL-8?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13119656#comment-13119656 ]
Christian Grobmeier edited comment on OGNL-8 at 10/3/11 9:52 PM:
-----------------------------------------------------------------
Daniel, i tried to include your patches. So far it looks like the right direction. I have even managed to compile it one time. But mvn test fails because of unresolved compilation failures. My assumption is that the generated sources are not put on the classpath when the mvn test runs. Can you confirm that?
Another problem is Eclipse (at least for me). You generate into the target folder, which is surely correct, but did you somehow manage to add the generated sources on your ides classpath? (assuming you do not use Notepad and mvn only hehe :-))
I made a similar page this evening before I read you message. I have generated it into the src/main/java/.../javacc package which at least does solve the eclipse problem. My suggestion is we reenable code generation first and then care about the maven problems. What do you think?
Any hints to make your patch working are very welcome of course.
was (Author: cgrobmeier):
Daniel, i tried to include your second patch (assuming the first one is already in svn - could not find any differences). So far it looks like the right direction. I have even managed to compile it one time. But mvn test fails because of unresolved compilation failures. My assumption is that the generated sources are not put on the classpath when the mvn test runs. Can you confirm that?
Another problem is Eclipse (at least for me). You generate into the target folder, which is surely correct, but did you somehow manage to add the generated sources on your ides classpath? (assuming you do not use Notepad and mvn only hehe :-))
I made a similar page this evening before I read you message. I have generated it into the src/main/java/.../javacc package which at least does solve the eclipse problem. My suggestion is we reenable code generation first and then care about the maven problems. What do you think?
Any hints to make your patch working are very welcome of course.
> use javacc to generate AST* and OGNLParser Java sources
> -------------------------------------------------------
>
> Key: OGNL-8
> URL: https://issues.apache.org/jira/browse/OGNL-8
> Project: OGNL
> Issue Type: Task
> Reporter: Olivier Lamy
> Assignee: Olivier Lamy
> Attachments: OGNL-8-keep-custom-AST.patch, OGNL-8.patch
>
>
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira