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

          Activity

          Hide
          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
          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
          Hide
          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.
          Hide
          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

            People

            • Assignee:
              Unassigned
              Reporter:
              Brett Porter
            • Votes:
              2 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: