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

Add Little Snitch uninstaller #10257

Closed
wants to merge 1 commit into from
Closed

Add Little Snitch uninstaller #10257

wants to merge 1 commit into from

Conversation

epsimatic
Copy link
Contributor

No description provided.

@fanquake fanquake added the cask label Mar 27, 2015
@vitorgalvao
Copy link
Member

That won’t work. It isn’t a script. Calling that just opens the GUI anyway.

@epsimatic
Copy link
Contributor Author

Objection: That actually works:

  • You type brew cask rm little-snitch
  • a window opens:
  • You click Continue
  • Little Snitch is gone.

And it works the same way for controllermate.

A GUI uninstaller is the only way to get rid of the app.

You should really take into account that there's currently no uninstaller for Little Snitch. That literally means that typing brew cask rm little-snitch does not remove the app. Read: the package is badly broken. And I offer a fix that gets the job done.

@vitorgalvao
Copy link
Member

Yes, we know, Little Snitch’s uninstall has been discussed on this project since before uninstall was even implemented. We stay clear from GUI operations. installer :manual is the closest we have, and even so we don’t automatically call the GUI (important distinction from what you’re doing here).

We may in the future implement something for GUI uninstallers but this is definitely not the way. Again, it is not a script, uninstall :script is absolutely the wrong way of approaching this.

In addition, no, the GUI uninstaller is not the only way to get rid of it. It is the only official way, but a lot of the apps in homebrew-cask don’t provide an installer and we can uninstall them anyway via other techniques.

@epsimatic
Copy link
Contributor Author

Then someone has to reverse-engineer the uninstaller, I guess. Until that, little-snitch.rb is a one-way ticket. You can get it installed, but can't get it removed.

Well, do what you think is best for the project. I still ask for two things:

* Please add a caveats for Little Snitch to tell that one has to get rid of an app manually.

@vitorgalvao
Copy link
Member

Nice catch on controllermate. I saw the name correction and hadn’t noticed it was using the app. Thank you.

Then someone has to reverse-engineer the uninstaller, I guess.

No, someone needs to contact the developer first and simply ask. As far as I’m aware, no one has done that yet for this cask, and asking has yielded very good results in the past. It’s also way more effective than building hacks on top of hacks.

Until that, little-snitch.rb is a one-way ticket.

No, it is not, as you still have the manual uninstaller.

Many others are in that situation, it’s not like we’re not aware of it. Finding a solution for those is needed, yes, but once more, yours isn’t it. It was decided long ago that one of the core features of this project would be to not call GUIs automatically, to allow for scripting.

@epsimatic epsimatic deleted the little-snitch-uninst branch April 4, 2015 22:53
@miccal miccal removed the cask label Dec 23, 2016
@Homebrew Homebrew locked and limited conversation to collaborators May 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants