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

Add java.activation module to Java 10 Docker packages

    XMLWordPrintable

Details

    Description

      Currently we have the module enabled automatically only for Java 11.
      duemir recently reported that it was needed to Run on his instance, so it would worth to add the module to Java 10 packaging as well.

      The following file should be patched: https://github.com/jenkinsci/docker/blob/java10/jenkins.sh

      Attachments

        Issue Links

          relates to

          New Feature - A new feature of the product, which has yet to be developed. JENKINS-51965 Add support of JAXB unpackaging for Java 9+ in WAR Exploder

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Activity

            Ascending order - Click to sort in descending order
            duemir Denys Digtiar added a comment -

            I ran:

            $JAVA10_HOME/bin/jmod describe $JAVA10_HOME/jmods/java.xml.bind.jmod > java.xml.bind.desc

            And here is what I have got: java.xml.bind.desc

            Specifically of interest is:

            requires java.activation transitive

            Therefore, dependency is explicit and transitive on Java 10 as well. It seems like it might be added automatically by --add-modules java.xml.bind, otherwise, the modular JVM shouldn't start with missing dependency error, as far as I understand.

            Hence, it looks like this task might be a matter of prudence, not a strict requirement.

            duemir Denys Digtiar added a comment - I ran: $JAVA10_HOME/bin/jmod describe $JAVA10_HOME/jmods/java.xml.bind.jmod > java.xml.bind.desc And here is what I have got:  java.xml.bind.desc Specifically of interest is: requires java.activation transitive Therefore, dependency is explicit and transitive on Java 10 as well. It seems like it might be added automatically by --add-modules java.xml.bind , otherwise, the modular JVM shouldn't start with missing dependency error, as far as I understand. Hence, it looks like this task might be a matter of prudence, not a strict requirement.
            oleg_nenashev Oleg Nenashev added a comment -

            Thanks for the analysis duemir!
            Taking JENKINS-51965 we will likely go forward with bundling the lib and removing need in special flags.
            So I will probably just close this ticket for now

            oleg_nenashev Oleg Nenashev added a comment - Thanks for the analysis duemir ! Taking JENKINS-51965 we will likely go forward with bundling the lib and removing need in special flags. So I will probably just close this ticket for now

            People

              Unassigned Unassigned
              oleg_nenashev Oleg Nenashev
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: