Maven 1.x Java Plugin
  1. Maven 1.x Java Plugin
  2. MPJAVA-6

java plugin not respecting fork settings

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Number of attachments :
      0

      Description

      In the Java jelly.plugin, it looks like fork is being set to true in all cases, even when I have maven.compile.fork=no in my project.properties (or my build.properties, or on the command line as a system property). What dumb thing am I doing wrong?

      Excerpt:

      <j:choose>
      <j:when test="$

      {sourcesPresent == 'true'}

      ">
      <ant:javac
      fork="true"
      destdir="$

      {maven.build.dest}

      "
      excludes="**/package.html"
      debug="$

      {maven.compile.debug}

      "
      deprecation="$

      {maven.compile.deprecation}

      "
      optimize="$

      {maven.compile.optimize}

      ">
      <ant:src>
      <ant:path refid="maven.compile.src.set"/>
      </ant:src>
      [snip]
      <j:if test="$

      {context.getVariable('maven.compile.fork') != null}

      ">
      <ant:setProperty name="fork" value="$

      {maven.compile.fork}

      " />
      </j:if>
      [snip]
      </j:choose>

      In my case, that ant:setProperty call is not being picked up; fork is apparently always "yes".

      Cheers,
      Laird (new and enthusiastic Maven user)

      Laird J. Nelson
      Electronic Channel Group, Fidelity eBusiness (Veritude)

        Issue Links

          Activity

          Hide
          dion gillard added a comment -

          It appears this has been fixed for quite some time. plugin.jelly for the java plugin doesn't use fork="" as a javac attribute

          Show
          dion gillard added a comment - It appears this has been fixed for quite some time. plugin.jelly for the java plugin doesn't use fork="" as a javac attribute

            People

            • Assignee:
              dion gillard
              Reporter:
              dion gillard
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: