Re: [gwt-contrib] Re: user tests fail to compile
On Sunday, June 15, 2014 3:07:17 AM UTC+2, Jens wrote: The gwt.javac macro in common.ant.xml excludes **/super/** by default, but the call point in compile.tests in user/build.xml overrides the excludes= and removed the **/super/** exclusion. I think that was a bug (by oversight). Hmm I don't see anything that tells gwt.javac to exclude **/super/** by default. But maybe it is a good idea to add it. The compile task explicitly excludes **/super/** in user/build.xml and the compile.tests task does not exclude it because there was no need to do so until now. Whoops! Totally misread my git grep results! ;-) However I am wondering if the super source files in #7865 are better placed into user/test-super ? I guess thats why test-super exists (?) There's no strict rule. com.google.gwt.regexp.super is in user/src, not user/test for instance. I think the super/ root was originally more about IDEs than a real will to segregate those files. I would prefer to have one strategy instead of two. But I guess that will happen anyways sooner or later with gwt modularization. I remember making that same remark when c.g.g.regexp was added that broke the rule, but that ship has sailed years ago. I must say I wouldn't be opposed to having everything in src/ and test/ and removing super/ and test-super/ altogether (for user/ we have test_i18n_bar and test_i18n_dollar, that was added a while back [1,2], I wonder if we couldn't refactor the tests with a single source root) [1] https://code.google.com/p/google-web-toolkit/source/detail?r=3855 [2] https://gwt.googlesource.com/gwt/+/253054 -- You received this message because you are subscribed to the Google Groups GWT Contributors group. To unsubscribe from this group and stop receiving emails from it, send an email to google-web-toolkit-contributors+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/google-web-toolkit-contributors/119df8cc-9a43-4262-9b5d-898358a9b1e6%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[gwt-contrib] Re: user tests fail to compile
Thanks Jens, that worked great. This .classpath file is in source control, so this should probably go into a patch. Is anyone working on one already? classpathentry excluding=**/super/** kind=src path=core/test/ On Friday, June 13, 2014 4:28:29 AM UTC-4, Jens wrote: In Eclipse any folder that contains super source should be excluded from the classpath so that its not treated as Java package but instead a simple folder with java files. So go to gwt-user project settings - build path - source. Select gwt-user/core/test - Excluded and click edit. Add exclusion pattern **/super/** . Then the errors are gone in Eclipse. -- J. -- You received this message because you are subscribed to the Google Groups GWT Contributors group. To unsubscribe from this group and stop receiving emails from it, send an email to google-web-toolkit-contributors+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/google-web-toolkit-contributors/d545e8aa-d94d-4043-b7ce-e286ce09d263%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[gwt-contrib] Re: user tests fail to compile
I've uploaded a patch for this: https://gwt-review.googlesource.com/#/c/7890/3 On Saturday, June 14, 2014 10:22:47 AM UTC-4, Michael Prentice wrote: Thanks Jens, that worked great. This .classpath file is in source control, so this should probably go into a patch. Is anyone working on one already? classpathentry excluding=**/super/** kind=src path=core/test/ On Friday, June 13, 2014 4:28:29 AM UTC-4, Jens wrote: In Eclipse any folder that contains super source should be excluded from the classpath so that its not treated as Java package but instead a simple folder with java files. So go to gwt-user project settings - build path - source. Select gwt-user/core/test - Excluded and click edit. Add exclusion pattern **/super/** . Then the errors are gone in Eclipse. -- J. -- You received this message because you are subscribed to the Google Groups GWT Contributors group. To unsubscribe from this group and stop receiving emails from it, send an email to google-web-toolkit-contributors+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/google-web-toolkit-contributors/6d1e9862-1217-49cd-994c-7faf9dfc36be%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: [gwt-contrib] Re: user tests fail to compile
I'm not sure why or when that happened. I checked in a big patch (#7600) recently that may or may not have caused it. I wonder if someone can seek back before my commit and see if the failure is still present. Its not your commit that caused these errors. The RequestFactory error simply exists because no one seems to keep requestfactory-apt.jar up-to-date in the SVN tools repo. Maybe the build script can be updated to first build requestfactory-apt.jar and then use this build version in other tasks. That way we don't need to have requestfactory-apt.jar in the SVN repo. The other error has been introduced in https://gwt-review.googlesource.com/#/c/7865/ . Prior to this CL there wasn't any super source folder present in /user/test so ant script and the eclipse .classpath file did not exclude **/super/**. However I am wondering if the super source files in #7865 are better placed into user/test-super ? I guess thats why test-super exists (?) and we would not need to change the ant build script and eclipse .classpath. -- J. -- You received this message because you are subscribed to the Google Groups GWT Contributors group. To unsubscribe from this group and stop receiving emails from it, send an email to google-web-toolkit-contributors+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/google-web-toolkit-contributors/a388b7b9-a4b2-40ba-95ca-5a0f34cab81a%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: [gwt-contrib] Re: user tests fail to compile
The gwt.javac macro in common.ant.xml excludes **/super/** by default, but the call point in compile.tests in user/build.xml overrides the excludes= and removed the **/super/** exclusion. I think that was a bug (by oversight). Hmm I don't see anything that tells gwt.javac to exclude **/super/** by default. But maybe it is a good idea to add it. The compile task explicitly excludes **/super/** in user/build.xml and the compile.tests task does not exclude it because there was no need to do so until now. However I am wondering if the super source files in #7865 are better placed into user/test-super ? I guess thats why test-super exists (?) There's no strict rule. com.google.gwt.regexp.super is in user/src, not user/test for instance. I think the super/ root was originally more about IDEs than a real will to segregate those files. I would prefer to have one strategy instead of two. But I guess that will happen anyways sooner or later with gwt modularization. -- J. -- You received this message because you are subscribed to the Google Groups GWT Contributors group. To unsubscribe from this group and stop receiving emails from it, send an email to google-web-toolkit-contributors+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/google-web-toolkit-contributors/d185e6a2-43a7-4d6d-b5a2-96fa5bbfa5c7%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[gwt-contrib] Re: user tests fail to compile
Eclipse still unhappy though of course. In Eclipse any folder that contains super source should be excluded from the classpath so that its not treated as Java package but instead a simple folder with java files. So go to gwt-user project settings - build path - source. Select gwt-user/core/test - Excluded and click edit. Add exclusion pattern **/super/** . Then the errors are gone in Eclipse. -- J. -- You received this message because you are subscribed to the Google Groups GWT Contributors group. To unsubscribe from this group and stop receiving emails from it, send an email to google-web-toolkit-contributors+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/google-web-toolkit-contributors/9a8c52bd-fd0b-46ed-9f2e-c5fbf4eba38b%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[gwt-contrib] Re: user tests fail to compile
With some help from Jens Nehlmeier over in ##gwt, it looks like there are two distinct issues preventing the build from passing presently The first is that the class ImmediateCompileFails does in fact cause problems with compiling - the simplest fix was to tell the compile.tests target to leave off compiling any super sources: diff --git a/user/build.xml b/user/build.xml index 274f4ef..54fc17d 100755 --- a/user/build.xml +++ b/user/build.xml @@ -147,7 +147,7 @@ depends=compile.dev.tests, compile.emma.if.enabled unless=compile.tests.complete mkdir dir=${javac.junit.out}/ -gwt.javac srcdir=test excludes=com/google/gwt/langtest/** destdir=${javac.junit.out} +gwt.javac srcdir=test excludes=com/google/gwt/langtest/**,**/super/** destdir=${javac.junit.out} classpath pathelement location=${javac.out}/ pathelement location=${gwt.tools.lib}/junit/junit-4.8.2.jar/ The second issue is that the requestfactory-apt.jar in the GWT Tools SVN repo appears to be out of date. Doing a dist-dev build and copying the newly generated -apt over to my local SVN checkout seems to bring this back into working order. -- You received this message because you are subscribed to the Google Groups GWT Contributors group. To unsubscribe from this group and stop receiving emails from it, send an email to google-web-toolkit-contributors+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/google-web-toolkit-contributors/79c83872-2dfc-4272-be9d-9b08845ace22%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[gwt-contrib] Re: user tests fail to compile
I hit the same thing last night. The fix you suggested may help with Ant, but it does not resolve the compile errors in Eclipse :( It doesn't look like Eclipse has any Annotations to ignore these types of severe errors either, I just pulled in the 4 commits from today and they don't help with this: - List cannot be resolved to a type ImmediateCompileFails.java - /gwt-user/core/test/com/google/gwt/dev/testdata/incrementalbuildsystem/super/com/google/gwt/dev/testdata/incrementalbuildsystem line 20 Java Problem - ArrayList cannot be resolved to a type ImmediateCompileFails.java - /gwt-user/core/test/com/google/gwt/dev/testdata/incrementalbuildsystem/super/com/google/gwt/dev/testdata/incrementalbuildsystem line 20 Java Problem - The declared package com.google.gwt.dev.testdata.incrementalbuildsystem does not match the expected package com.google.gwt.dev.testdata.incrementalbuildsystem.super.com.google.gwt.dev.testdata.incrementalbuildsystem - ImmediateCompileFails.java /gwt-user/core/test/com/google/gwt/dev/testdata/incrementalbuildsystem/super/com/google/gwt/dev/testdata/incrementalbuildsystem line 14 Java Problem I also get errors on the Ant build even after replacing the requestfactory-apt.jar in the GWT Tools SVN repo with the output of my dist-dev build. -- You received this message because you are subscribed to the Google Groups GWT Contributors group. To unsubscribe from this group and stop receiving emails from it, send an email to google-web-toolkit-contributors+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/google-web-toolkit-contributors/98c574dc-ba43-4906-926b-9804f39a7533%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
[gwt-contrib] Re: user tests fail to compile
Full clean Ant build just finished and it passed with the build.xml tweak and the requestfactory-apt.jar update. Thanks Colin. Eclipse still unhappy though of course. Is there an issue/patch for this? Has anyone tracked down the original change set that caused this? I might be able to take a look in the morning if not. On Friday, June 13, 2014 12:34:21 AM UTC-4, Michael Prentice wrote: I hit the same thing last night. The fix you suggested may help with Ant, but it does not resolve the compile errors in Eclipse :( It doesn't look like Eclipse has any Annotations to ignore these types of severe errors either, I just pulled in the 4 commits from today and they don't help with this: - List cannot be resolved to a type ImmediateCompileFails.java - /gwt-user/core/test/com/google/gwt/dev/testdata/incrementalbuildsystem/super/com/google/gwt/dev/testdata/incrementalbuildsystem line 20 Java Problem - ArrayList cannot be resolved to a type ImmediateCompileFails.java - /gwt-user/core/test/com/google/gwt/dev/testdata/incrementalbuildsystem/super/com/google/gwt/dev/testdata/incrementalbuildsystem line 20 Java Problem - The declared package com.google.gwt.dev.testdata.incrementalbuildsystem does not match the expected package com.google.gwt.dev.testdata.incrementalbuildsystem.super.com.google.gwt.dev.testdata.incrementalbuildsystem - ImmediateCompileFails.java /gwt-user/core/test/com/google/gwt/dev/testdata/incrementalbuildsystem/super/com/google/gwt/dev/testdata/incrementalbuildsystem line 14 Java Problem I also get errors on the Ant build even after replacing the requestfactory-apt.jar in the GWT Tools SVN repo with the output of my dist-dev build. -- You received this message because you are subscribed to the Google Groups GWT Contributors group. To unsubscribe from this group and stop receiving emails from it, send an email to google-web-toolkit-contributors+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/google-web-toolkit-contributors/93d20ccf-91c2-491a-8519-f321bbab4535%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.