Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 269569 - Add ICU4J replacement jar to remote server packages
Summary: Add ICU4J replacement jar to remote server packages
Status: RESOLVED FIXED
Alias: None
Product: PTP
Classification: Tools
Component: RDT (show other bugs)
Version: 2.2   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 2.2   Edit
Assignee: Chris Recoskie CLA
QA Contact: Greg Watson CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-03-20 15:29 EDT by Vivian Kong CLA
Modified: 2011-01-31 08:15 EST (History)
0 users

See Also:


Attachments
proposed patch (5.02 KB, patch)
2009-03-20 15:29 EDT, Vivian Kong CLA
no flags Details | Diff
updated patch (7.95 KB, patch)
2009-03-25 11:24 EDT, Vivian Kong CLA
no flags Details | Diff
updated patch with additional changes (8.75 KB, patch)
2009-03-25 11:35 EDT, Vivian Kong CLA
g.watson: iplog+
Details | Diff
patch (3.66 KB, patch)
2009-04-13 09:59 EDT, Vivian Kong CLA
g.watson: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Vivian Kong CLA 2009-03-20 15:29:51 EDT
Created attachment 129495 [details]
proposed patch

CDT 6.0 has a dependency on ICU4J (bug 254200).  In order for RDT to work with CDT 6.0, we need to add ICU4J replacement jar to the remote server packages.
Comment 1 Vivian Kong CLA 2009-03-25 11:24:08 EDT
Created attachment 129847 [details]
updated patch
Comment 2 Vivian Kong CLA 2009-03-25 11:27:07 EDT
The updated patch also updates the build script to include the latest version numbers of the Eclipse and CDT plug-in dependencies (Eclipse 3.5M6 and CDT 6.0M6).
Comment 3 Vivian Kong CLA 2009-03-25 11:35:44 EDT
Created attachment 129850 [details]
updated patch with additional changes
Comment 4 Greg Watson CLA 2009-04-10 08:43:50 EDT
I'm getting conflicts on the patches. Can you update and create a new patch?
Comment 5 Greg Watson CLA 2009-04-10 12:42:19 EDT
Nevermind, I applied it anyway. Please check that it is ok.
Comment 6 Vivian Kong CLA 2009-04-13 09:59:56 EDT
Created attachment 131647 [details]
patch

Thanks for checking the patches in Greg.  I checked out the latest from CVS and there are a few more changes I need to make for this bug.  Please see the attached patch for HEAD.  Thanks.
Comment 7 Greg Watson CLA 2009-04-13 11:09:46 EDT
Applied.
Comment 8 Vivian Kong CLA 2009-04-13 13:04:40 EDT
Thanks Greg.  Marking it as fixed.