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

Add java.activation module to Java 10 Docker packages

      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

          [JENKINS-52004] Add java.activation module to Java 10 Docker packages

          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.

          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 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 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

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

              Created:
              Updated:
              Resolved: