search

OpenLiberty / open-liberty

Open Liberty is a highly composable, fast to start, dynamic application server runtime environment
https://openliberty.io
Eclipse Public License 2.0
1.16k stars 599 forks source link

Verify io.openliberty.pages.3.1.internal.factories bnd.bnd #23332

Open isaacrivriv opened 2 years ago

isaacrivriv commented 2 years ago

While working on version ranges updates, noticed that there were a couple of ranges that were using an incorrect version see io.openliberty.pages.3.1.internal.factories/bnd.bnd snippet below

   org.apache.el; version="3.0.0", \
   org.apache.el.lang; version="3.0.0", \
   org.apache.el.parser; version="3.0.0", \
   org.apache.el.util; version="3.0.0", \

This should be updated to the correct EL 5.0 packages and would need to verify that all the other package versions are also correct or if no longer used, if they can be cleaned up.

https://github.com/OpenLiberty/open-liberty/pull/23262/files

isaacrivriv commented 1 year ago

The only remaining changes for these packages would be on the JSP 2.3 bundle and JSP bundle for transformations on EE9 and any other additional usage of these packages. This work was completed for Pages 3.1 EE10 bundle in the mentioned PR which brought forward this issue.

This does not require any functional changes since it establishes a lower version range and the bundles pulled in carry the correct package versions on each feature. This change would be to clearly establish the appropriate versions for reading and maintainability purposes.

isaacrivriv commented 1 year ago

Moving this to post-GA EE10