Skip to content

Remove Python tarballs from pythonSrc/ext dir#1256

Merged
tuhaihe merged 1 commit intoapache:mainfrom
tuhaihe:remove-bundles-pyfiles
Jul 29, 2025
Merged

Remove Python tarballs from pythonSrc/ext dir#1256
tuhaihe merged 1 commit intoapache:mainfrom
tuhaihe:remove-bundles-pyfiles

Conversation

@tuhaihe
Copy link
Member

@tuhaihe tuhaihe commented Jul 24, 2025

This commit removes Python package tarballs from the pythonSrc/ext dir to comply with Apache Release requirements. The changes include:

  1. Removed all Python package tarballs from pythonSrc/ext:

    • behave-1.2.6.tar.gz
    • mock-1.0.1.tar.gz
    • parse-1.8.2.tar.gz
    • psutil-5.7.0.tar.gz
    • PyGreSQL-5.2.tar.gz
    • PyYAML-5.4.1.tar.gz
    • setuptools-36.6.0.tar.gz
  2. Modified build system to use pip for installing Python dependencies:

    • Updated gpMgmt/bin/Makefile to install dependencies via pip
    • Removed --with-pythonsrc-ext option from configure.ac
    • Updated all platform-specific config flags in gpAux/Makefile
  3. Updated documentation and references:

    • Updated LICENSE, README.apache.md and python-dependencies.txt to reflect these changes.

These changes ensure compliance with Apache Release requirements while maintaining all functionality by installing dependencies via pip instead of building from source tarballs.

See: #961

Fixes #ISSUE_Number

What does this PR do?

Type of Change

  • Bug fix (non-breaking change)
  • New feature (non-breaking change)
  • Breaking change (fix or feature with breaking changes)
  • Documentation update

Breaking Changes

Test Plan

  • Unit tests added/updated
  • Integration tests added/updated
  • Passed make installcheck
  • Passed make -C src/test installcheck-cbdb-parallel

Impact

Performance:

User-facing changes:

Dependencies:

Checklist

Additional Context

CI Skip Instructions


@tuhaihe tuhaihe force-pushed the remove-bundles-pyfiles branch from 85005e2 to 2e30145 Compare July 24, 2025 06:13
@tuhaihe tuhaihe marked this pull request as draft July 24, 2025 06:33
@tuhaihe tuhaihe force-pushed the remove-bundles-pyfiles branch 2 times, most recently from f8b2155 to f39a616 Compare July 25, 2025 04:49
@tuhaihe tuhaihe marked this pull request as ready for review July 25, 2025 05:27
@tuhaihe tuhaihe changed the title [Test] Remove Python tarballs from pythonSrc/ext dir Remove Python tarballs from pythonSrc/ext dir Jul 25, 2025
Replace bundled Python source packages with pip3 download to comply with
Apache Release policy.

- Remove 7 bundled tar.gz packages from gpMgmt/bin/pythonSrc/ext/
- Add download-python-deps target using pip3 download
- Update build targets to depend on download step
- Preserve original compilation logic and --with-pythonsrc-ext functionality
- Update LICENSE, python-dependencies.txt, and Windows build script
- Add .gitignore rules for downloaded files

This maintains the same build behavior while ensuring Apache compliance
by removing third-party packages from source tree.

See: apache#961
@tuhaihe tuhaihe merged commit 2b510cb into apache:main Jul 29, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants