Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-54306

cbdep java download munges URL if PATCH has >1 segment

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Major
    • None
    • 6.6.6, 7.0.5
    • build
    • Untriaged
    • 1
    • Unknown
    • Build Team 2022 Sprint 20, Build Team 2022 Sprint 21, Build Team 2022 Sprint 22, Build Team 2022 Sprint 23, Build Team 2023 Sprint 1, Build Team 2023 Sprint 2, Build Team 2023 Sprint 3, Build Team 2023 Sprint 4, Build Team 2023 Sprint 5

    Description

      The current latest version of JDK 11 is 11.0.16.1+1. The download script doesn't handle this properly, and replaces the wrong segment (should be the +, but is instead the last .) with an underscore.

      e.g.

      17:42:51   requests.exceptions.HTTPError: 404 Client Error: Not Found for url:
      17:42:51   https://github.com/adoptium/temurin11-binaries/releases/download/jdk-11.0.16.1+1/OpenJDK11U-jdk_x64_linux_hotspot_11.0.16_1+1.tar.gz
      17:42:51 
      17:42:51 
      17:42:51 Call Stack (most recent call first):
      17:42:51   analytics/cmake/Modules/FindCouchbaseJava.cmake:102 (CBDEP_INSTALL)
      17:42:51   analytics/cmake/Modules/FindCouchbaseJava.cmake:114 (GET_JAVA_VERSION)
      17:42:51   analytics/CMakeLists.txt:8 (INCLUDE)
      

      The end of the URL should instead be 11.0.16.1_1 not 11.0.16_1+1.

      e.g. https://review.couchbase.org/c/cbas-core/+/181970

      Attachments

        Activity

          People

            michael.blow Michael Blow
            michael.blow Michael Blow
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 8h
                8h

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty