Community
Participate
Working Groups
The bundle org.eclipse.team.svn.core has a hard dependency to the com.ibm.icu bundle. Please replace this "Require-Bundle" declaration with "Import-Package" declarations for com.ibm.icu.text and com.ibm.icu.util. The reason for this is that a hard requirement on the bundle will force deployments to include the fairly huge full bundle (over 2MB) instead of the minimalistic com.ibm.icu.base. The huge bundle is in most cases completely unnecessary when running a modern JVM (Java 1.6) since most of its functionality has been merged in to the JVM runtime.
Created attachment 122859 [details] Patch that removes the dependency Apparently, the dependency is not currently used so this patch just removes it.
Created attachment 122868 [details] Patch that replaces the dependency with package imports The previous patch was flawed. My IDE didn't rebuild properly so I missed the actual usage of com.ibm.icu. This patch replaces the bundle requirement with imported packages. It also removes the StringId.isStringId() method since it references the com.ibm.icu.lang package which for some reason is not present in the com.ibm.icu.base bundle. The StringId.isStringId() is not referenced from any of the team.svn bundles.
(In reply to comment #2) > Created an attachment (id=122868) > Patch that replaces the dependency with package imports > > The previous patch was flawed. My IDE didn't rebuild properly so I missed the > actual usage of com.ibm.icu. > > This patch replaces the bundle requirement with imported packages. It also > removes the StringId.isStringId() method since it references the > com.ibm.icu.lang package which for some reason is not present in the > com.ibm.icu.base bundle. The StringId.isStringId() is not referenced from any of > the team.svn bundles. Thomas, I applied your patch, thanks a lot. I also removed strong dependency to com.ibm.icu from UI plugin
