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

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          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
          Permalink
          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
          Permalink
          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