Maven 2.x Exec Plugin

Add outputFile support for capturing exec invocations (patch attached)

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.0.2
  • Fix Version/s: 1.1
  • Component/s: exec
  • Labels:
    None
  • Patch Submitted:
    Yes
  • Number of attachments :
    2

Description

Attached is a patch which adds optional outputFile support so that exec invocations may be captured to a text file, e.g.:

<configuration>
<executable>/usr/bin/svnversion</executable>
<outputFile>src/main/webapp/version.txt</outputFile>
</configuration>

When no outputFile is present, System.out is used for stdout rather than getLog().info().

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Jay Dolan added a comment -

I've attached an extension to my original patch. The extension introduces a "failOnError" boolean to allow the plugin to perform "optional" jobs (nice-to-have's). Set failOnError to false in the <configuration> element of your pom.

Show
Jay Dolan added a comment - I've attached an extension to my original patch. The extension introduces a "failOnError" boolean to allow the plugin to perform "optional" jobs (nice-to-have's). Set failOnError to false in the <configuration> element of your pom.
Hide
Permalink
Jerome Lacoste added a comment -

I've committed something that should fix the issue.

Some small functional differences from Jay's patch:

  • both standard & error output are redirected to the same file
  • I kept the standard maven output instead of using System.out

$ svn commit -m "MEXEC-27 support redirect execution output to a file. Original patch by Jay Dolan"
Sending src/main/java/org/codehaus/mojo/exec/ExecMojo.java
Adding src/main/java/org/codehaus/mojo/exec/StreamLog.java
Transmitting file data ..
Committed revision 6912.

Jay feel free to comment.

Show
Jerome Lacoste added a comment - I've committed something that should fix the issue. Some small functional differences from Jay's patch: both standard & error output are redirected to the same file I kept the standard maven output instead of using System.out $ svn commit -m " MEXEC-27 support redirect execution output to a file. Original patch by Jay Dolan" Sending src/main/java/org/codehaus/mojo/exec/ExecMojo.java Adding src/main/java/org/codehaus/mojo/exec/StreamLog.java Transmitting file data .. Committed revision 6912. Jay feel free to comment.
Hide
Permalink
Jay Dolan added a comment -

It's been a year, so I hardly remember why I wanted System.out instead of the log. I can't think of a scenario where that makes sense, really. So anyway, thanks for merging this in

Show
Jay Dolan added a comment - It's been a year, so I hardly remember why I wanted System.out instead of the log. I can't think of a scenario where that makes sense, really. So anyway, thanks for merging this in

People

  • Assignee:
    Jerome Lacoste
    Reporter:
    Jay Dolan
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: