lucee / lucee-dockerfiles

Official Lucee Dockerfiles for Docker Hub build images
https://hub.docker.com/u/lucee/
MIT License
85 stars 50 forks source link

add support for Lucee 6.1 and use Java 21 for Lucee 6 #85

Closed michaeloffner closed 3 months ago

zspitzer commented 3 months ago

Yes, but Java 21 isn't supported yet, date handling is broken by Unicode changes https://luceeserver.atlassian.net/browse/LDEV-4658

michaeloffner commented 3 months ago

@zspitzer that is expected and not a bug, date/time format always change with every java version, that is expected. solution is to define java versions in test cases. This has nothing to do with Lucee, that is simply the difference in the implementation of "SimpleDateFormat" between versions.

michaeloffner commented 3 months ago

@zspitzer also that is nothing that should stop this, 21 is a requirement for Lucee 6, no way around it.

michaeloffner commented 3 months ago

@zspitzer that ticket is resolved now

michaeloffner commented 3 months ago

@zspitzer there was indeed a mistake, i have fixed it, it now looks like this

exclusions:

  # Tomcat 10.1 is not needed for Lucee 5.3
  # - TOMCAT_VERSION: '10.1'
  #   LUCEE_MINOR: '5.3'

  # Zero variant is not needed for Lucee 5.3/5.4
  # - LUCEE_MINOR: '5.3'
  #   LUCEE_VARIANT: '-zero'
  # - LUCEE_MINOR: '5.4'
  #   LUCEE_VARIANT: '-zero'

  # Java 8 is not needed for Lucee 6.0
  - TOMCAT_JAVA_VERSION: 'jre8-temurin-jammy'
    LUCEE_MINOR: '6.0'
  - TOMCAT_JAVA_VERSION: 'jdk8-temurin-jammy'
    LUCEE_MINOR: '6.0'
  # Java 8 is not needed for Lucee 6.1
  - TOMCAT_JAVA_VERSION: 'jre8-temurin-jammy'
    LUCEE_MINOR: '6.1'
  - TOMCAT_JAVA_VERSION: 'jdk8-temurin-jammy'
    LUCEE_MINOR: '6.1'
  # Java 21 is not needed for Lucee 5.3
  - TOMCAT_JAVA_VERSION: 'jre21-temurin-jammy'
    LUCEE_MINOR: '5.3'
  - TOMCAT_JAVA_VERSION: 'jdk21-temurin-jammy'
    LUCEE_MINOR: '5.3'
  # Java 21 is not needed for Lucee 5.4
  - TOMCAT_JAVA_VERSION: 'jre21-temurin-jammy'
    LUCEE_MINOR: '5.4'
  - TOMCAT_JAVA_VERSION: 'jdk21-temurin-jammy'
    LUCEE_MINOR: '5.4'

so excluding Java 21 for Lucee 5 and excluding Java 8 for Lucee 6