bodewig 2005/04/04 00:40:32 Modified: . WHATSNEW docs/manual/OptionalTasks rpm.html src/main/org/apache/tools/ant/taskdefs/optional Rpm.java Log: Rename rpm's showoutput attribute to quiet, document it - Submitted by Zach Garner Revision Changes Path 1.801 +1 -1 ant/WHATSNEW Index: WHATSNEW =================================================================== RCS file: /home/cvs/ant/WHATSNEW,v retrieving revision 1.800 retrieving revision 1.801 diff -u -r1.800 -r1.801 --- WHATSNEW 31 Mar 2005 06:37:09 -0000 1.800 +++ WHATSNEW 4 Apr 2005 07:40:32 -0000 1.801 @@ -317,7 +317,7 @@ used to document packages that don't hold source files but a package.html file. Bugzilla Report 25339. -* <rpm> has new attributes failonerror and showoutput attributes. +* <rpm> has new attributes failonerror and quiet. * Added two tutorials - beginner: introduction into Ant 1.10 +16 -5 ant/docs/manual/OptionalTasks/rpm.html Index: rpm.html =================================================================== RCS file: /home/cvs/ant/docs/manual/OptionalTasks/rpm.html,v retrieving revision 1.9 retrieving revision 1.10 diff -u -r1.9 -r1.10 --- rpm.html 7 Mar 2005 18:11:14 -0000 1.9 +++ rpm.html 4 Apr 2005 07:40:32 -0000 1.10 @@ -31,7 +31,7 @@ <tr> <td valign="top">topDir</td> <td valign="top"> - this is the directory which will have the expected + This is the directory which will have the expected subdirectories, SPECS, SOURCES, BUILD, SRPMS. If this isn't specified, the baseDir value is used </td> @@ -45,12 +45,12 @@ </tr> <tr> <td valign="top">removeSpec</td> - <td valign="top">this will remove the spec file from SPECS</td> + <td valign="top">This will remove the spec file from SPECS</td> <td align="center" valign="top">No</td> </tr> <tr> <td valign="top">removeSource</td> - <td valign="top"> Flag (optional, default=false) + <td valign="top">Flag (optional, default=false) to remove the sources after the build. See the the <tt>--rmsource</tt> option of rpmbuild.</td> <td align="center" valign="top">No</td> @@ -66,12 +66,23 @@ </tr> <tr> <td valign="top">command</td> - <td valign="top">very similar idea to the cvs task. the default is "-bb"</td> + <td valign="top">Very similar idea to the cvs task. the default is "-bb"</td> + <td align="center" valign="top">No</td> + </tr> + <tr> + <td valign="top">quiet</td> + <td valign="top">Suppress output. Defaults to false.</td> <td align="center" valign="top">No</td> </tr> <tr> <td valign="top">output/error</td> - <td valign="top">where standard output and error go</td> + <td valign="top">Where standard output and error go</td> + <td align="center" valign="top">No</td> + </tr> + <tr> + <td valign="top">failOnError</td> + <td valign="top">Stop the buildprocess if the RPM build command exits with + a non-zero retuncode. Defaults to false</td> <td align="center" valign="top">No</td> </tr> </table> 1.27 +15 -15 ant/src/main/org/apache/tools/ant/taskdefs/optional/Rpm.java Index: Rpm.java =================================================================== RCS file: /home/cvs/ant/src/main/org/apache/tools/ant/taskdefs/optional/Rpm.java,v retrieving revision 1.26 retrieving revision 1.27 diff -u -r1.26 -r1.27 --- Rpm.java 21 Mar 2005 13:50:09 -0000 1.26 +++ Rpm.java 4 Apr 2005 07:40:32 -0000 1.27 @@ -95,10 +95,10 @@ private boolean failOnError = false; /** - * Show output of RPM build command on console. This does not affect + * Don't show output of RPM build command on console. This does not affect * the printing of output and error messages to files. */ - private boolean showoutput = true; + private boolean quiet = false; /** * Execute the task @@ -135,7 +135,7 @@ OutputStream outputstream = null; OutputStream errorstream = null; if (error == null && output == null) { - if (showoutput) { + if (!quiet) { streamhandler = new LogStreamHandler(this, Project.MSG_INFO, Project.MSG_WARN); } else { @@ -151,7 +151,7 @@ } catch (IOException e) { throw new BuildException(e, getLocation()); } - } else if (showoutput) { + } else if (!quiet) { outputstream = new LogOutputStream(this, Project.MSG_INFO); } else { outputstream = new LogOutputStream(this, Project.MSG_DEBUG); @@ -164,7 +164,7 @@ } catch (IOException e) { throw new BuildException(e, getLocation()); } - } else if (showoutput) { + } else if (!quiet) { errorstream = new LogOutputStream(this, Project.MSG_WARN); } else { errorstream = new LogOutputStream(this, Project.MSG_DEBUG); @@ -273,10 +273,10 @@ } /** - * If true, stop the build process when the rpmbuild command exits with - * an error status. - * @param value <tt>true</tt> if it should halt, otherwise - * <tt>false</tt> + * If <code>true</code>, stop the build process when the rpmbuild command + * exits with an error status. + * @param value <code>true</code> if it should halt, otherwise + * <code>false</code>. The default is <code>false</code>. * * @since Ant 1.6.3 */ @@ -285,14 +285,14 @@ } /** - * If false, no output from the RPM build command will be logged. - * @param value <tt>true</tt> if output should be logged, otherwise - * <tt>false</tt> + * If true, output from the RPM build command will only be logged to DEBUG. + * @param value <code>false</code> if output should be logged, otherwise + * <code>true</code>. The default is <code>false</code>. * * @since Ant 1.6.3 */ - public void setShowoutput(boolean value) { - showoutput = value; + public void setQuiet(boolean value) { + quiet = value; } /** @@ -330,7 +330,7 @@ } /** - * @since Ant 1.7 + * @since Ant 1.6.3 */ protected Execute getExecute(Commandline toExecute, ExecuteStreamHandler streamhandler) {
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]