Skip to content

Remove Python tarballs from pythonSrc/ext dir - #1256

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

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

Conversation

@tuhaihe

Copy link
Copy Markdown
Member

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
tuhaiheforce-pushed the remove-bundles-pyfiles branch from 85005e2 to 2e30145CompareJuly 24, 2025 06:13
@tuhaihe
tuhaihe marked this pull request as draft July 24, 2025 06:33
@tuhaihe
tuhaiheforce-pushed the remove-bundles-pyfiles branch 2 times, most recently from f8b2155 to f39a616CompareJuly 25, 2025 04:49
@tuhaihe
tuhaihe marked this pull request as ready for review July 25, 2025 05:27
@tuhaihetuhaihe changed the title [Test] Remove Python tarballs from pythonSrc/ext dirRemove Python tarballs from pythonSrc/ext dirJul 25, 2025
Comment threadgpMgmt/bin/Makefile Outdated
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
Comment threadgpMgmt/bin/Makefile
@tuhaihe
tuhaihe merged commit 2b510cb into apache:mainJul 29, 2025
26 checks passed
Sign up for freeto 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

@tuhaihe@yjhjstz@jiaqizho@my-ship-it