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

Fixed missing includes that prevented build from working with Node 0.11+ on *nix and with Node 0.10+ on Windows #17

Merged
merged 3 commits into from
Jul 31, 2015

Conversation

eppsilon
Copy link
Contributor

I was not able to build/install sync-prompt on Windows without this change. Fixes #14.

@eppsilon
Copy link
Contributor Author

Note: this also fixes the Node 0.11 CI build which was broken because of a missing #include.

@eppsilon
Copy link
Contributor Author

I think this PR will now fix #11, #13, #15, and #16.

@eppsilon eppsilon changed the title Fixed missing include that prevented build from working on Windows. Fixed missing includes that prevented build from working with Node 0.11+ and with Node 0.10+ on Windows Jul 13, 2015
@eppsilon eppsilon changed the title Fixed missing includes that prevented build from working with Node 0.11+ and with Node 0.10+ on Windows Fixed missing includes that prevented build from working with Node 0.11+ on *nix and with Node 0.10+ on Windows Jul 13, 2015
@chencoyote
Copy link

this PR is worked. but i can't use npm install -g ember-tools with sync-prompt 😂

shovon added a commit that referenced this pull request Jul 31, 2015
Fixed missing includes that prevented build from working with Node 0.11+ on *nix and with Node 0.10+ on Windows
@shovon shovon merged commit 24b4f7e into shovon:master Jul 31, 2015
@rosspi
Copy link

rosspi commented Aug 1, 2015

@chencoyote
I managed to get it installed and working by cloning a copy of ember-tools, and after installing this fixed sync-prompt, I edited out the dependency in ember-tools' package.json file.

@chencoyote
Copy link

@rosspi
Thx for your help but I got another way to let it work now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants