Introduced API to report build errors and warnings back to build workspace host. The idea is to allow the host associate messages with file/line/column.

git-svn-id: file:///opt/svn/repositories/sonatype.org/spice/trunk/plexus-build-api@1791 5751e0cb-dcb7-432f-92e2-260806df54be

Author: Igor Fedorenko

src/main/java/org/sonatype/plexus/build/incremental/BuildContext2.java

@@ -0,0 +1,21 @@
+/*
+Copyright (c) 2008 Sonatype, Inc. All rights reserved.
+
+This program is licensed to you under the Apache License Version 2.0, 
+and you may not use this file except in compliance with the Apache License Version 2.0. 
+You may obtain a copy of the Apache License Version 2.0 at http://www.apache.org/licenses/LICENSE-2.0.
+
+Unless required by applicable law or agreed to in writing, 
+software distributed under the Apache License Version 2.0 is distributed on an 
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. 
+See the Apache License Version 2.0 for the specific language governing permissions and limitations there under.
+*/
+package org.sonatype.plexus.build.incremental;
+
+import java.io.File;
+
+public interface BuildContext2 extends BuildContext {
+  public void addWarning(File file, int line, int column, String message, Throwable cause);
+
+  public void addError(File file, int line, int column, String message, Throwable cause);
+}

src/main/java/org/sonatype/plexus/build/incremental/DefaultBuildContext.java

@@ -77,4 +77,10 @@ public Object getValue(String key) {
 
   public void setValue(String key, Object value) {
   }
+
+  public void addWarning(File file, int line, int column, String message, Throwable cause) {
+  }
+
+  public void addError(File file, int line, int column, String message, Throwable cause) {
+  }
 }

src/main/java/org/sonatype/plexus/build/incremental/ThreadBuildContext.java

@@ -18,8 +18,6 @@
 import java.io.OutputStream;
 import java.util.List;
 
-import org.sonatype.plexus.build.incremental.BuildContext;
-import org.sonatype.plexus.build.incremental.DefaultBuildContext;
 import org.codehaus.plexus.util.Scanner;
 
 /**
@@ -31,7 +29,7 @@
  * Apparently, older version of plexus used by maven-filtering and likely
  * other projects, does not honour "default" role-hint.
  */
-public class ThreadBuildContext implements BuildContext {
+public class ThreadBuildContext implements BuildContext, BuildContext2 {
 
   private static final ThreadLocal threadContext = new ThreadLocal();
 
@@ -89,4 +87,18 @@ public void setValue(String key, Object value) {
     getContext().setValue(key, value);
   }
 
+  public void addWarning(File file, int line, int column, String message, Throwable cause) {
+    BuildContext context = getContext();
+    if (context instanceof BuildContext2) {
+	  ((BuildContext2) context).addWarning(file, line, column, message, cause);
+    }
+  }
+
+  public void addError(File file, int line, int column, String message, Throwable cause) {
+    BuildContext context = getContext();
+    if (context instanceof BuildContext2) {
+      ((BuildContext2) context).addError(file, line, column, message, cause);
+    }
+  }
+
 }

src/test/java/org/sonatype/plexus/build/incremental/test/TestIncrementalBuildContext.java

@@ -17,6 +17,7 @@
 import java.io.FileOutputStream;
 import java.io.IOException;
 import java.io.OutputStream;
+import java.util.ArrayList;
 import java.util.HashSet;
 import java.util.Iterator;
 import java.util.List;
@@ -26,10 +27,11 @@
 import org.codehaus.plexus.util.DirectoryScanner;
 import org.codehaus.plexus.util.Scanner;
 import org.sonatype.plexus.build.incremental.BuildContext;
+import org.sonatype.plexus.build.incremental.BuildContext2;
 
 
-public class TestIncrementalBuildContext implements BuildContext {
-  
+public class TestIncrementalBuildContext implements BuildContext, BuildContext2 {
+
   private final File basedir;
 
   private final HashSet refresh = new HashSet();
@@ -74,16 +76,25 @@ public File getBasedir() {
 
   private final Map context;
 
+  private final List warnings;
+  
+  private final List errors;
 
-  public TestIncrementalBuildContext(File basedir, Set files, Map context) {
-    this(basedir, files, new HashSet(), context);
+  public TestIncrementalBuildContext(File basedir, Set changedFiles, Map context) {
+    this(basedir, changedFiles, new HashSet(), context);
   }
 
   public TestIncrementalBuildContext(File basedir, Set changedFiles, Set deletedFiles, Map context) {
+    this(basedir, changedFiles, new HashSet(), context, new ArrayList(), new ArrayList());
+  }
+
+  public TestIncrementalBuildContext(File basedir, Set changedFiles, Set deletedFiles, Map context, List warnings, List errors) {
     this.basedir = basedir;
     this.changedFiles = changedFiles;
     this.deletedFiles = deletedFiles;
     this.context = context;
+    this.warnings = warnings;
+    this.errors = errors;
   }
 
   public boolean hasDelta(String relpath) {
@@ -149,4 +160,9 @@ public Set getRefreshFiles() {
     return refresh;
   }
 
+  public void addError(File file, int line, int column, String message, Throwable cause) {
+  }
+
+  public void addWarning(File file, int line, int column, String message, Throwable cause) {
+  }
 }