Details

    • Type: Task Task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.2
    • Labels:
      None
    1. MANTRUN-55-maven-antrun-plugin.patch
      28 kB
      franz allan valencia see
    2. MANTRUN-55-maven-antrun-plugin-2.patch
      29 kB
      franz allan valencia see
    3. MANTRUN-55-maven-antrun-plugin-3.patch
      7 kB
      franz allan valencia see

      Activity

      Allan Ramirez created issue -
      Allan Ramirez made changes -
      Field Original Value New Value
      Assignee Allan Ramirez [ aramirez ]
      Hide
      Franz Allan Valencia See added a comment -

      Good day,

      MANTRUN-55-maven-antrun-plugin.patch contains documentation regarding maven-antrun-plugin and also, allows it to pass the document check for plugins (mvn docck:plugin).

      Franz

      Show
      Franz Allan Valencia See added a comment - Good day, MANTRUN-55 -maven-antrun-plugin.patch contains documentation regarding maven-antrun-plugin and also, allows it to pass the document check for plugins (mvn docck:plugin). Franz
      franz allan valencia see made changes -
      Attachment MANTRUN-55-maven-antrun-plugin.patch [ 21925 ]
      Hide
      Allan Ramirez added a comment -

      Looks good to me. Just a comment in your code block in usage.html, there are so many "[...]".
      I think somehing like below is enough

      <project>
        [...]
        <build>
          <plugins>
            <plugin>
              <artifactId>maven-antrun-plugin</artifactId>
              <executions>
                <execution>
                  <phase> <!-- a lifecycle phase --> </phase>
                  <configuration>
                    <tasks>
      
                      <!--
                        Place any ant task here. You can add anything
                        you can add between <target> and </target> in a
                        build.xml.
                      -->
                      
                    </tasks>
                  </configuration>
                  <goals>
                    <goal>run</goal>
                  </goals>
                </execution>
              </executions>
            </plugin>
          </plugins>
        </build>
        [...]
      </project>
      
      Show
      Allan Ramirez added a comment - Looks good to me. Just a comment in your code block in usage.html, there are so many " [...] ". I think somehing like below is enough <project> [...] <build> <plugins> <plugin> <artifactId> maven-antrun-plugin </artifactId> <executions> <execution> <phase> <!-- a lifecycle phase --> </phase> <configuration> <tasks> <!-- Place any ant task here. You can add anything you can add between <target> and </target> in a build.xml. --> </tasks> </configuration> <goals> <goal> run </goal> </goals> </execution> </executions> </plugin> </plugins> </build> [...] </project>
      Hide
      Franz Allan Valencia See added a comment -

      Changes with MANTRUN-55-maven-antrun-plugin-2.patch from MANTRUN-55-maven-antrun-plugin.patch

      In usage.html
      Removed excess "[...]" (Reported by Allan Ramirez)

      Show
      Franz Allan Valencia See added a comment - Changes with MANTRUN-55 -maven-antrun-plugin-2.patch from MANTRUN-55 -maven-antrun-plugin.patch In usage.html Removed excess " [...] " (Reported by Allan Ramirez)
      franz allan valencia see made changes -
      Attachment MANTRUN-55-maven-antrun-plugin-2.patch [ 21957 ]
      Hide
      Allan Ramirez added a comment -

      Applied patch.. and I also deployed your site.

      http://people.apache.org/~aramirez/maven-antrun-plugin/index.html

      Thanks

      Show
      Allan Ramirez added a comment - Applied patch.. and I also deployed your site. http://people.apache.org/~aramirez/maven-antrun-plugin/index.html Thanks
      Hide
      Franz Allan Valencia See added a comment -

      Changes with MANTRUN-55-maven-antrun-plugin-3.patch

      In usage.html

      • maven.dependency.classpath (Reported by Vincent Siveton)
      • Review <<<inheritRefs>> (Reported by Vincent Siveton)
      • it seems that there is bug here (from what i can dig up in the maven user's mailing list) such that referencing maven.xxx.classpath's within the build.xml does not work. thus, i changed the example to a workaround (assigning the maven.xxx.classpath's value to an ant property). Furthermore, this page may not be needed in the future once Vincent Siventon's submits his example of using external build.xml
      • changed "maven-dependencies-plugin" to "maven-dependency-plugin"

      In FAQ.html

      • "Maven for Ant Users" is for Maven1 and it's link is wrong (Reported by Vincent Siventon)
      • thus, i removed this link
      Show
      Franz Allan Valencia See added a comment - Changes with MANTRUN-55 -maven-antrun-plugin-3.patch In usage.html maven.dependency.classpath (Reported by Vincent Siveton) Review <<<inheritRefs>> (Reported by Vincent Siveton) it seems that there is bug here (from what i can dig up in the maven user's mailing list) such that referencing maven.xxx.classpath's within the build.xml does not work. thus, i changed the example to a workaround (assigning the maven.xxx.classpath's value to an ant property). Furthermore, this page may not be needed in the future once Vincent Siventon's submits his example of using external build.xml changed "maven-dependencies-plugin" to "maven-dependency-plugin" In FAQ.html "Maven for Ant Users" is for Maven1 and it's link is wrong (Reported by Vincent Siventon) thus, i removed this link
      franz allan valencia see made changes -
      Attachment MANTRUN-55-maven-antrun-plugin-3.patch [ 21994 ]
      Hide
      Allan Ramirez added a comment -

      Applied patch, and I also updated your staging site

      http://people.apache.org/~aramirez/maven-antrun-plugin/index.html

      Thanks!

      Show
      Allan Ramirez added a comment - Applied patch, and I also updated your staging site http://people.apache.org/~aramirez/maven-antrun-plugin/index.html Thanks!
      Allan Ramirez logged work - 04/Aug/06 07:13
      • Time Spent:
        2h
         
        reviewed patches and deployed to staging site
      Allan Ramirez made changes -
      Remaining Estimate 12h [ 43200 ] 10h [ 36000 ]
      Time Spent 2h [ 7200 ]
      Dennis Lundberg made changes -
      Fix Version/s 1.2 [ 12210 ]
      Status Open [ 1 ] Closed [ 6 ]
      Resolution Fixed [ 1 ]
      Mark Thomas made changes -
      Project Import Sun Apr 05 09:04:27 UTC 2015 [ 1428224667361 ]
      Mark Thomas made changes -
      Workflow jira [ 12716355 ] Default workflow, editable Closed status [ 12759881 ]
      Mark Thomas made changes -
      Project Import Sun Apr 05 22:13:33 UTC 2015 [ 1428272013336 ]
      Mark Thomas made changes -
      Workflow jira [ 12953678 ] Default workflow, editable Closed status [ 12990816 ]
      Transition Time In Source Status Execution Times Last Executer Last Execution Date
      Open Open Closed Closed
      600d 9h 42m 1 Dennis Lundberg 10/Mar/08 14:52

        People

        • Assignee:
          Allan Ramirez
          Reporter:
          Allan Ramirez
        • Votes:
          0 Vote for this issue
          Watchers:
          1 Start watching this issue

          Dates

          • Due:
            Created:
            Updated:
            Resolved:

            Time Tracking

            Estimated:
            Original Estimate - 12h
            12h
            Remaining:
            Time Spent - 2h Remaining Estimate - 10h
            10h
            Logged:
            Time Spent - 2h Remaining Estimate - 10h
            2h

              Development