damccorm commented on code in PR #34473:
URL: https://github.com/apache/beam/pull/34473#discussion_r2021130620


##########
.github/workflows/run_rc_validation_python_mobile_gaming.yml:
##########
@@ -114,46 +200,43 @@ jobs:
           echo "Downloading from ${{ env.APACHE_DIST_URL_BASE }}"
           wget ${{ env.APACHE_DIST_URL_BASE }}/python/${{ 
env.BEAM_PYTHON_SDK_TAR_GZ }}
           wget ${{ env.APACHE_DIST_URL_BASE }}/python/${{ 
env.BEAM_PYTHON_SDK_TAR_GZ }}.sha512
-          wget ${{ env.APACHE_DIST_URL_BASE }}/${{ env.BEAM_SOURCE_TAR_GZ }}
-          wget ${{ env.APACHE_DIST_URL_BASE }}/${{ env.BEAM_SOURCE_TAR_GZ 
}}.sha512
+          wget ${{ env.APACHE_DIST_URL_BASE }}/${{ env.BEAM_SOURCE_ZIP }}
+          wget ${{ env.APACHE_DIST_URL_BASE }}/${{ env.BEAM_SOURCE_ZIP 
}}.sha512
         shell: bash
 
       - name: Verify Hashes
         run: |
           echo "Verifying sha512 checksums..."
           sha512sum -c ${{ env.BEAM_PYTHON_SDK_TAR_GZ }}.sha512
-          sha512sum -c ${{ env.BEAM_SOURCE_TAR_GZ }}.sha512
-        shell: bash
-
-      - name: Build Python SDK from Source
-        run: |
-          echo "Building Python SDK sdist..."
-          tar -xzf ${{ env.BEAM_SOURCE_TAR_GZ }}
-          # Navigate into the extracted directory (assuming it matches the 
tarball name without .tar.gz)
-          SOURCE_DIR=$(basename ${{ env.BEAM_SOURCE_TAR_GZ }} .tar.gz)
-          cd ${SOURCE_DIR}/sdks/python
-          python setup.py sdist
-          # Move the built dist back to the working directory
-          mv dist/apache-beam-${{ env.RELEASE_VERSION }}.tar.gz ../../../
+          sha512sum -c ${{ env.BEAM_SOURCE_ZIP }}.sha512
         shell: bash
 
       - name: Setup Python Virtual Environment
         run: |
           echo "Setting up Python virtual environment..."
           python -m venv beam_env
           source beam_env/bin/activate
-          pip install --upgrade pip setuptools wheel
+          pip install --upgrade pip setuptools wheel build
           echo "Virtual environment ready."
         shell: bash
 
+      - name: Build Python SDK from Source
+        run: |
+          echo "Building Python SDK sdist..."
+          source beam_env/bin/activate
+          unzip ${{ env.BEAM_SOURCE_ZIP }}
+          mkdir -p beam-${{ env.RELEASE_VERSION 
}}/website/www/site/content/en/documentation/sdks
+          sudo mkdir -p /website/www/site/content/en/documentation/sdks

Review Comment:
   Why are we making a directory here?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to