Skip to content

Conversation

@copybara-service
Copy link
Contributor

@copybara-service copybara-service bot commented Dec 20, 2025

Fix TF-text for TF 2.20.

This is a combination of a collection of changes to fix the TF-text build. All are required
for a successful build.

  1. Upgrade ICU to v77.1
    • Requires changes to ICU build file, updating ICU data, updating test goldens
    • Added custom_http_archive rule to allow excluding files.
  2. Replace tensorflow.bzl:pybind_extension with pywrap.bzl:pybind_extension, pywrap_libraries, pywrap_binaries
    • Forces creation of pybind bundled .so file to include in the wheel.
  3. Patch TF build file to exclude libtensorflow_framework source files.
    • Avoids ODR issues causing double-registrations.
  4. Update requirements files
    • TF 3.9, 3.10 are somewhat fundamentally broken because of tensorflow-metadata,
      which restricts the protobuf version below what TF requires. Though this turns out
      not to be a hard requirement, by skipping the requirements.update in configure and manually
      setting package versions, we can create a successful build.
  5. Replace the build_pip_package.sh with a python version
    • Allows using the hermetic python toolchain to build the wheel. Avoids the need for docker.
  6. Fix random missing deps.

@copybara-service copybara-service bot force-pushed the test_847172303 branch 9 times, most recently from 7f3e326 to fd3eae5 Compare December 22, 2025 20:19
@copybara-service copybara-service bot changed the title Fix TF-text. Fix TF-text for TF 2.20. Dec 22, 2025
@copybara-service copybara-service bot force-pushed the test_847172303 branch 5 times, most recently from 0574af1 to 81c8e8b Compare January 9, 2026 19:46
@cantonios cantonios closed this Jan 9, 2026
@cantonios cantonios deleted the test_847172303 branch January 9, 2026 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant