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

Use Java 11 built-in functionality instead of Animal Sniffer

    XMLWordPrintable

Details

    • 4.40

    Description

      Currently there is no Java 9+ signatures for Animal Sniffer, so we cannot really compile code for Java 11 without disable static checks. CC olamy who may know workarounds

       

      Proposal:

      • Disable Animal sniffer when java.level is 9 or above
      • Enable http://openjdk.java.net/jeps/247 flags for newer versions (maybe Maven Compiler does it automatically)

       

       

      Attachments

        Issue Links

          is blocking

          Improvement - An improvement or enhancement to an existing feature or task. JENKINS-54828 Support building plugins with Java 11 and 17

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

          Web Link jenkinsci/plugin-pom#523

          Activity

            Ascending order - Click to sort in descending order
            olamy Olivier Lamy added a comment -

            IMHO we do not need anymore animal sniffer but simply use the release compiler flag...

            Read the motivation section here http://openjdk.java.net/jeps/247  I guess this was inspired by our signatures.

            This simply insure the code is RELEASE compatible.  

            olamy Olivier Lamy added a comment - IMHO we do not need anymore animal sniffer but simply use the release compiler flag... Read the motivation section here http://openjdk.java.net/jeps/247   I guess this was inspired by our signatures. This simply insure the code is RELEASE compatible.  
            jglick Jesse Glick added a comment -

            basil’s https://github.com/jenkinsci/plugin-pom/pull/478 does this.

            jglick Jesse Glick added a comment - basil ’s https://github.com/jenkinsci/plugin-pom/pull/478 does this.
            basil Basil Crow added a comment -

            Released in 4.40.

            basil Basil Crow added a comment - Released in 4.40 .

            People

              basil Basil Crow
              oleg_nenashev Oleg Nenashev
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: