Skip to content

Fix tcmalloc patch context for 12f2552 - #74

Closed
phlax with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-patch-application-error
Closed

phlax with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-patch-application-error

Conversation

Copilot AI commented Sep 4, 2026 •

Copy link
Copy Markdown
Contributor

tcmalloc@0.0.0-250926-12f2552.envoy reused the older 20241022 patch, but upstream tcmalloc/BUILD changed, causing Bazel patch application to fail with CONTENT_DOES_NOT_MATCH_TARGET.

  • Patch context
    • Regenerated patches/tcmalloc.patch against the 12f2552 upstream tcmalloc/BUILD.
    • Kept the intended change limited to package visibility:
-package(default_visibility = ["//visibility:private"])
+package(default_visibility = ["//visibility:public"])
  • Registry metadata
    • Updated the tcmalloc.patch checksum in source.json.
    • Left the upstream tarball integrity unchanged.

Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix patch application issue for tcmalloc Fix tcmalloc patch context for 12f2552 Sep 4, 2026
Copilot AI requested a review from phlax September 4, 2026 11:50
@phlax phlax closed this Sep 4, 2026
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.

2 participants