I think it is better to take validationtck out of the general ant build.
That way it will not get built into the distributable bits.

A few modified files lack the "Modified by Google" notice.


http://gwt-code-reviews.appspot.com/1085801/diff/40002/42001
File samples/build.xml (right):

http://gwt-code-reviews.appspot.com/1085801/diff/40002/42001#newcode59
samples/build.xml:59: <antcall target="validationtck" />
Remove this to prevent validationtck from being built and copied into
the distributable bits.

http://gwt-code-reviews.appspot.com/1085801/diff/40002/42013
File
samples/validationtck/src/org/jboss/testharness/super/org/jboss/testharness/AbstractTest.java
(right):

http://gwt-code-reviews.appspot.com/1085801/diff/40002/42013#newcode23
samples/validationtck/src/org/jboss/testharness/super/org/jboss/testharness/AbstractTest.java:23:

Add the "modified by Google" snippet

http://gwt-code-reviews.appspot.com/1085801/diff/40002/42014
File
samples/validationtck/src/org/jboss/testharness/super/org/jboss/testharness/api/Configuration.java
(right):

http://gwt-code-reviews.appspot.com/1085801/diff/40002/42014#newcode20
samples/validationtck/src/org/jboss/testharness/super/org/jboss/testharness/api/Configuration.java:20:
*
Add the "modified by Google" snippet

http://gwt-code-reviews.appspot.com/1085801/diff/40002/42014#newcode35
samples/validationtck/src/org/jboss/testharness/super/org/jboss/testharness/api/Configuration.java:35:
public static final String DEFAULT_OUTPUT_DIRECTORY =
"/tmp";//System.getProperty("java.io.tmpdir") + "/jsr-299-tck/";
Comment out the whole line. DEFAULT_OUTPUT_DIRECTORY is not used
anywhere

http://gwt-code-reviews.appspot.com/1085801/diff/40002/42017
File
samples/validationtck/src/org/jboss/testharness/super/org/jboss/testharness/spi/Containers.java
(right):

http://gwt-code-reviews.appspot.com/1085801/diff/40002/42017#newcode14
samples/validationtck/src/org/jboss/testharness/super/org/jboss/testharness/spi/Containers.java:14:
*
Add the modified by Google snippet

http://gwt-code-reviews.appspot.com/1085801/show

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

Reply via email to