info: add Binaries section listing executables#22150
info: add Binaries section listing executables#22150MikeMcQuaid merged 4 commits intoHomebrew:mainfrom
Conversation
List the formula's bin/ and sbin/ executables in a new `==> Binaries` section before `==> Caveats`. For installed formulae the list is read from the linked keg; for not-installed formulae `--fetch-manifest` triggers a fallback that reads from the bottle manifest, mirroring how `Bottle Size` and `Installed Size` already behave.
MikeMcQuaid
left a comment
There was a problem hiding this comment.
Sorry @HaraldNordgren, I think this is too wordy to be the default. Having it with --verbose might be a better fit?
|
@MikeMcQuaid Yeah, make sense! The complication is that we need to fetch the manifest, so it's essentially gated by |
Move the `==> Binaries` section behind `--verbose` (`-v`) so the default `brew info` output stays concise.
|
Hmm, I checked the |
|
@HaraldNordgren I think it's bit weird to have I don't think we're going to be able to get to an output that everyone agrees is not too much/just enough. I think the specific binaries are useful to have but you're the first person I've heard request it directly. #22141 may be interesting because I'm moving more in a |
Trigger the bottle manifest fetch under `--verbose` so users don't need to combine `-v` with `--fetch-manifest`. Mirrors how `-v` already pulls analytics inline.
|
@MikeMcQuaid Done, hid this behind the |
|
@MikeMcQuaid One thing I realized now, Git has multiple levels of verbosity ('-v' and '-vv'), potentially the analytics could be hidden behind double verbosity level since it takes over the output otherwise, and leave single verbosity for things that are more customer-facing (binaries and formula size). |
|
@HaraldNordgren I'd rather we kept with one level of verbosity for now. |
|
Thanks! |
List the formula's bin/ and sbin/ executables in a new
==> Binariessection before==> Caveats.For installed formulae the list is read from the linked keg; for not-installed formulae
--fetch-manifesttriggers a fallback that reads from the bottle manifest, mirroring howBottle SizeandInstalled Sizealready behave: