-
Notifications
You must be signed in to change notification settings - Fork 6
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
It is not possible to install it on windows #15
Comments
Python is required by If you are familiar with this topic, please feel free to submit a pull request and I will take a look at it :) |
Why not include pre-compiled versions of lzo library to the package? |
I guess this is a possibility, but then you would also need to either pre-compile it for all platforms or have a fallback that will try to compile it locally when there's no pre-compiled version available. However, in the latter case, one needs to have python installed again. You also need to include all those builds in the package tar-ball, which would considerably increase the size. Of course, you can host those builds on a server and just download them as needed, but such infrastructure requires some maintenance. |
I just wanted to add that you should absolutely share your ideas and thoughts! My last post sounded kind of cynical and like I just wanted to shut this issue down, but that's not the case :) |
I'm not very familiar with npm. I just thought that "npm install" simply downloads dependencies without any additional actions. |
Windows does not have a python, so npm install finished with the exception
The text was updated successfully, but these errors were encountered: