Reviewers: Lex,

Description:
Users should no longer use the SoycDashboard directly, because the
compiler option -compileReport will already cause all compile report
files to be written to the extras directory.  This patch adds a simple
warning to users of the dashboard.

For people who use -soyc, the particular wording of the warning seems to
imply that they're to use -compileReport if they want the files written
directly.  This is a bit misleading, but its only effect will be to move
more people to using -compileReport rather than the deprecated -soyc,
which I think is ok.

Please review this at http://gwt-code-reviews.appspot.com/111803

Affected files:
   dev/core/src/com/google/gwt/soyc/SoycDashboard.java


Index: dev/core/src/com/google/gwt/soyc/SoycDashboard.java
===================================================================
--- dev/core/src/com/google/gwt/soyc/SoycDashboard.java (revision 7116)
+++ dev/core/src/com/google/gwt/soyc/SoycDashboard.java (working copy)
@@ -65,7 +65,14 @@
      }
    }

-  public static void main(final String[] args) {
+  public static void main(final String[] args) throws InterruptedException  
{
+
+    System.out.println("WARNING: The direct use of the SoycDashboard is  
deprecated and will be removed.  " +
+      "The preferred usage is to invoke the compiler with the  
-compileReport option, which" +
+      " writes the compile report directly to the extra directory.");
+    Thread.currentThread();
+    Thread.sleep(1000);
+
      Settings settings;
      try {
        settings = Settings.fromArgumentList(args);


-- 
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to