enapps-enorman commented on PR #309:
URL: https://github.com/apache/felix-dev/pull/309#issuecomment-2077778050

   > It would be interesting to see if this approach would also work for the 
jetty12 bundle, although i'm not a fan of needing to install all those jetty 
bundles in the OSGi runtime. But that's a matter of taste i suppose.
   
   Yes that could be interesting, but I am not going to spend any time on 
experimenting with jetty12 until I get a yes or no regarding whether there is 
any chance that this proposal is going to be accepted or not.  The source code 
is available here if you want to try something similar on jetty12 and tell us 
if it worked.
   
   FYI: In my projects, I am using the "light" classifier variant of the 
http.jetty bundle which doesn't embed the classes from any of the jetty bundles 
so I am used to declaring all of the individual jetty bundles that I need.  I 
prefer this so I have the flexibility to switch to a newer release of jetty 
without waiting for felix to do a new release with the newer version of the 
jetty classes embedded.
   
   If the consensus is to embed the content of the extra websocket-* bundles 
into the non-light variant then I wound not care too much either way.  It would 
just make the binary a bit bigger.


-- 
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-unsubscr...@felix.apache.org

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

Reply via email to