Maven 2.x JavaCC Plugin

Allow customized parser sources to reside in src/main/java

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.4
  • Fix Version/s: 2.5
  • Component/s: javacc, jjtree, jtb
  • Labels:
    None
  • Number of attachments :
    0

Description

The plugin currently requires users to place custom source files next to the grammar file as explained in the FAQ. As Jason Dillon pointed out on the mail thread JavaCC Maven Plugin at Geronimo, this is odd and I agree. Why should custom Java sources for the parser not live next to all the other Java sources in src/main/java?

Issue Links

is related to

Bug - A problem which impairs or prevents the functions of the product. MJAVACC-20 Custom sources get re-generated

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Paul Gier added a comment -

I think MJAVACC-20 explains why it implemented that way initially. Although I agree that it should be changed so that you can keep java sources in the main java source tree instead of with the javacc files.

Show
Paul Gier added a comment - I think MJAVACC-20 explains why it implemented that way initially. Although I agree that it should be changed so that you can keep java sources in the main java source tree instead of with the javacc files.
Hide
Permalink
Benjamin Bentmann added a comment -

Done in r7754, snapshot deployed.

Show
Benjamin Bentmann added a comment - Done in r7754 , snapshot deployed.
Hide
Permalink
Benjamin Bentmann added a comment -

Just to be explicit: The deprecated goals jjtree and jtb have not been enhanced. Only jjtree-javacc and jtb-javacc support this feature.

Show
Benjamin Bentmann added a comment - Just to be explicit: The deprecated goals jjtree and jtb have not been enhanced. Only jjtree-javacc and jtb-javacc support this feature.
Hide
Permalink
Benjamin Bentmann added a comment -

For the record: There seems to be an issue with IDEA's Maven integration (IDEA-19177) that causes the interim output of JJTree/JTB to be registered as a compile source root although the JavaCC Plugin does not actually declare it as such. This causes duplicate class errors...

Show
Benjamin Bentmann added a comment - For the record: There seems to be an issue with IDEA's Maven integration ( IDEA-19177 ) that causes the interim output of JJTree/JTB to be registered as a compile source root although the JavaCC Plugin does not actually declare it as such. This causes duplicate class errors...
Hide
Permalink
Jason Dillon added a comment -

Finally verified that setting:

<interimDirectory>${project.build.directory}/tmp/jjtree</interimDirectory>

Does fix the problem. Which is good enough for my needs.

Show
Jason Dillon added a comment - Finally verified that setting: <interimDirectory>${project.build.directory}/tmp/jjtree</interimDirectory> Does fix the problem. Which is good enough for my needs.

People

  • Assignee:
    Benjamin Bentmann
    Reporter:
    Benjamin Bentmann
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: