Transitive dependencies not uninstalled in correct order about homebrew-rmtree

[ggPeti]

In the meantime I put together a quick hack to recursively remove dependencies. Check it out: https://github.com/ggPeti/homebrew-rmrec

[beeftornado]

Thanks for reporting this, and thanks for the detailed example. I'll look into this

[jsfaint]

Hi @Argon- My PR was already merged by @beeftornado
Thanks @beeftornado 😄

[Argon-]

Unfortunately this repo seems to be dead/inactive. I guess it's best to settle for one of the forks to continue, e.g. https://github.com/jsfaint/homebrew-rmtree as it solves the (as of now probably most important) problem explained in #11

[irundaia]

This issue is also a problem in the linked repo. I'd create an issue there as well, if the repo allowed the creation of issues

[ilovezfs]

If you export HOMEBREW_DEVELOPER=1 the errors will be warnings so it at least won't go 💥

[metakirby5]

I got around this by adding the --ignore-dependencies flag to this line. I'm not sure if this is a "clean" solution, but it seems to have worked for me.

[irundaia]

@metakirby5 Doesn't this mean that the dependencies aren't removed?

[ilovezfs]

No.

If HOMEBREW_DEVELOPER is not set, and there are reverse dependencies using a formula, brew will error out if you try to remove that formula. --ignore-dependencies will suppress the error.

If HOMEBREW_DEVELOPER is set, and there are reverse dependencies using a formula, brew will issue a warning if you try to remove that formula. --ignore-dependencies will suppress the warning.

The fact that rmtree needs to use --ignore-dependencies means that it is in fact removing things that have reverse dependencies.

[irundaia]

I'm not sure if this will be the behaviour I'm looking for though, as this might uninstall packages that might be needed for other packages.

[ilovezfs]

@irundaia theoretically it should be fine because the cause is that the packages are being removed in the "wrong" order instead of all at once or in the "right" order. The "correct" fix here is to remove them in a single call to brew uninstall, "brew uninstall A B C D" or to do the uninstalls in an order that always removes a reverse dependency before its dependencies.

[irundaia]

@ggPeti, I've tested your work, it seems to work fine :) It does seem to be a tad slow tbh. But I can deal with that. :)

[beeftornado]

Related to #21 #24