ghenzler commented on a change in pull request #11: [FELIX-6246] Packaged External Library Packages inside referenced Health Check Bundles URL: https://github.com/apache/felix-dev/pull/11#discussion_r399718728
########## File path: healthcheck/webconsoleplugin/bnd.bnd ########## @@ -1,10 +1,6 @@ -Bundle-Category: healthcheck - -Bundle-Description: ${project.description} - -Bundle-DocURL: https://felix.apache.org - -Bundle-License: Apache License, Version 2.0 - -Bundle-Vendor: The Apache Software Foundation - +Bundle-Category : healthcheck +Bundle-Description : ${project.description} +Bundle-DocURL : https://felix.apache.org +Bundle-License : Apache License, Version 2.0 +Bundle-Vendor : The Apache Software Foundation +-conditionalpackage : org.apache.commons.lang3.* Review comment: So for server applications we do use the webconsole in production, I just thought that for `small IoT devices` you wouldn't use it at all (and maybe rather the gogo shell). But if you use it that's fine obviously, it just makes it one bundle more in your setup and the use case for embedding even weaker. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services