From eed2bea1190abd51f3c621ccc923fecb6585a67c Mon Sep 17 00:00:00 2001 From: David Lakin Date: Tue, 4 Jun 2024 10:09:48 -0400 Subject: [PATCH] [tensorflow-addons] Fix Build & Hardcoded References to Python 3.8 tensorflow-addons was failing to build due to the outdated version of Pyinstaller provided by the base-builder image. This upgrades Pyinstaller to the latest version known to work, and updates a hardcoded path pointing to Python 3.8 now that the base-builder image installs Python 3.10. --- projects/tensorflow-addons/Dockerfile | 6 +----- projects/tensorflow-addons/build.sh | 2 +- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/projects/tensorflow-addons/Dockerfile b/projects/tensorflow-addons/Dockerfile index f1706b9f1ea8..9c864de79e0c 100644 --- a/projects/tensorflow-addons/Dockerfile +++ b/projects/tensorflow-addons/Dockerfile @@ -19,11 +19,7 @@ RUN apt-get update && apt-get install -y --no-install-recommends \ curl \ rsync -RUN python3 -m pip install --upgrade pip chardet - -# Due to Bazel bug, need to symlink python3 to python -# See https://github.com/bazelbuild/bazel/issues/8665 -RUN ln -s /usr/local/bin/python3 /usr/local/bin/python +RUN python3 -m pip install --upgrade chardet 'pyinstaller==6.7.0' # Install Bazelisk to keep bazel in sync with the version required by TensorFlow RUN curl -Lo /usr/bin/bazel \ diff --git a/projects/tensorflow-addons/build.sh b/projects/tensorflow-addons/build.sh index 9d172022543f..c200652dd301 100644 --- a/projects/tensorflow-addons/build.sh +++ b/projects/tensorflow-addons/build.sh @@ -23,5 +23,5 @@ bazel-bin/build_pip_pkg artifacts python3 -m pip install artifacts/tensorflow_addons-*.whl for fuzzer in $(find $SRC -name 'fuzz_*.py'); do - compile_python_fuzzer $fuzzer --add-data '/usr/local/lib/python3.8/site-packages/tensorflow_addons/custom_ops/text/:tensorflow_addons/custom_ops/text/' + compile_python_fuzzer $fuzzer --add-data '/usr/local/lib/python3.10/site-packages/tensorflow_addons/custom_ops/text/:tensorflow_addons/custom_ops/text/' done