Reviewers: jlabanca,

Description:
This fixes the ServletMappingSuite buildbreak.

The issue appears to be that a <pathelement location=""/> is taken, much
like a :: substring in -classpath, to mean "this directory" rather than
"nothing": it puts gwt-root/user onto the classpath.  And, when the
test.ServletMappingSuite module is in play, "test/..." is in the
<source> path, so gwt-root/user/test/... comes into play and wreaks
havoc.

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

Affected files:
   common.ant.xml


Index: common.ant.xml
===================================================================
--- common.ant.xml      (revision 6874)
+++ common.ant.xml      (working copy)
@@ -179,7 +179,8 @@
        <antcall target="-create.emma.coverage.if.enabled">
          <param name="test.emma.coverage" value="@{test.emma.coverage}"/>
        </antcall>
-      <condition property="emma.lib"  
value="${emma.dir}/emma-2.0.5312-patched.jar" else="">
+      <condition property="emma.lib"  
value="${emma.dir}/emma-2.0.5312-patched.jar"
+            else="${emma.dir}/no-emma-requested">
          <isset property="emma.enabled" />
        </condition>



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

Reply via email to