Open joshaber opened 8 years ago
We just tried to update to electron 0.37.8 and we're see some issues with runas not being able to be compiled as well. This is the error that we're getting.
I am curious though, why does atom-keymap
need node-pathwatcher
? It's only watching a single file and fs.watch
can handle that pretty well with no additional compiler requirements. It's not the administrator rights since runas
is only needed when writing a file which is never called here. Is there something gained with node-pathwatcher
that I'm missing? Should it be removed in atom-keymap
?
If node-pathwatcher
is removed then this module becomes JavaScript only and doesn't need to have a C++ compiler dependency.
Ah, with the above gist the issues was having a target=^0.37.8
instead of target=0.37.8
in the .npmrc
.
However the point still stands that this requirement might be more that what's actually needed for this package.
@joshaber: did you manage to fix this locally?
Running npm install
twice seems to work for me, although I haven't been able to track down why this is happening. Interestingly this also works smoothly on CI, although the fact that we use Linux on those machines might have an impact on this strange behaviour (which, by the way, could highlight a problem with the installation order of dependencies and it's likely related to the runas
package).
No 😞 IIRC in talking with @nathansobo the consensus seemed to be that it's related to apm
using an old version of npm
.
@joshaber - This is strange and I expect unrelated, but I am getting the following error with you name in the stacktrace:
My setup is:
2017-02-09 20:00:17.874 code[8756:241228] *** Assertion failure in +[SQRLDirectoryManager currentApplicationManager], /Users/joshaber/Documents/Development/GitHub/Squirrel.Mac/Squirrel/SQRLDirectoryManager.m:30
@mdaymond Ha, it's because the builds for the auto updater framework (Squirrel.Mac) are done locally, usually by me. That looks like an issue specific to VSCode. I'd recommend contacting them for support.
I currently can't install this package, or any package with a
pathwatcher
dependency.@atom/core I've been banging by head against this for a couple days. Any ideas? @kuychaco and @BinaryMuse are also hitting this.