Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-52014

EOL JENKINS-20679

    XMLWordPrintable

Details

    Description

      Previously Java versions strings were 1.x, so the plugin-pom hardcodes a java.level property of "8" like so:

      <source>1.${java.level}</source>

      See: https://github.com/jenkinsci/plugin-pom/blob/master/pom.xml#L604

      https://blog.codefx.org/java/java-9-migration-guide/#New-Version-Strings

      But that won't work if we need to use a version of 9+, where the version string is now just "9" etc

      In general we need multi-release JAR support here because if we change the way the java.level is handled then we don't inadvertently break Java 8 or earlier which expect the 1.x format.

      Attachments

        Activity

          oleg_nenashev Oleg Nenashev added a comment -

          I would vote against Java9-only plugins in short term.
          We likely need to get the Multi-Release JAR support in Jenkins tooling over the fence

          oleg_nenashev Oleg Nenashev added a comment - I would vote against Java9-only plugins in short term. We likely need to get the Multi-Release JAR support in Jenkins tooling over the fence
          svanoort Sam Van Oort added a comment -

          I would agree with that being a better solution – perhaps I should rephrase the issue to better express the problem.

          svanoort Sam Van Oort added a comment - I would agree with that being a better solution – perhaps I should rephrase the issue to better express the problem.
          oleg_nenashev Oleg Nenashev added a comment - https://github.com/jenkinsci/plugin-pom/pull/133  
          basil Basil Crow added a comment -

          We may want to rethink the Minimum-Java-Version setting in maven-hpi-plugin as part of this effort.

          basil Basil Crow added a comment - We may want to rethink the Minimum-Java-Version setting in maven-hpi-plugin as part of this effort.
          basil Basil Crow added a comment - Mailing list thread started: https://groups.google.com/g/jenkinsci-dev/c/V2CH4pNQitQ/m/ZNmNb0x7BwAJ

          People

            basil Basil Crow
            svanoort Sam Van Oort
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated: