Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fail bundle install if a formula is installed but not linked #1511

Closed
andreineculau opened this issue Nov 30, 2024 · 5 comments
Closed

Fail bundle install if a formula is installed but not linked #1511

andreineculau opened this issue Nov 30, 2024 · 5 comments

Comments

@andreineculau
Copy link

We have a situation where people install software outside of homebrew, and running brew bundle install will install the formulae but then fail creating the symlinks. Trying again though will succeed, although the symlinks are not enabled, thus hiding the problem.

At least my expectation is that brew bundle install is deterministic.
No problems: we installed AND linked n formulae.
Problems: we installed n but failed to link m formulae.

Right now it's not. Either there are no problems, or if there are, trying again and again and again will lead to a success message, with problems being ignored.

@jacobbednarz
Copy link
Contributor

if i understand this correctly, this is less of a bundle problem and more of a brew + overwriting problem. what does the bundle file look like? what arguments are you passing in? what is the PATH setup on these systems?

@MikeMcQuaid
Copy link
Member

Can you provide a minimal Brewfile and, ideally, brew commands that reproduce this behaviour?

Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@yelsa
Copy link
Contributor

yelsa commented Feb 6, 2025

@MikeMcQuaid i think this got fixed by 37f0345

If I understand the OP was after the failure messaging to be fixed, so that failure to link is a homebrew-bundle failure.

It is similar to the two examples in #1442 where linking fails but a success message is still printed and the exit code is 0. The underlying problem there was that success was based only on if the install step was successfull, and it ignored any link (or service state change) failures. That is what your commit changed.

@MikeMcQuaid
Copy link
Member

i think this got fixed by 37f0345

Yes, quite likely, thanks for chiming in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants