Hi Reto
I am the author of this 'enhancement'.
I completely agree with you, that it's not ideal to include com.sun.*
packages, because it enforces a sun jdk. On the other hand the
requirements are not in our hands and we could only correct'em by
writing (and submitting) a patch to these 3rd party libs.
The reason behind it is that we need a 3rd party lib, that allows us to
import users from the wwf nz ldap. I tried 2 frameworks (spring ldap and
openldap java API). Bot have dependencies to com.sun.* packages - they
both seem to directly use sun packages. It looks like we have no choice
here or we'd decide to write a patch that hides the com.sun.*-classes
behind an interface with interchangeable (optional) implementations.
Cheers
André
On 06/03/2010 02:36 PM, Reto Bachmann-Gmuer wrote:
Hi Manuel
On http://java.sun.com/products/jdk/faq/faq-sun-packages.html I read:
"In general, writing java programs that rely on sun.* is risky: they
are not portable, and are not supported. "
I think we should remove the sun dependencies. Why did you add this
ssl dependency? Would the BouncyCastleProvider be an open source
alternative?
Cheers,
reto
On Wed, Jun 2, 2010 at 4:13 PM,<[email protected]> wrote:
Author: mir
Date: Wed Jun 2 14:13:40 2010
New Revision: 950571
URL: http://svn.apache.org/viewvc?rev=950571&view=rev
Log:
added additional com.sun packages to 'org.osgi.framework.system.packages.extra'
Modified:
incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.launcher.storageless.parent/org.apache.clerezza.platform.launcher.storageless/src/main/java/org/apache/clerezza/platform/launcher/Main.java
Modified:
incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.launcher.storageless.parent/org.apache.clerezza.platform.launcher.storageless/src/main/java/org/apache/clerezza/platform/launcher/Main.java
URL:
http://svn.apache.org/viewvc/incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.launcher.storageless.parent/org.apache.clerezza.platform.launcher.storageless/src/main/java/org/apache/clerezza/platform/launcher/Main.java?rev=950571&r1=950570&r2=950571&view=diff
==============================================================================
---
incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.launcher.storageless.parent/org.apache.clerezza.platform.launcher.storageless/src/main/java/org/apache/clerezza/platform/launcher/Main.java
(original)
+++
incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.launcher.storageless.parent/org.apache.clerezza.platform.launcher.storageless/src/main/java/org/apache/clerezza/platform/launcher/Main.java
Wed Jun 2 14:13:40 2010
@@ -254,9 +254,12 @@ public class Main implements BundleActiv
configProps.put("org.osgi.service.http.port", port);
}
configProps.put("org.osgi.framework.system.packages.extra",
-
"sun.rmi.server;sun.rmi.transport;sun.rmi.transport.tcp;" +
-
"sun.rmi.registry;com.sun.jndi.rmi.registry;sun.io;" +
- "com.sun.jmx;sun.misc;");
+
"sun.rmi.server;sun.rmi.transport;sun.rmi.transport.tcp;"
+ +
"sun.rmi.registry;com.sun.jndi.rmi.registry;"
+ + "sun.io;"
+ + "com.sun.jmx;"
+ + "sun.misc;"
+ +
"com.sun.net.ssl.internal.ssl;com.sun.net.ssl");
configProps.put("org.ops4j.pax.url.mvn.repositories",
getCommaSeparatedListOfMavenRepos());
