Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 323375

Summary: [backport] fix for bug 296576 in 3.0 and 3.1 branches for WTP
Product: [WebTools] WTP ServerTools Reporter: Krzysztof Kazmierczyk <kazm>
Component: jst.serverAssignee: Angel Vera <arvera>
Status: RESOLVED FIXED QA Contact: Angel Vera <arvera>
Severity: normal    
Priority: P3 CC: krzysztof.daniel
Version: 3.0.5   
Target Milestone: 3.0.5 P   
Hardware: PC   
OS: All   
See Also: https://bugs.eclipse.org/bugs/show_bug.cgi?id=296576
Whiteboard:
Attachments:
Description Flags
patch for 3.0-maintenance
none
Simpler patch
none
v1.1 none

Description Krzysztof Kazmierczyk CLA 2010-08-23 06:14:02 EDT
Build Identifier: 

We need fix for bug 296576 (RuntimeClasspathProviderWrapper methods can be called by multiple threads but aren't thread safe) in branches 3.0-maintenance and 3.1-maintenance.
The fix is necessary for Eclipse based product to increase user satisfaction.

Reproducible: Always
Comment 1 Krzysztof Kazmierczyk CLA 2010-08-23 06:16:42 EDT
Patch for 3.1 is the same as to 3.2 branch and available under https://bugs.eclipse.org/bugs/attachment.cgi?id=157464
Comment 2 Krzysztof Kazmierczyk CLA 2010-08-23 06:52:47 EDT
Created attachment 177202 [details]
patch for 3.0-maintenance

The patch for 3.0.
We needed to add import of java.util.Collections and manually modify changes for resolveClasspathContainerImpl method.
Comment 3 Angel Vera CLA 2010-09-02 10:37:25 EDT
Kryztof, why was the RutnimeClasspathProviderWrapper part not included into your 3.0 patch?
Comment 4 Krzysztof Daniel CLA 2010-09-06 04:03:50 EDT
*** Bug 324550 has been marked as a duplicate of this bug. ***
Comment 5 Krzysztof Daniel CLA 2010-09-06 04:05:39 EDT
Created attachment 178241 [details]
Simpler patch

This is much simpler patch which also seems to resolve the issue.
Comment 6 Angel Vera CLA 2010-10-13 16:18:21 EDT
Created attachment 180824 [details]
v1.1

Added updated copyrights from the patch# 178241
Comment 7 Angel Vera CLA 2010-11-02 15:11:41 EDT
It looks like this fix was already included in the patches build, but it was
never marked as fixed. I checked the R3_0_5_patches version of RuntimeClasspathProviderDelegate and it
does indeed look like the changes were committed. I am now marking this as
fixed.
Comment 8 Angel Vera CLA 2010-11-24 10:56:05 EST
*** Bug 330478 has been marked as a duplicate of this bug. ***