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

Cannot install because space is in directory. #11

Closed
Glavin001 opened this issue Jan 8, 2014 · 4 comments
Closed

Cannot install because space is in directory. #11

Glavin001 opened this issue Jan 8, 2014 · 4 comments
Assignees
Labels
Milestone

Comments

@Glavin001
Copy link
Member

Important Notice

This is actually an issue in nodegit which traces even further back to node-gyp, which apparently passes along the issue to a gyp bug.


Temporarily Solution

Use @Glavin001's nodegit which uses @Glavin001's fixed node-gyp.

Until @TooTallNate merges my fixed Pull Request we will have to stay away from using the actual libraries.

@ghost ghost assigned Glavin001 Jan 8, 2014
@Glavin001
Copy link
Member Author

Temporarily solution ended up not working...

Solution 2

Rename the path so that there is no spaces... 👎

@DylanYoung
Copy link

Haha. Oh well. There shouldn't really be spaces in paths anyways. It still
makes my teeth grind me when people name files or folders with spaces.

On Tue, Jan 7, 2014 at 10:39 PM, Glavin [email protected] wrote:

Temporarily solution ended up not working... Solution 2

Rename the path so that there is no spaces... [image: 👎]


Reply to this email directly or view it on GitHubhttps://github.com//issues/11#issuecomment-31801219
.

@Glavin001
Copy link
Member Author

I don't usually do, but since I know other users do, I made my Project Development directory while a space so I could check for this corner case.. and finally when I catch it, it's not solvable. Sigh... spent two nights trying to solve nodejs/node-gyp#65 Would've been worth it if only I'd succeeded :P.


  • I think I will write up a install.js script that will check for these things and notify the user.

@Glavin001
Copy link
Member Author

Automated Installer script (#12) will warn of this issue.

To use installer:

node install.js

or

./install.js

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

No branches or pull requests

2 participants