keg: fix any_skip_relocation cellar on Linux

[cho-m]

---

• Have you followed the guidelines in our Contributing document?
• Have you checked to ensure there aren't other open Pull Requests for the same change?
• Have you added an explanation of what your changes do and why you'd like us to include them? Performance claims (e.g. "this is faster") must include Hyperfine benchmarks.
• Have you written new tests (excluding integration tests) for your changes? Here's an example.
• Have you successfully run brew lgtm (style, typechecking and tests) with your changes locally?

---

• AI was used to generate or assist with generating this PR.

---

On Linux, :any_skip_relocation vs :any detection has been historically broken which is why we've ignored the value.

brew/Library/Homebrew/extend/os/linux/bottle_specification.rb

Lines 6 to 8 in edf610a

	def skip_relocation?(tag: Utils::Bottles.tag)
	false
	end

This PR tries to fix the actual cellar symbol used.

It does not fix the above skip_relocation? check yet. Thinking of fixing a few bottles first and using those to test this part later on. Will likely need to access tab information to check parsed_homebrew_version aligned to next release (e.g. >= 5.1.15).

---

Ran a few local checks on formulae via brew install --build-bottle and brew bottle:

• snappy - should be any bottle but currently isn't https://github.com/Homebrew/homebrew-core/blob/main/Formula/s/snappy.rb#L17-L18. Local test outputs:

      sha256 cellar: :any, arm64_linux: "3932df8110e3267ed98d0ae0ed335123dc971a9a5d3603c0617babe0341f8383"
  

      sha256 cellar: :any, arm64_tahoe: "61056dfcc56bd23496be6f12eb041bbd91e653dd507e32712adbcf25fa698595"
  

• fx - checking no regression on any_skip_relocation

      sha256 cellar: :any_skip_relocation, arm64_linux: "8350dd22443c51fc0fcff34c2c6ae1c2751650f11464b2b4336bdf93525733f1"
  

      sha256 cellar: :any_skip_relocation, arm64_tahoe: "0a04f691be96462647f02090a624c1d1b933f4b08d8873fd57f4d22493b2e2ba"
  

• libfaketime - checking no regression on non-relocatable

      sha256 arm64_linux: "6767394bc1f6b5823884957d6aee9d18f386705d53c12aa3654c5599eb43275e"
  

      sha256 arm64_tahoe: "e8b25817117c07f425b89871f6fe9179db9969998f076ece777413fc0ec9526c"
  

Not sure if our tests cover any of this yet.

[Copilot]

Pull request overview

This pull request fixes Linux bottle cellar selection for :any_skip_relocation vs :any by properly tracking when a keg actually required binary relocation, aligning the skip_relocation decision in brew bottle with real ELF patching activity.

Changes:

• Add a typed @require_relocation flag to Keg and introduce a require_relocation! setter.
• Use require_relocation! in macOS Mach-O relocation helpers instead of directly mutating the ivar.
• On Linux, mark relocation as required when ELF rpath/interpreter patching occurs (and avoid reporting relocation when no changes are made).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
Library/Homebrew/keg.rb	Types @require_relocation and adds require_relocation! to centralize relocation state changes.
Library/Homebrew/extend/os/mac/keg.rb	Switches Mach-O change helpers to call require_relocation! instead of setting the ivar directly.
Library/Homebrew/extend/os/linux/keg_relocate.rb	Sets relocation-required state when ELF patching happens, improving :any vs :any_skip_relocation detection.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[MikeMcQuaid]

Thanks!