Skip to content

docs: add brew trust#105

Open
robertoschiavone wants to merge 1 commit into
libkrun:mainfrom
robertoschiavone:main
Open

docs: add brew trust#105
robertoschiavone wants to merge 1 commit into
libkrun:mainfrom
robertoschiavone:main

Conversation

@robertoschiavone

Copy link
Copy Markdown

Since June 11 2026, Homebrew 6.0.0 introduces a new Tap Trust mechanism. When following the README, I got the following error

Error: Refusing to load formula slp/krun/krunkit from untrusted tap slp/krun.
Run `brew trust --formula slp/krun/krunkit` or `brew trust slp/krun` to trust it.

so I am proposing to add the brew trust command to the README.

I also took the liberty to add shell highlighting to all code blocks:

  • before
Screenshot 2026-06-15 at 15-08-16 libkrun_krunkit Launch configurable virtual machines with libkrun
  • after
Screenshot 2026-06-15 at 15-08-09 robertoschiavone_krunkit Launch configurable virtual machines with libkrun

@tylerfanelli tylerfanelli left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Sounds reasonable to me, but how would this work with <6.0.0? I imagine with no knowledge of a trust subcommand and subsequent error, users would think this is failing?

Comment thread README.md Outdated
Comment on lines +27 to +28
brew tap slp/krun
brew trust slp/krun

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

while you're here, these should also be libkrun/krun instead of slp/krun

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Yeah, this is so confusing... lack of attention to detail.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

the tap name just changed since we moved the project into a different org, so it just hadn't trickled into the docs yet

Signed-off-by: Roberto Schiavone <5054879+robertoschiavone@users.noreply.github.com>

@jakecorrenti jakecorrenti left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can you also fix the CI as well since it's related to this?

run: brew tap slp/krun && brew install virglrenderer clang-format libkrun

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.

4 participants