Reviewers: kpreid2,

Description:
The recently @Deprecated classes are not used any more by Caja, so
they are no longer compiled automatically via transitive dependency,
so they never end up in the caja jar file, which breaks external
packages that still use those classes.

This CL adds a rule to explicitly build the deprecated classes.
I'm keeping this short and simple because I don't expect us to need
any more such classes, and I'm hoping that the external references
will be deleted soon anyway.

Please review this at https://codereview.appspot.com/13944043/

Affected files (+4, -0 lines):
  M     build.xml


Index: build.xml
===================================================================
--- build.xml   (revision 5606)
+++ build.xml   (working copy)
@@ -571,6 +571,10 @@
       <include name="com/google/caja/plugin/*Main.java"/>
       <!-- used by Closure Compiler JsonMLConversionTest -->
       <include name="com/google/caja/parser/js/JsonMLParser.java"/>
+      <!-- deprecated classes. TODO(felix8a): delete sometime -->
+      <include name="com/google/caja/util/Lists.java"/>
+      <include name="com/google/caja/util/Maps.java"/>
+      <include name="com/google/caja/util/Sets.java"/>
     </javac>
     <copy todir="${lib}">
       <fileset dir="${src}">


--

--- You received this message because you are subscribed to the Google Groups "Google Caja Discuss" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to