AnandInguva commented on code in PR #22635:
URL: https://github.com/apache/beam/pull/22635#discussion_r941337047
##########
.github/workflows/build_wheels.yml:
##########
@@ -240,6 +240,7 @@ jobs:
working-directory: apache-beam-source
env:
CIBW_BUILD: ${{ matrix.os_python.python }}
+ CIBW_ENVIRONMENT: "SETUPTOOLS_USE_DISTUTILS=stdlib"
Review Comment:
So the image cibuildwheel uses has a very old setuptools. For the effort to
support python 3.10, I would need setuptools>=60 since the stdlib distutils
module is deprecated and distutils provided by setuptools is recommended to
use(it has a local copy of distutils from 60.0)
Cibuildwheel==1.11.0 uses get-pip.py to install packages. This combined with
latest version of setuptools fails because in the setuptools, there is an
assertion statement which checks if _distutils is imported or not. There has
been some issues on this bug as well.
So to by pass this, i tell the interpreter to use standard lib distutils for
building wheels.
As part of this effort, we would need to update the cibuildwheel to latest
version and remove this workaround.
It may take a little time to investigate which version of cibuildwheel we
need as the latest one builds both manylinux, musllinux, which i suspect is
taking longer time to complete the build and resulting in time out.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
