Summary: | Building/cleaning projects in RAD does not work properly | ||||||
---|---|---|---|---|---|---|---|
Product: | [Tools] AJDT | Reporter: | Ricardo <r_olivieri> | ||||
Component: | Core | Assignee: | Helen Beeken <hlhawkins> | ||||
Status: | RESOLVED FIXED | QA Contact: | |||||
Severity: | critical | ||||||
Priority: | P3 | ||||||
Version: | 1.3.0 | ||||||
Target Milestone: | --- | ||||||
Hardware: | PC | ||||||
OS: | Windows 2000 | ||||||
Whiteboard: | |||||||
Attachments: |
|
Description
Ricardo
2005-06-23 11:56:16 EDT
Created attachment 23870 [details]
web project to reproduce problem
Hi Ricardo, Thanks for providing a simple project to use to reproduce this problem, which I have been able to do. The reason this behaviour is happening is that the fix for bug 86941 involved providing an empty implementation of the clean(..) method. This is ok if when you do Project > Clean you have the "start a build immediately" box checked. However, if you don't then the behaviour you're seeing fits what we have. Therefore, the fix is to do as in the javabuilder i.e. to clear all problem markers, clear the state (something we can now do) and clear the output folder. Hi Ricardo, Just to let you know that the fix for this is on it's way and should be in a development build sometime tomorrow. I'm just doing some final testing as this is a sensitive area to put fixes in and I'll update this bug report when there's a build available for you. Thanks, Helen This bug raises two issues: 1) Clean doesn't remove .class files from output directory and doesn't remove problems markers 2) when you choose to build after a clean then the .class files aren't there in the output directory Regarding (1) the fix and tests have been integrated and are available in build BUILD COMPLETE - build.432 Date of build: 06/28/2005 11:39:23 Time to build: 41 minutes 3 seconds Last changed: 06/28/2005 11:22:58 which is available via the update site (see the downloads page for more information: http://www.eclipse.org/ajdt/downloads ) The fix involved implementing the clean(..) method in AJBuilder to 1) remove the incremental state 2) remove the error and problem markers for the selected project 3) clean the output folder The test for this is in AJBuilderTest. Leaving this bug open to look at the refreshing issue. Thanks, Helen The problem that in order to see the class files you had to manually refresh is due to us doing some cleanup. We used to refresh the entire project after a build (which is why the class files were being seen). However, this is quite expensive and not always required. Therefore, unless there were dependent projects we only refreshed the direct members (IResource.DEPTH_ONE). This, of course, doesn't cover the various levels in the output directory. The fix is to have an extra step where we refresh the output directory. The manual builder test and fix have been integrated and are contained in build: BUILD COMPLETE - build.433 Date of build: 06/28/2005 14:55:06 Time to build: 38 minutes 20 seconds Last changed: 06/28/2005 14:40:22 which is available via the update site (see the downloads page for more information: http://www.eclipse.org/ajdt/downloads ). I'm leaving this bug open to fix the final thing which is that when you do Project > Clean on an AJ project, the output directory is cleared, but you have to do a manual refresh to see the results. The final fix and test have been integrated. When we clean the output folder from a clean build, provided build automatically is off we do a refresh of the output folder. We only do the refresh if build automatically is off because we refresh anyway as part of the build. The eclipse servers are overloaded by people downloading eclipse 3.1. This is messing with our build system (as we do a cvs lookup for any changes). Therefore, a good build containing this fix will probably not be available until tomorrow. I'll update and close the bug when there's a build available. There is a build now which contains the last fix: BUILD COMPLETE - build.436 Date of build: 06/30/2005 09:08:31 Time to build: 41 minutes 31 seconds which is available via the update site (see the downloads page for more information: http://www.eclipse.org/ajdt/downloads ) Therefore, closing this bug as fixed. |