You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@servicemix.apache.org by GitBox <gi...@apache.org> on 2022/12/29 22:18:09 UTC

[GitHub] [servicemix-bundles] dalbani opened a new pull request, #195: Fix issue related to mandatory dependency in OkHttp bundle

dalbani opened a new pull request, #195:
URL: https://github.com/apache/servicemix-bundles/pull/195

   When trying to use `org.apache.servicemix.bundles:org.apache.servicemix.bundles.okhttp:4.10.0_2`, I'm running into the issue of `dalvik.system.*` being required.
   For the record, out of `MANIFEST.MF`:
   ```
   Import-Package:
   okio;version="[3.0.0,4)",
   org.bouncycastle.jsse;resolution:=optional,
   org.bouncycastle.jsse.provider;resolution:=optional,
   org.openjsse.javax.net.ssl;resolution:=optional,
   org.openjsse.net.ssl;resolution:=optional,
   android.net.http;resolution:=optional,
   android.net.ssl;resolution:=optional,
   android.os;resolution:=optional,
   android.security;resolution:=optional,
   android.util;resolution:=optional,
   org.conscrypt;resolution:=optional,
   dalvik.system,
   javax.net,
   javax.net.ssl,
   javax.security.auth.x500,
   kotlin,
   kotlin.annotation,
   kotlin.collections,
   kotlin.comparisons,
   kotlin.internal,
   kotlin.io,
   kotlin.jvm,
   kotlin.jvm.functions,
   kotlin.jvm.internal,
   kotlin.jvm.internal.markers,
   kotlin.ranges,
   kotlin.sequences,
   kotlin.text,
   okhttp3;version="[4.10,5)",
   okhttp3.internal;version="[4.10,5)",
   okhttp3.internal.authenticator;version="[4.10,5)",
   okhttp3.internal.cache;version="[4.10,5)",
   okhttp3.internal.concurrent;version="[4.10,5)",
   okhttp3.internal.connection;version="[4.10,5)",
   okhttp3.internal.http;version="[4.10,5)",
   okhttp3.internal.http1;version="[4.10,5)",
   okhttp3.internal.http2;version="[4.10,5)",
   okhttp3.internal.io;version="[4.10,5)",
   okhttp3.internal.platform;version="[4.10,5)",
   okhttp3.internal.platform.android;version="[4.10,5)",
   okhttp3.internal.proxy;version="[4.10,5)",
   okhttp3.internal.publicsuffix;version="[4.10,5)",
   okhttp3.internal.tls;version="[4.10,5)",
   okhttp3.internal.ws;version="[4.10,5)",
   sun.security.ssl
   ```
   
   I suggest to mark `dalvik.system` as optional.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@servicemix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [servicemix-bundles] jbonofre commented on pull request #195: Fix issue related to mandatory dependency in OkHttp bundle

Posted by GitBox <gi...@apache.org>.
jbonofre commented on PR #195:
URL: https://github.com/apache/servicemix-bundles/pull/195#issuecomment-1368494679

   Fixed as part of https://github.com/apache/servicemix-bundles/commit/2eb4d093a183831655329cfa78aa40515eb94358 (I did other cleanups).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@servicemix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [servicemix-bundles] jbonofre closed pull request #195: Fix issue related to mandatory dependency in OkHttp bundle

Posted by GitBox <gi...@apache.org>.
jbonofre closed pull request #195: Fix issue related to mandatory dependency in OkHttp bundle
URL: https://github.com/apache/servicemix-bundles/pull/195


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@servicemix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [servicemix-bundles] dalbani commented on pull request #195: Fix issue related to mandatory dependency in OkHttp bundle

Posted by GitBox <gi...@apache.org>.
dalbani commented on PR #195:
URL: https://github.com/apache/servicemix-bundles/pull/195#issuecomment-1367635992

   Actually, the upstream OkHttp project _does_ contain OSGi configuration in the build setup: https://github.com/square/okhttp/blob/master/okhttp/build.gradle.kts#L126.
   It's not clear to me why the official JARs don't contain any OSGi metadata then. 🤔
   
   Anyway, it looks like the following change should be included as well:
   ```
   sun.security.ssl;resolution:=optional,*
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@servicemix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org