Adobe-Consulting-Services / acs-aem-commons

http://adobe-consulting-services.github.io/acs-aem-commons/
Apache License 2.0
456 stars 603 forks source link

Extract OSGI configuration to new package ui.config #3434

Closed SzymonSzafraniec closed 1 month ago

SzymonSzafraniec commented 2 months ago

Corresponding issue: https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/3433

Currently acs-aem-commons-all package mixes subpackages with configuration files. This change bundles the configurations in a separate package which is also included in the all package.

Another benefit is that it allows other projects to include individual ACS Commons packages while not missing the OSGI configurations (e.g. include the acs-aem-commons-bundle and acs-aem-commons-ui.config only).

Also, by including individual packages separately it's possible to control the packages installation order. This is currently impossible since including the acs-aem-commons-all is the only way to install OSGI Configurations.

davidjgonzalez commented 1 month ago

@kwin any problems with this?