It would be useful if the javadoc task could fail a build if the javadoc
program issued any warnings.
One way to do this would be to add an option, failonwarning, and to modify
org.apache.tools.ant.taskdefs.Javadoc as follows:
---
/cygdrive/c/Users/tboemker/Desktop/jars/apache-ant-1.9.1-src/src/main/org/apache/tools/ant/taskdefs/Javadoc.java
2013-05-15 22:37:06.000000000 -0400
+++ Javadoc.java 2013-05-23 12:55:38.875978100 -0400
@@ -15,7 +15,7 @@
* limitations under the License.
*
*/
-package org.apache.tools.ant.taskdefs;
+package com.eLynx;
import java.io.File;
import java.io.FileWriter;
@@ -39,6 +39,9 @@
import org.apache.tools.ant.Project;
import org.apache.tools.ant.ProjectComponent;
import org.apache.tools.ant.Task;
+import org.apache.tools.ant.taskdefs.Execute;
+import org.apache.tools.ant.taskdefs.LogOutputStream;
+import org.apache.tools.ant.taskdefs.PumpStreamHandler;
import org.apache.tools.ant.types.Commandline;
import org.apache.tools.ant.types.DirSet;
import org.apache.tools.ant.types.EnumeratedAttribute;
@@ -423,6 +426,11 @@
* Javadoc error.
*/
private boolean failOnError = false;
+ /**
+ * Flag which indicates if the task should fail if there is a
+ * Javadoc warning.
+ */
+ private boolean failOnWarning = false;
private Path sourcePath = null;
private File destDir = null;
private Vector<SourceFile> sourceFiles = new Vector<SourceFile>();
@@ -1540,6 +1548,17 @@
}
/**
+ * Should the build process fail if Javadoc warns (as indicated by
+ * the word "warnings" in stdout)?
+ *
+ * <p>Default is false.</p>
+ * @param b a <code>boolean</code> value
+ */
+ public void setFailonwarning(boolean b) {
+ failOnWarning = b;
+ }
+
+ /**
* Enables the -source switch, will be ignored if Javadoc is not
* the 1.4 version.
* @param source a <code>String</code> value
@@ -1765,6 +1784,10 @@
throw new BuildException("Javadoc returned " + ret,
getLocation());
}
+ if (out.sawWarnings() && failOnWarning) {
+ throw new BuildException("Javadoc issued warnings.",
+ getLocation());
+ }
} catch (IOException e) {
throw new BuildException("Javadoc failed: " + e, e, getLocation());
} finally {
@@ -2431,7 +2454,10 @@
// unless they appear after what could be an informational message.
//
private String queuedLine = null;
+ private boolean sawWarnings = false;
protected void processLine(String line, int messageLevel) {
+ if(line.contains("warnings"))
+ sawWarnings = true;
if (messageLevel == Project.MSG_INFO
&& line.startsWith("Generating ")) {
if (queuedLine != null) {
@@ -2458,6 +2484,10 @@
queuedLine = null;
}
}
+
+ public boolean sawWarnings() {
+ return sawWarnings;
+ }
}
/**
[cid:[email protected]] <http://www.elynx.com/>
Tim Boemker
Sr. Software Engineer
[email protected]<mailto:[email protected]>
office: 513.612.5945
fax: 513.612.5978
www.elynx.com
enter the world of [cid:[email protected]]
[cid:[email protected]]
<http://www.linkedin.com/company/30651?trk=tyah>
[cid:[email protected]]
<http://www.facebook.com/pages/Elynx/183936758316994>
Electronic Privacy Notice. This email may be covered by electronic
communications privacy laws and may be confidential or proprietary in nature.
If you are not the intended recipient, you are prohibited from retaining,
using, copying, distributing or disclosing this email. Instead, please reply to
the sender that you have received this email in error, and then delete it.
Thank you.
