-
Notifications
You must be signed in to change notification settings - Fork 144
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
brew install fails when coreutils is already installed #48
Comments
I know about this conflict, but I also don't know the best solution. Maybe |
I have created pull request to rename it - Homebrew/homebrew-core#74871 |
You don't actually need to unlink I'm personally not a fan of renaming the binary: I think this is surprising for all your current users, whereas the current situation is surprising for only those who have both However, this is your software, so I'm happy to go with whatever you think is best. |
Ahh - that's a much better solution IMO. Maybe that could be printed in the error message instead? |
I think the solution with If am not missing something, then you have to unlink the conflicting package first (coreutils), install the second one (gdu) and unlink it. Then link the first one (coreutils) and finally |
No, there is no need to unlink
|
Ah, that's better. Ok, I will close the pull request and update installation instructions for gdu. Thanks both of you! |
just a FYI, I was using the overwrite but it failed to upgrade recently, it would be nice to reconsider renaming it or at least publish an "alternative" if not only for the sake of keeping backwards compatibility for existing users. |
@brodock The conflict with |
On my macOS system that has the
coreutils
formula already installed:In
coreutils
, they linkdu
togdu
to indicate that it's "GNU du". Personally I'm OK with unlinkingcoreutils
, but this could be surprising to many users.One suggestion could be to provide a non-conflicting binary as an option (like
go-gdu
or something?)... Not sure what the best fix is.And, maybe there's no fix possible - but I just wanted to provide some new-user feedback 😉
The text was updated successfully, but these errors were encountered: