-
Notifications
You must be signed in to change notification settings - Fork 10
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
[Windows] Error: watch ENOENT #41
Comments
I have this error by just adding your package to mantra-sample-blog-app :
I'm using
|
It seems like this is undefined : // meteor-hmr/accelerator/src/accelerator.js:15
const meteorToolPath = process.argv[4]; |
@Vincent-P I'll address this in #56. |
The issue is with toDosPath function in hot-build/hot-server.js. |
There : Hot.prototype.processFilesForTarget = function(inputFiles) {
...
file = convertToOSPath(path.join(
inputFile._resourceSlot.packageSourceBatch.sourceRoot,
inputFile.getPathInPackage()
));
...
} In Windows I have this : // /E/Code/WEB/mantra-sample-blog-app
inputFile._resourceSlot.packageSourceBatch.sourceRoot
// lib/collections.js
inputFile.getPathInPackage()
// \E\Code\WEB\mantra-sample-blog-app\lib\collections.js
path.join(inputFile._resourceSlot.packageSourceBatch.sourceRoot, inputFile.getPathInPackage())
or
///E/Code/WEB/mantra-sample-blog-app/lib/collections.js
inputFile._resourceSlot.packageSourceBatch.sourceRoot + '/' + inputFile.getPathInPackage() So if (p[0] === '/' && ! partialPath) {
if (! /^\/[A-Za-z](\/|$)/.test(p))
throw new Error("Surprising path: " + p);
// transform a previously windows path back
// "/C/something" to "c:/something"
p = p[1] + ":" + p.slice(2);
} So replacing file = convertToOSPath(path.join(
inputFile._resourceSlot.packageSourceBatch.sourceRoot,
inputFile.getPathInPackage()
));
// to
file = convertToOSPath(inputFile._resourceSlot.packageSourceBatch.sourceRoot + '/' + inputFile.getPathInPackage()); |
oh wow, amazingly helpful! thanks!! |
So now it seems to work on Windows, the app can start but I still have no idea on how to use this with Mantra. I have replaced |
Mmm let me actually make a quick repo of mantra-sample-blog-app, I think a lot of people would benefit from it. |
Sorry I said quickly but I ran out of time and will only get to this tomorrow. I guess you'd put the AppContainer where you have the |
Okay I'll try thank you. |
@Vincent-P, check out #60. Unfortunately it's not so neat... the Mantra design isn't so compatible with react-hot-loader. Maybe we can find a compromise in the future. |
And maybe others :) Would love some help from Windows users. Otherwise will setup a Windows VM and look into this when I have a chance.
The text was updated successfully, but these errors were encountered: