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

Error: no suport windows 8 #53

Closed
gladson opened this issue Feb 27, 2014 · 9 comments
Closed

Error: no suport windows 8 #53

gladson opened this issue Feb 27, 2014 · 9 comments

Comments

@gladson
Copy link

gladson commented Feb 27, 2014

File "C:\Users\Gladson\Desktop\Teste\env_nodeenv\lib\site-packages\nodeenv.py", line 299, in callit % (cmd_desc, proc.returncode)) OSError: Command curl --silent -L http://nodejs.org/dist/node-v-RC1.tar.gz | tar xzf - -C "'C:\Users\Gladson\De...este\env_node\src'" failed with error code 255

Have to install all dependencies:
https://github.com/ekalinin/nodeenv#dependency

It would be better if there were not so many dependencies.

@gladson
Copy link
Author

gladson commented Feb 27, 2014

As I'm not very good, even in coding in Python, I think this would be the way:

To dependence:

curl
http://stackoverflow.com/a/22721/3191497
http://docs.python.org/2/library/urllib2.html#examples

egrep
http://docs.python.org/2/library/re.html
http://docs.python.org/2/library/re.html#examples

sort
http://docs.python.org/2/library/functions.html#sorted
http://www.tutorialspoint.com/python/list_sort.htm

tail
https://gist.github.com/amitsaha/5990310

tar
http://docs.python.org/2/library/tarfile.html#module-tarfile
http://docs.python.org/2/library/tarfile.html#examples

I do not know if this list is correct with the list of dependencies, it may be that some are not compatible with the functionality of the project.
But I think this is the way.
What do you think, @ekalinin?

@ekalinin
Copy link
Owner

It's a good idea, but I have no windows-like system near me.
But patches are welcome.

@gladson
Copy link
Author

gladson commented Feb 28, 2014

👍
Thanks man.

@neverfox
Copy link

It's failing because curl --silent -L http://nodejs.org/dist/node-v-RC1.tar.gz returns a 404.

@asottile
Copy link
Contributor

asottile commented Feb 8, 2015

I'm going to poke at this today (trying to get better windows support in pre-commit/pre-commit#159). I think eliminating a few of the requirements is pretty straightforward.

@ekalinin
Copy link
Owner

ekalinin commented Feb 8, 2015

👍

@asottile
Copy link
Contributor

asottile commented Feb 8, 2015

Some notes for now:

The configuration I'm targeting is Git Shell (provided by github tools), though I should be able to get it working with less than that probably.

  • Don't require sort or egrep #114 eliminates requirements for egrep and sort
  • curl (at least in my configuration) has different options than unix. Should be easy enough to replace with the urllib equivalent however and eliminate that binary dependency.
  • tar (at least in my configuration) seems identical to POSIX tar
  • The node binaries are packaged differently for windows (ugh)
    • For nodejs.org they are at /dist/{version}/node.exe for x86 and /dist/{version}/x64/node.exe for x86_64
    • For iojs.org they are at /dist/{version}/win-x86 and /dist/{version}/win-x64

@techtonik
Copy link
Contributor

I believe this now can be moved to #140, because error message have changed since then, and some progress was made on --prebuilt binaries.

@ekalinin
Copy link
Owner

Will be fixed in #140

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

No branches or pull requests

5 participants