Skip to content

Commit

Permalink
GH-42006: [CI][Python] Use pip install -e instead of setup.py build_e…
Browse files Browse the repository at this point in the history
…xt --inplace for installing pyarrow on verification script (#42007)

### Rationale for this change

Due to #37929 we require a higher version of setuptools and setuptools_scm to be installed otherwise the job fails with setuptools_scm failing with ` TypeError: Configuration.__init__() got an unexpected keyword argument 'version_file'`

### What changes are included in this PR?

Remove the dependencies for the environment and let installation handle those using pip install -e instead of setup.py build_ext --inplace for installing pyarrow on verification script

### Are these changes tested?

Via Archery

### Are there any user-facing changes?

No
* GitHub Issue: #42006

Lead-authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Co-authored-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
Signed-off-by: Raúl Cumplido <raulcumplido@gmail.com>
  • Loading branch information
raulcd and jorisvandenbossche authored Jun 7, 2024
1 parent 1dde399 commit a045770
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions dev/release/verify-release-candidate.sh
Original file line number Diff line number Diff line change
Expand Up @@ -756,7 +756,7 @@ test_python() {
show_header "Build and test Python libraries"

# Build and test Python
maybe_setup_virtualenv "cython>=0.29.31" numpy "setuptools_scm<8.0.0" setuptools
maybe_setup_virtualenv
maybe_setup_conda --file ci/conda_env_python.txt

if [ "${USE_CONDA}" -gt 0 ]; then
Expand Down Expand Up @@ -788,7 +788,7 @@ test_python() {
pushd python

# Build pyarrow
python setup.py build_ext --inplace
python -m pip install -e .

# Check mandatory and optional imports
python -c "
Expand Down

0 comments on commit a045770

Please sign in to comment.