-
Notifications
You must be signed in to change notification settings - Fork 163
Package installation fails(Remove PathWatcher) #2
Comments
Can't wait to try this out, I find it pretty ironic that atom doesn't have git command integration :( |
Im getting this error aswell |
Sorry to hear your having install issues. From that log, it seems like the problem is with npm. Did you try removing the file it's complaining about? I'll look into this when I get back to my apartment |
I was able to fix this by doing a manual install in the terminal just run: |
@ColinPacitti getting the same errors by trying to install it in commandline. |
@dmsimard do you have sudo access? Make sure to check the |
@dmsimard Are you still having issues? |
I flushed atom, the atom and apm binaries, the .atom directory and even the application itself. From scratch, here's what it looks like:
If I try with sudo:
|
I saw this discussion about the "node" command not found: http://discuss.atom.io/t/node-command-not-found/5604/3 |
I guess the problem arises while installing the PathWatcher module. I'll create an issue to remove that dependency. Maybe I'll have a separate branch that is PathWatcher-free |
Thanks, let me know when to try again - i'll test and report back the results. |
Just pushed a nowatch branch that doesn't use PathWatcher. Hopefully it works for you. Apparently Node's FileWatcher is still unstable so I don't plan to removing PathWatcher from the apm version unless there is another alternative. |
Thanks @akonwi ! I would like to try it but the package manager doesn't exactly allow to specify a location to download the package from. Do you know how I could test this ? Appreciate it ! |
You should be able to clone the repo into
The third step will create a new branch that tracks the nowatch branch here |
Slight edits to the steps to make it work properly. |
Thanks @alexparker. Did you have installation issues too and is this fix working for you? |
Yup! |
This worked for me, but I had to use
|
Alright, I think apm is the better command to use. I should have put it up initially. |
Haven't had time to test until now. I had to use sudo apm install, otherwise it would fail. |
I just published an update that replaces The pathwatcher dependency with fs-plus. So I think it is safe to close this now |
@akonwi im currently having the same pathwatcher issue in my own project. Das fs-plus has a own stable filewatcher? |
@philipgiuliani the default fs in node has file watching. It isn't very robust but we replaced filewatching in this package with the atom api for subscribing to buffer events like saving and destroying. |
Thanks a lot @akonwi, im now using the fs fileWatcher. It fires some event twice, but im just processing a small amount of data on a change so its not a big problem. Thanks anyways for your help. Nice package 👍 |
Hi,
Installation doesn't seem to work from within Atom's package manager GUI:
The text was updated successfully, but these errors were encountered: