Skip to content

Commit fac1ec8

Browse files
authored
Revert "HOTFIX Don't use setuptools 58.5 which introduces regression (scikit-learn#21549)" (scikit-learn#21560)
This reverts commit 953d101. The fix for setuptools#21549 has been introduced in 58.5.3. See: pypa/setuptools#2855 (comment)
1 parent c5af93c commit fac1ec8

File tree

7 files changed

+7
-21
lines changed

7 files changed

+7
-21
lines changed

build_tools/azure/install.sh

+2-6
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,6 @@ fi
1313

1414
make_conda() {
1515
TO_INSTALL="$@"
16-
# TODO: Remove this line once setuptools#2849 is resolved.
17-
TO_INSTALL="$TO_INSTALL setuptools<58.5"
1816
if [[ "$DISTRIB" == *"mamba"* ]]; then
1917
mamba create -n $VIRTUALENV --yes $TO_INSTALL
2018
else
@@ -90,8 +88,7 @@ elif [[ "$DISTRIB" == "ubuntu" ]]; then
9088
source $VIRTUALENV/bin/activate
9189
setup_ccache
9290
python -m pip install $(get_dep cython $CYTHON_VERSION) \
93-
$(get_dep joblib $JOBLIB_VERSION) \
94-
"setuptools<58.5" # TODO: Remove this line once setuptools#2849 is resolved.
91+
$(get_dep joblib $JOBLIB_VERSION)
9592

9693
elif [[ "$DISTRIB" == "debian-32" ]]; then
9794
apt-get update
@@ -101,8 +98,7 @@ elif [[ "$DISTRIB" == "debian-32" ]]; then
10198
source $VIRTUALENV/bin/activate
10299
setup_ccache
103100
python -m pip install $(get_dep cython $CYTHON_VERSION) \
104-
$(get_dep joblib $JOBLIB_VERSION) \
105-
"setuptools<58.5" # TODO: Remove this line once setuptools#2849 is resolved.
101+
$(get_dep joblib $JOBLIB_VERSION)
106102

107103
elif [[ "$DISTRIB" == "conda-pip-latest" ]]; then
108104
# FIXME: temporary fix to link against system libraries on linux

build_tools/azure/install_win.sh

+2-4
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,7 @@ set -e
44
set -x
55

66
if [[ "$PYTHON_ARCH" == "64" ]]; then
7-
conda create -n $VIRTUALENV -q -y python=$PYTHON_VERSION numpy scipy cython matplotlib wheel pillow joblib \
8-
"setuptools<58.5" # TODO: Remove this line once setuptools#2849 is resolved.
7+
conda create -n $VIRTUALENV -q -y python=$PYTHON_VERSION numpy scipy cython matplotlib wheel pillow joblib
98

109
source activate $VIRTUALENV
1110

@@ -17,8 +16,7 @@ if [[ "$PYTHON_ARCH" == "64" ]]; then
1716
pip install pytest==$PYTEST_VERSION
1817
fi
1918
else
20-
pip install numpy scipy cython pytest wheel pillow joblib threadpoolctl \
21-
"setuptools<58.5" # TODO: Remove this line once setuptools#2849 is resolved.
19+
pip install numpy scipy cython pytest wheel pillow joblib threadpoolctl
2220
fi
2321

2422
if [[ "$PYTEST_XDIST_VERSION" != "none" ]]; then

build_tools/circle/build_doc.sh

-2
Original file line numberDiff line numberDiff line change
@@ -181,8 +181,6 @@ pip install "$(get_dep sphinx-gallery $SPHINX_GALLERY_VERSION)"
181181
pip install "$(get_dep numpydoc $NUMPYDOC_VERSION)"
182182
pip install "$(get_dep sphinx-prompt $SPHINX_PROMPT_VERSION)"
183183
pip install "$(get_dep sphinxext-opengraph $SPHINXEXT_OPENGRAPH_VERSION)"
184-
# TODO: Remove this line once setuptools#2849 is resolved.
185-
pip install "setuptools<58.5"
186184

187185
# Set parallelism to 3 to overlap IO bound tasks with CPU bound tasks on CI
188186
# workers with 2 cores when building the compiled extensions of scikit-learn.

build_tools/circle/build_test_arm.sh

+1-3
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,7 @@ mamba install --verbose -y ccache \
5050
$(get_dep joblib $JOBLIB_VERSION) \
5151
$(get_dep threadpoolctl $THREADPOOLCTL_VERSION) \
5252
$(get_dep pytest $PYTEST_VERSION) \
53-
$(get_dep pytest-xdist $PYTEST_XDIST_VERSION) \
54-
"setuptools<58.5" # TODO: Remove this line once setuptools#2849 is resolved.
55-
53+
$(get_dep pytest-xdist $PYTEST_XDIST_VERSION)
5654
setup_ccache
5755

5856
if [[ "$COVERAGE" == "true" ]]; then

build_tools/circle/build_test_pypy.sh

+1-2
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,7 @@ conda install -y mamba
1313
mamba create -n pypy -y \
1414
pypy numpy scipy cython \
1515
joblib threadpoolctl pillow pytest \
16-
sphinx numpydoc docutils \
17-
"setuptools<58.5" # TODO: remove this line once setuptools#2849 is resolved.
16+
sphinx numpydoc docutils
1817

1918
eval "$(conda shell.bash hook)"
2019
conda activate pypy

build_tools/travis/install_main.sh

-2
Original file line numberDiff line numberDiff line change
@@ -47,8 +47,6 @@ conda create -n testenv --yes python=3.7
4747
source activate testenv
4848
conda install -y scipy numpy pandas cython
4949
pip install joblib threadpoolctl
50-
# TODO: This line once setuptools#2849 is resolved.
51-
pip install "setuptools<58.5"
5250

5351
pip install $(get_dep pytest $PYTEST_VERSION) pytest-xdist
5452

pyproject.toml

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
[build-system]
22
# Minimum requirements for the build system to execute.
33
requires = [
4-
# TODO: Remove the specifier once setuptools#2849 is resolved.
5-
"setuptools<58.5",
4+
"setuptools",
65
"wheel",
76
"Cython>=0.28.5",
87

0 commit comments

Comments
 (0)