Tapestry 5
  1. Tapestry 5
  2. TAP5-1204

tapestry-core JAR includes some compiled Groovy test scripts

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 5.2.0
    • Fix Version/s: 5.2.3
    • Component/s: tapestry-core
    • Labels:
      None

      Activity

      Hide
      Howard M. Lewis Ship added a comment -

      I believe this block of pom.xml is unnecessary:

      <resources>
      <resource>
      <!-- GMaven isn't smart enough to add this automatically. -->
      <directory>src/test/groovy</directory>
      </resource>
      <resource>
      <!--
      Once you mention one resource directory, you have to be explicit about all of them,
      apparently.
      -->
      <directory>src/main/resources</directory>
      </resource>
      </resources>

      Show
      Howard M. Lewis Ship added a comment - I believe this block of pom.xml is unnecessary: <resources> <resource> <!-- GMaven isn't smart enough to add this automatically. --> <directory>src/test/groovy</directory> </resource> <resource> <!-- Once you mention one resource directory, you have to be explicit about all of them, apparently. --> <directory>src/main/resources</directory> </resource> </resources>
      Hide
      Howard M. Lewis Ship added a comment -

      Also this is not needed:

      <dependency>
      <groupId>org.codehaus.groovy.maven.runtime</groupId>
      <artifactId>gmaven-runtime-1.6</artifactId>
      <version>1.0-rc-5</version>
      <scope>test</scope>
      </dependency>

      ... apparently I was thrashing when I first put the Groovy support in place.

      Show
      Howard M. Lewis Ship added a comment - Also this is not needed: <dependency> <groupId>org.codehaus.groovy.maven.runtime</groupId> <artifactId>gmaven-runtime-1.6</artifactId> <version>1.0-rc-5</version> <scope>test</scope> </dependency> ... apparently I was thrashing when I first put the Groovy support in place.
      Hide
      Hudson added a comment -

      Integrated in tapestry-5.2-freestyle #225 (See https://hudson.apache.org/hudson/job/tapestry-5.2-freestyle/225/)

      Show
      Hudson added a comment - Integrated in tapestry-5.2-freestyle #225 (See https://hudson.apache.org/hudson/job/tapestry-5.2-freestyle/225/ )

        People

        • Assignee:
          Howard M. Lewis Ship
          Reporter:
          Howard M. Lewis Ship
        • Votes:
          0 Vote for this issue
          Watchers:
          0 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development