Java 7 is the version for most (all?) of the Maven project. It would be a bit confusing to have a special exception for this. I'm also a bit worried compiling this plugin for Java 9 might cause problems when it's included in a pom.xml that also supports Java 7. E.g. some weird scenario where a build that doesn't actually use the jlink plugin nonetheless gets confused by trying to read its .class files or some such.
Nothing concrete, and I may just be paranoid, but unless there's a clear reason we need to update the Java version to make this plugin work, I'd prefer to stick to Java 7. On Wed, Feb 19, 2020 at 2:59 AM Olivier Lamy <[email protected]> wrote: > > yup it's only a cli generator/executor and this upgrade is not really > mandatory. > but it just doesn't make much sense for me to be java7 as the goal is to > execute a cli which is only available >=9 :) > > > On Wed, 19 Feb 2020 at 17:53, Romain Manni-Bucau <[email protected]> > wrote: > > > If upgrading helps I dont see how it could hurt but I see it as a process > > executor so shouldnt be important, no? > > > > Le mer. 19 févr. 2020 à 08:39, Olivier Lamy <[email protected]> a écrit : > > > > > Hi > > > I was looking at jlink plugin and trying it. > > > I'm using java11 and got issues because of this.. > > > Easy fix is upgrading plexus-java version > > > > > > But I noticed java for jlink plugin is java7. > > > As jlink has been released with java9 I wonder if this java7 minimum > > really > > > makes sense? > > > > > > cheers > > > -- > > > Olivier Lamy > > > http://twitter.com/olamy | http://linkedin.com/in/olamy > > > > > > > > -- > Olivier Lamy > http://twitter.com/olamy | http://linkedin.com/in/olamy -- Elliotte Rusty Harold [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
