Maven 1.x JCoverage Plugin
  1. Maven 1.x JCoverage Plugin
  2. MPJCOVERAGE-20

allow comma separeted values for maven.jcoverage.instrumentation.excludes

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.0, 1.0.1, 1.0.2, 1.0.3, 1.0.4, 1.0.5, 1.0.6, 1.0.7, 1.0.8, 1.0.9
    • Fix Version/s: 1.1
    • Labels:
      None
    • Number of attachments :
      1

      Description

      With following modification in the plugin.jelly a user can specify more than one exclude filters via maven.jcoverage.instrumentation.excludes (comma separeted).

      Here the patch in the plugin.jelly:

      Index: plugin.jelly
      ===================================================================
      RCS file: /home/cvspublic/maven-plugins/jcoverage/plugin.jelly,v
      retrieving revision 1.16
      diff -u -r1.16 plugin.jelly
      — plugin.jelly 17 Aug 2004 14:00:48 -0000 1.16
      +++ plugin.jelly 27 Oct 2004 08:09:25 -0000
      @@ -121,10 +121,16 @@
      <maven:addPath id="maven.dependency.classpath" refid="jcoverage.classpath"/>
      <maven:addPath id="maven.dependency.classpath" refid="$

      {pom.getDependencyClasspath()}

      "/>

      +
      + <u:tokenize var="maven.jcoverage.instrumentation.excludes.split" delim=",">$

      {maven.jcoverage.instrumentation.excludes}</u:tokenize>
      +
      <instrument todir="${maven.jcoverage.instrumentation}">
      - <ant:fileset dir="${maven.build.dest}"
      - includes="${maven.jcoverage.instrumentation.includes}"
      - excludes="${maven.jcoverage.instrumentation.excludes}

      "/>
      + <ant:fileset dir="$

      {maven.build.dest}">
      + <ant:include name="*/.class" />
      + <j:forEach var="var" items="${maven.jcoverage.instrumentation.excludes.split}">
      + <ant:exclude name="${var}" />
      + </j:forEach>
      + </ant:fileset>
      <ant:classpath>
      <ant:path refid="jcoverage.classpath"/>
      </ant:classpath>
      @@ -133,9 +139,12 @@
      <ant:copy todir="${maven.jcoverage.instrumentation}">
      <ant:fileset dir="${maven.build.dest}

      ">
      <ant:exclude name="*/.class" />

      • <ant:exclude name="$ {maven.jcoverage.instrumentation.excludes}

        " />

      • </ant:fileset>
        + <j:forEach var="var" items="$ {maven.jcoverage.instrumentation.excludes.split}

        ">
        + <ant:exclude name="$

        {var}

        " />
        + </j:forEach>
        + </ant:fileset>
        </ant:copy>
        +

      </j:catch>
      <j:if test="$

      {ex != null}

      ">

        Activity

        Hide
        Tobias Joch added a comment -

        here is the patch as file...

        Show
        Tobias Joch added a comment - here is the patch as file...
        Hide
        Emmanuel Venisse added a comment -

        Applied.

        Show
        Emmanuel Venisse added a comment - Applied.

          People

          • Assignee:
            Emmanuel Venisse
            Reporter:
            Tobias Joch
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: