Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Auto Closed
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      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 P. 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
          Hide
          Michael Osipov added a comment -

          This issue has been auto closed because it has been inactive for a long period of time. If you think this issue still applies, retest your problem with the most recent version of Maven and the affected component, reopen and post your results.

          Show
          Michael Osipov added a comment - This issue has been auto closed because it has been inactive for a long period of time. If you think this issue still applies, retest your problem with the most recent version of Maven and the affected component, reopen and post your results.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development