Maven 2.x Compiler Plugin

improve documentation

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: backlog
  • Labels:
    None
  • Number of attachments :
    0

Description

some aspects are unclear:

  • compilerVersion is only used when forking
  • fork should indicate that it uses the built in version when false, not an executable.

Also:

  • don't set parameters that only apply to forking when fork is false to make the code more readable
  • specifiy which are specific to javac, or java in general (perhaps we should be able to assign a @category to a plugin parameter)

Issue Links

is related to

Improvement - An improvement or enhancement to an existing feature or task. MCOMPILER-58 Improve documentation for cross-compiling Java code

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Improvement - An improvement or enhancement to an existing feature or task. MCOMPILER-57 State default source (1.3) and target (1.1) in the documentation

  • Trivial - Cosmetic problem like misspelt words or misaligned text.
  • 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
Brett Porter made changes -
Field Original Value New Value
Fix Version/s 2.0-beta-3 [ 11991 ]
John Casey made changes -
Fix Version/s 2.0.1 [ 12000 ]
Fix Version/s 2.0.2 [ 12107 ]
Jason van Zyl made changes -
Project Maven 2 [ 10500 ] Maven 2.x Compiler Plugin [ 11130 ]
Component/s maven-compiler-plugin [ 11806 ]
Workflow Maven [ 39767 ] jira [ 44059 ]
Key MNG-972 MCOMPILER-6
Fix Version/s 2.0.3 [ 12107 ]
Jason van Zyl made changes -
Workflow jira [ 44059 ] Maven [ 44697 ]
Brett Porter made changes -
Workflow Maven [ 44697 ] Maven New [ 51792 ]
Carlos Sanchez made changes -
Fix Version/s 2.0.1 [ 12178 ]
Hide
Permalink
Carlos Sanchez added a comment -

The two first bullet points are fixed

Show
Carlos Sanchez added a comment - The two first bullet points are fixed
Carlos Sanchez made changes -
Fix Version/s 2.0.2 [ 12484 ]
Fix Version/s 2.0.1 [ 12178 ]
Carlos Sanchez made changes -
Fix Version/s 2.0.2 [ 12484 ]
Fix Version/s 2.1 [ 12304 ]
Hide
Permalink
Ian Springer added a comment -

Here's one more:

  • The docs for the source and target config props should state that the default value for those properties is "1.3".

http://maven.apache.org/plugins/maven-compiler-plugin/compile-mojo.html#source
http://maven.apache.org/plugins/maven-compiler-plugin/compile-mojo.html#target
http://maven.apache.org/plugins/maven-compiler-plugin/testCompile-mojo.html#source
http://maven.apache.org/plugins/maven-compiler-plugin/testCompile-mojo.html#target

Show
Ian Springer added a comment - Here's one more: The docs for the source and target config props should state that the default value for those properties is "1.3". http://maven.apache.org/plugins/maven-compiler-plugin/compile-mojo.html#source http://maven.apache.org/plugins/maven-compiler-plugin/compile-mojo.html#target http://maven.apache.org/plugins/maven-compiler-plugin/testCompile-mojo.html#source http://maven.apache.org/plugins/maven-compiler-plugin/testCompile-mojo.html#target
Stefano Lenzi made changes -
Link This issue is related to MCOMPILER-57 [ MCOMPILER-57 ]
Stefano Lenzi made changes -
Link This issue is depended upon by MCOMPILER-58 [ MCOMPILER-58 ]
Stefano Lenzi made changes -
Link This issue is depended upon by MCOMPILER-58 [ MCOMPILER-58 ]
Stefano Lenzi made changes -
Link This issue is related to MCOMPILER-58 [ MCOMPILER-58 ]
Stephen Connolly made changes -
Fix Version/s 2.1 [ 12304 ]
Fix Version/s 2.2 [ 16097 ]
Hide
Permalink
Milos Kleint added a comment -

actually according to the sources, the target default value is 1.1 and the source default value is 1.3 for jdks 1.4+ and no source is set for older jdks.

Show
Milos Kleint added a comment - actually according to the sources, the target default value is 1.1 and the source default value is 1.3 for jdks 1.4+ and no source is set for older jdks.
Milos Kleint made changes -
Fix Version/s 2.2 [ 16097 ]
Fix Version/s 2.3 [ 16208 ]
Hide
Permalink
Olivier Lamy added a comment -

this is a permanent issue. Not sure set a fix version is possible

Show
Olivier Lamy added a comment - this is a permanent issue. Not sure set a fix version is possible
Olivier Lamy made changes -
Fix Version/s backlog [ 18455 ]
Fix Version/s 2.4 [ 16208 ]

People

  • Assignee:
    Unassigned
    Reporter:
    Brett Porter
Vote (2)
Watch (1)

Dates

  • Created:
    Updated: