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

Extras Executable WAR should warn everybody that Java 9 is not supported

      Currently Jenkins does not support Java 9, but Extras Executable WAR does not warn about that: https://github.com/jenkinsci/extras-executable-war/blob/49322726d410fac8aa0b5cb7b00934e62f3b89e3/src/main/java/Main.java#L118-L138

      It would be great to add Java 9 check there and to at least print the warning. Maybe it's better to just fail fast.

          [JENKINS-49737] Extras Executable WAR should warn everybody that Java 9 is not supported

          Oleg Nenashev created issue -
          Oleg Nenashev made changes -
          Epic Link New: JENKINS-40689 [ 177392 ]
          Oleg Nenashev made changes -
          Labels New: newbie-friendly

          Oleg Nenashev added a comment -

          Actually it is a newbie-friendly ticket.
          Anybody is welcome to take and implement it.

          Oleg Nenashev added a comment - Actually it is a newbie-friendly ticket. Anybody is welcome to take and implement it.
          Oleg Nenashev made changes -
          Assignee Original: Oleg Nenashev [ oleg_nenashev ]
          Oleg Nenashev made changes -
          Component/s New: core [ 15593 ]

          Amit Magar added a comment - - edited

          Hi Oleg,

          I want to ask you if you want just print an error message stating "Jenkins does not support Java 9", Or want to throw error and print error message(capture this event in a log file). ?

          Thanks,
          Amit

          Amit Magar added a comment - - edited Hi Oleg, I want to ask you if you want just print an error message stating "Jenkins does not support Java 9", Or want to throw error and print error message(capture this event in a log file). ? Thanks, Amit

          Oleg Nenashev added a comment -

          For starters I would suggest to use the same logic as the service uses for Java 7 and older. If you are interested to improve logging, you are more than welcome to do that

          Oleg Nenashev added a comment - For starters I would suggest to use the same logic as the service uses for Java 7 and older. If you are interested to improve logging, you are more than welcome to do that
          Amit Magar made changes -
          Assignee New: Amit Magar [ amit_magar ]

          Amit Magar added a comment -

          Hi Oleg,

          I have simply followed the same logic which was followed for Java 7 or older. Should I submit the pull request for this ?

          Thanks,
          Amit

          Amit Magar added a comment - Hi Oleg, I have simply followed the same logic which was followed for Java 7 or older. Should I submit the pull request for this ? Thanks, Amit

            amit_magar Amit Magar
            oleg_nenashev Oleg Nenashev
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: