Skip to content

Conversation

@Rylan12
Copy link
Member

@Rylan12 Rylan12 commented Oct 17, 2025

Final part extracted from #20830

This PR actually makes the fundamental change to Formulary::resolve to attempt to load formulae from their kegs (and therefore, their install receipts).

I've marked this as a draft for now, because I want to gate this with an environment variable. For now, I'm thinking I'll just do HOMEBREW_USE_INTERNAL_API, but let me know if a separate one is preferred.

Copy link
Member

@MikeMcQuaid MikeMcQuaid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For now, I'm thinking I'll just do HOMEBREW_USE_INTERNAL_API, but let me know if a separate one is preferred.

This makes the most sense to me 👍🏻

Base automatically changed from add-installed-runtime-dep-methods to main October 17, 2025 18:18
@Rylan12 Rylan12 force-pushed the load-installed-formulae-from-kegs branch from cbd4d8b to 8861d19 Compare October 17, 2025 19:16
@Rylan12 Rylan12 marked this pull request as ready for review October 17, 2025 19:16
Copilot AI review requested due to automatic review settings October 17, 2025 19:16
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Implements loading installed formulae directly from their kegs (and install receipts) when HOMEBREW_USE_INTERNAL_API (use_internal_api?) is enabled.

  • Adds conditional logic to prefer FromKegLoader when resolving formula paths.
  • Adjusts FromKegLoader.try_new to use canonical formula names and include tap metadata.

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Rylan12 Rylan12 added this pull request to the merge queue Oct 20, 2025
Merged via the queue into main with commit b98c58d Oct 20, 2025
37 checks passed
@Rylan12 Rylan12 deleted the load-installed-formulae-from-kegs branch October 20, 2025 19:00
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.

3 participants