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

Can't build, no 'parser' task? #673

Closed
ThatRendle opened this issue Dec 13, 2013 · 6 comments
Closed

Can't build, no 'parser' task? #673

ThatRendle opened this issue Dec 13, 2013 · 6 comments

Comments

@ThatRendle
Copy link

I'm obviously missing something, but having cloned the repo, when I grunt build I get

Running "clean:0" (clean) task
Cleaning "dist"...OK

Running "clean:1" (clean) task

Running "parser" task
Fatal error: spawn ENOENT

Looking at the Gruntfile, the parser task is referenced by the build task but doesn't appear to be defined.

@ThatRendle
Copy link
Author

It builds fine on my Linux box, though.

@osiris43
Copy link

+1 on Windows 7, I get the same error.

@ot-brett-bim
Copy link
Contributor

This has to do with child_process.spawn not working very well on Windows. As a workaround, modify tasks/parser.js like this:

var child = childProcess.spawn('node_modules\\.bin\\jison.cmd', ['-m', 'js', 'src/handlebars.yy', 'src/handlebars.l'], {stdio: 'inherit'});

Some background:
nodejs/node-v0.x-archive#2318

@kpdecker
Copy link
Collaborator

I'm trying to get an environment in place that I can actually test this. Any advice on how to get grunt, etc working under Windows? I'd love to support windows devs here but not having worked with Node/git environments on Windows I'm not clear on how to even get this up and running.

@kpdecker
Copy link
Collaborator

Alternatively if someone can submit a PR that resolves the issue and works across all platforms I'd be glad to get it into master.

@kpdecker
Copy link
Collaborator

Closing this as Windows build support isn't something we can support right now but I'll gladly accept a PR from someone who has such an environment and is able to test it.

osiris43 pushed a commit to osiris43/handlebars.js that referenced this issue Jan 2, 2014
…ctly on windows environments because of forward/backward slash issues
kpdecker added a commit that referenced this issue Jan 3, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants