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

Added: list of additional files to check during uninstall of older ve… #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

stevebroskey
Copy link

…rsions

I had 1.6.6 installed (IIRC), and there were additional paths not covered in the uninstall section of the Readme.md file. Note that I have tested removal of all of the paths covered in my change except for the /var/db/receipts/com.cycling74.soundflower.* files.

I'm happy to also test removal of those files if someone more familiar with the files can tell me that they're safe to remove.

screen shot 2018-09-06 at 5 25 14 pm

@elvey
Copy link

elvey commented Oct 24, 2018

Useful info. However, I don't think deleting receipts is a good idea, even when uninstalling. See here Also, the App is actually Soundflowerbed, which is a separate thing - see here. Soundflower is just a kernel extension.

So I doubt this should be pulled, as is, anyway.
... and I'm just some guy.

On the other hand, the script that comes with the latest binary:

on run

do shell script "sudo rm -rf /System/Library/Extensions/Soundflower.kext" with administrator privileges
do shell script "sudo rm -rf /Library/Extensions/Soundflower.kext" with administrator privileges
do shell script "sudo rm -rf /Library/Receipts/Soundflower*" with administrator privileges
do shell script "sudo rm -rf /var/db/receipts/com.cycling74.soundflower.*" with administrator privileges
do shell script "sudo rm -rf /Applications/Soundflower" with administrator privileges
"Done!"

end run

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

Successfully merging this pull request may close these issues.

2 participants