Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-27894

create-release is broken by recent gitbox changes

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 4.0.0-alpha-1
    • scripts
    • None
    • Reviewed

    Description

      The error looks like:

      ...
      GIT_BRANCH [branch-2.5]: 
      -:1: parser error : Space required after the Public Identifier
      <!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
      

      but is misleading. What is going on is create-release tries to retrieve the top level POM from the specified branch using a gitbox URL but gitbox is returning an HTML redirect, which is not what is expected.

      ++ read -r -p 'GIT_BRANCH [branch-2.5]: ' REPLY
      GIT_BRANCH [branch-2.5]: 
      ++ local RETVAL=branch-2.5
      ++ '[' -z branch-2.5 ']'
      ++ echo branch-2.5
      + GIT_BRANCH=branch-2.5
      + export GIT_BRANCH
      + local version
      ++ curl -s 'https://gitbox.apache.org/repos/asf?p=hbase.git;a=blob_plain;f=pom.xml;hb=refs/heads/branch-2.5'
      ++ parse_version
      ++ xmllint --xpath '//*[local-name()='\''project'\'']/*[local-name()='\''version'\'']/text()' -
      -:1: parser error : Space required after the Public Identifier
      <!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
      
      $ curl -s 'https://gitbox.apache.org/repos/asf?p=hbase.git;a=blob_plain;f=pom.xml;hb=refs/heads/branch-2.5'
      <!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
      <html><head>
      <title>302 Found</title>
      </head><body>
      <h1>Found</h1>
      <p>The document has moved <a href="https://raw.githubusercontent.com/apache/hbase/refs/heads/branch-2.5/pom.xml">here</a>.</p>
      </body></html>
      

      The solution is to retrieve content using github URLs (via raw.githubusercontent.com) instead of gitbox URLs.

      Attachments

        Issue Links

          Activity

            People

              apurtell Andrew Kyle Purtell
              apurtell Andrew Kyle Purtell
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: