updated dockerfile manylinux#6236
Open
HarishPermal wants to merge 2 commits intogoogle-ai-edge:masterfrom
Open
updated dockerfile manylinux#6236HarishPermal wants to merge 2 commits intogoogle-ai-edge:masterfrom
HarishPermal wants to merge 2 commits intogoogle-ai-edge:masterfrom
Conversation
Updated the docker and shell file
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR is focused on making the manylinux wheel build reproducible and more resilient in a minimal environment, while making Python imports degrade gracefully when optional native/task components are missing. It also switches OpenCV to static linking for wheels and disables GenAI converter targets that are likely unavailable in the build environment
1. Dockerfile.manylinux_2_28_x86_64
JAVA_HOME: aligns with newer Bazel/JDK expectations and avoids build issues on newer toolchains.std::spansupport and newer libstdc++ in manylinux.protocversion bumped to 28.3: aligns with newer protobuf expectations in the build.*OpenCV build excludes
opencv_apps: trims build time and avoids extra deps..bazelrcgets explicit toolchain/linker env to ensure Bazel uses the newer libstdc++.MEDIAPIPE_PYTHON_BUILD_DIFF_URL: lets the Docker build continue even if the diff isn’t present.ENABLE_ODML_CONVERTER=0: avoids building GenAI converter components not needed for wheel build or missing deps.2. build_manylinux_wheel.sh
./wheelhouse.This is a convenience wrapper for the container build and wheel extraction.3. third_party/BUILD
OPENCV_SHARED_LIBS = False: forces static OpenCV linking, which is preferred for manylinux wheels to reduce runtime shared library dependencies.4. mediapipe/tasks/c/BUILD
5. setup.py
"dev"hardcode.__init__.pyis removed.6. metadata.py
(_pywrap_metadata_version)with a clear error when metadata export is attempted. This avoids hard crashes during import.