The module definitions need some more work now that the super source
files are in test-super


http://gwt-code-reviews.appspot.com/1267801/diff/10001/11106
File
user/test/org/hibernate/jsr303/tck/tests/constraints/application/TckTest.gwt.xml
(right):

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11106#newcode22
user/test/org/hibernate/jsr303/tck/tests/constraints/application/TckTest.gwt.xml:22:
<exclude name="super" />
These includes and excludes should not be necessary

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11110
File
user/test/org/hibernate/jsr303/tck/tests/constraints/constraintcomposition/TckTest.gwt.xml
(right):

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11110#newcode22
user/test/org/hibernate/jsr303/tck/tests/constraints/constraintcomposition/TckTest.gwt.xml:22:
<exclude name="super" />
These include and includes should not be necessary

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11113
File
user/test/org/hibernate/jsr303/tck/tests/constraints/customconstraint/TckTest.gwt.xml
(right):

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11113#newcode22
user/test/org/hibernate/jsr303/tck/tests/constraints/customconstraint/TckTest.gwt.xml:22:
<exclude name="super" />
These excludes and includes should not be necessary

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11116
File
user/test/org/hibernate/jsr303/tck/tests/constraints/groups/TckTest.gwt.xml
(right):

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11116#newcode22
user/test/org/hibernate/jsr303/tck/tests/constraints/groups/TckTest.gwt.xml:22:
<exclude name="super" />
These excludes and includes should not be necessary

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11119
File
user/test/org/hibernate/jsr303/tck/tests/constraints/inheritance/TckTest.gwt.xml
(right):

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11119#newcode22
user/test/org/hibernate/jsr303/tck/tests/constraints/inheritance/TckTest.gwt.xml:22:
<exclude name="super" />
These excludes and includes should not be necessary

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11121
File
user/test/org/hibernate/jsr303/tck/tests/constraints/validatorresolution/TckTest.gwt.xml
(right):

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11121#newcode22
user/test/org/hibernate/jsr303/tck/tests/constraints/validatorresolution/TckTest.gwt.xml:22:
<exclude name="super" />
These excludes and includes should not be necessary

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11125
File
user/test/org/hibernate/jsr303/tck/tests/messageinterpolation/TckTest.gwt.xml
(right):

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11125#newcode22
user/test/org/hibernate/jsr303/tck/tests/messageinterpolation/TckTest.gwt.xml:22:
<exclude name="super" />
These excludes and includes should not be necessary

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11132
File user/test/org/hibernate/jsr303/tck/tests/metadata/TckTest.gwt.xml
(right):

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11132#newcode22
user/test/org/hibernate/jsr303/tck/tests/metadata/TckTest.gwt.xml:22:
<exclude name="super" />
These excludes and includes should not be necessary

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11136
File user/test/org/hibernate/jsr303/tck/tests/validation/TckTest.gwt.xml
(right):

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11136#newcode22
user/test/org/hibernate/jsr303/tck/tests/validation/TckTest.gwt.xml:22:
<exclude name="super" />
These excludes and includes should not be necessary

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11142
File
user/test/org/hibernate/jsr303/tck/tests/validatorfactory/TckTest.gwt.xml
(right):

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11142#newcode22
user/test/org/hibernate/jsr303/tck/tests/validatorfactory/TckTest.gwt.xml:22:
<exclude name="super" />
These excludes and includes should not be necessary

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11150
File user/test/org/jboss/test/audit/JbossTestAudit.gwt.xml (right):

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11150#newcode9
user/test/org/jboss/test/audit/JbossTestAudit.gwt.xml:9: <exclude
name="super/" />
Redundant exclude

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11151
File user/test/org/jboss/testharness/JbossTestHarness.gwt.xml (right):

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11151#newcode9
user/test/org/jboss/testharness/JbossTestHarness.gwt.xml:9: <exclude
name="super/" />
It should not be necessary to exclude super

http://gwt-code-reviews.appspot.com/1267801/diff/10001/11151#newcode10
user/test/org/jboss/testharness/JbossTestHarness.gwt.xml:10: <include
name="api/DeploymentException.java"/>
I beleive source path="" implies all java files are included so this is
redundant

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

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

Reply via email to