-
Notifications
You must be signed in to change notification settings - Fork 33
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
Errors on install and uninstall. #81
Comments
You can uninstall the extension by deleting the |
@Equinoxdawg If you could send me the errors you get on uninstall, that would be great.
Thanks! |
I had done what you suggested Nicolo just after posting this issue, thankfully it worked. I have now reinstalled and was able to reproduce. I opened the dev tools, but there was nothing new between opening the tools and it giving me the error and reloading anyway. It now shows after reload: " GET file:///C:/Users/name/AppData/Roaming/Brackets/extensions/user/jasonsanjose.brackets-sass/main.js net::ERR_FILE_NOT_FOUND Edit: I just tried again, and this time as well, there was no errors or anything in the console. |
When I try to uninstall everything but this file gets deleted. So I guess this file causes the issue, Debug Console is empty. |
I think I know what's causing this now. The compiler runs as a child processes (see https://github.com/jasonsanjose/brackets-sass/blob/master/node/1.1.4-3/SASSDomain.js#L173) and I'm not killing the process when an remove/upgrade happens. This will be a tricky one to fix and might require something drastic:
Let me think about this a bit. |
If my theory is right, users should be able to uninstall if the compiler hasn't run in the current session of Brackets. For example:
Could be the same cause as these uninstall bugs: |
@jasonsanjose I've tested it (started Brackets with no files open & empty project-folder) but still got the same error on uninstalling. The workaround does not work I guess ^^ |
@ofhouse can you try a manual uninstall (instead of extension manager) instead? https://github.com/jasonsanjose/brackets-sass/wiki/Manual-Uninstall |
@jasonsanjose Are you working on this bug? Edit: |
Not currently. I'll take a look at you PR. Thanks! |
Quoting the wiki:
Same issue on Debian as well. |
I tried this out just before the last update to it, since then, when it updated it gave me an error saying it couldn't install the update then just now when trying to uninstall it's again giving me an error saying it cannot uninstall it.
I'm using Brackets 1.1 and Windows 7.
The text was updated successfully, but these errors were encountered: