From fd1d6bce8ff8d49ebb9accaf3fde122bb3973295 Mon Sep 17 00:00:00 2001 From: Jeff Widman Date: Sun, 7 Aug 2022 22:45:57 -0600 Subject: [PATCH] Workaround thread safety issue by limiting worker count The original intent of this code was to workaround a thread safety issue. However, according to the `poetry` maintainer, the more "future proof" workaround is [to limit the worker count rather than disable the new installer](https://github.com/python-poetry/poetry/issues/3010#issuecomment-1076518050). The underlying root issue is supposedly resolved in [the upcoming `1.2` release of `poetry`](https://github.com/python-poetry/poetry/issues/3010#issuecomment-1076518050). But until then let's use the cleaner workaround. --- python/helpers/build | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/python/helpers/build b/python/helpers/build index de6b4c40ea..37bded6659 100755 --- a/python/helpers/build +++ b/python/helpers/build @@ -20,7 +20,8 @@ cp -r \ cd "$install_dir" PYENV_VERSION=3.10.5 pyenv exec pip --disable-pip-version-check install --use-pep517 -r "requirements.txt" -# Workaround of https://github.com/python-poetry/poetry/issues/3010 +# TODO remove this workaround of https://github.com/python-poetry/poetry/issues/3010 +# once Poetry 1.2 ships. # By default poetry config file is stored under ~/.config/pypoetry # and is not bound to any specific Python version -PYENV_VERSION=3.10.5 pyenv exec poetry config experimental.new-installer false +PYENV_VERSION=3.10.5 pyenv exec poetry config installer.max-workers 1