I'm OK with Java 17 as the runtime minimum. Gary
On Tue, Dec 5, 2023, 7:19 AM Piotr P. Karwasz <piotr.karw...@gmail.com> wrote: > Hi Volkan, > > On Tue, 5 Dec 2023 at 11:50, Volkan Yazıcı <vol...@yazi.ci> wrote: > > > > I have just updated `main` to require Java 17 for compiling and to target > > Java 17. Previously both were Java 11. If you have objections, please > > discuss in this thread. > > I am Ok with bumping the bytecode to Java 17. > > Regarding the build requirements I would give by **default** a larger > spectrum of possible JDKs. > Right now we only accept JDK 17 in the default Maven profile. A Log4j > contributor is forced to install JDK 17 for a build to succeed, which > might be annoying. Therefore it might be better to: > > * set the Enforcer plugin to accept > [minimum_JDK_required_by_Maven_plugins,) > * set [17,18) **only** in the `release` plugin. > > BTW: when we bump BND to 7.x, minimum_JDK_required_by_Maven_plugins will > be 17. > > Piotr >
