ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Conor MacNeill" <>
Subject RE: [PATCH] build events
Date Sat, 24 Jun 2000 10:56:17 GMT

I have now applied your build events patch with some minor mods

1. Added flag to prevent ant trying to build when project configuration
failed or help requested

2. Added -listener to usage

3. minor comments

4. Made it work under jdk 1.1 (I left out setting the InputSource system Id
for this reason)

I do have a few questions, however, which may be worth further discussion.

You have changed Javac to remove the need for the JavacOutputStream. The
result is that javac output is not streamed out, but batched up in memory. I
presume that is so you can set the appropriate priority on the whole output
message from javac (warn or error versus info). Are people cool with that?
What about Jikes, modern compile, etc?

The detection of whether an error occurred is now based on the javac return
value rather than the detection of the word "error" in the javac output
stream. Is that behaviour of Javac guaranteed?

I haven't yet added the log.xsl stylesheet. I am tossing up whether to add
it at the root of the ant tree or in the etc directory? ideas anyone?


Conor MacNeill

> -----Original Message-----
> From: []
> Sent: Monday, 19 June 2000 12:31
> To:
> Subject: [PATCH] build events
> ok, here's a first attempt to add events to Ant. The basic idea is to keep
> the core build engine "clean" and free of any presentation logic, and to
> make it easier to extend Ant with other features without cluttering up the
> core. To do this, I've defined a BuildListener interface and added an
> "addBuildListener" method to Project that can be used to register listener
> objects. Listeners could be implemented to generate reports, send out
> emails when the build is complete, create a bill of materials, etc...
> The only new functionality visible to the end-user is a "-listener" option
> on the command line that will let you specify the name of a class. An
> instance of this class will be added as a listener to the project. I've
> included a listener that will generate an XML log file, which you can use
> by typing the command below. I've also included a simple stylesheet to
> display the generated XML:
> build -listener
> (See attached file: events.jar)
> Matt Foemmel
> ThoughtWorks, Inc.
> ----- Forwarded by Matthew P Foemmel/Corporate/ThoughtWorks/US on
> 06/18/2000 05:51 AM -----
>                     Matthew P
>                     Foemmel              To:
>                                          cc:
>                     06/14/2000           Subject:     build
> events
>                     05:39 PM
> I've made a few changes to Ant for my project here, and I'd like some
> feedback on whether its worth cleaning up and submitting as a patch.
> Basically, we needed a way to generate an XML file with a summary of what
> errors happened during the build. To do this cleanly, I ended up
> implementing event/listener classes so that one can add listeners to a
> project and be notified when various things happen. The classes look
> something like:
> public class BuildEvent extends java.util.EventObject {
>      public Project getProject();
>      public Target getTarget();
>      public Task getTask();
>      public Throwable getException();
>      public String getMessage();
>      public int getMessageLevel();
> }
> public interface BuildListener extends java.util.EventListener {
>      public void buildStarted(BuildEvent event);
>      public void buildFinished(BuildEvent event);
>      public void targetStarted(BuildEvent event);
>      public void targetFinished(BuildEvent event);
>      public void taskStarted(BuildEvent event);
>      public void taskFinished(BuildEvent event);
>      public void messageLogged(BuildEvent event);
> }
> public class Project {
>      public void addBuildListener(BuildListener listener);
>      ...
> }
> Then I simply defined an XmlLogger class that dumped whatever XML I wanted
> into a file, and added it as a listener to the Project. I was also able to
> move all of the "user interface" code for Ant (ie all of the out.println
> ()'s) into so that it was all in one place by making it a
> BuildListener. The makes the rest of the code cleaner if we want to create
> a gui version of Ant, say. It also makes it easy to create listeners to do
> profiling, debugging, etc...
> Is this worth pursuing?

View raw message