search

project-everest / everest

https://project-everest.github.io/
Apache License 2.0
193 stars 29 forks source link

SCons check broken on Windows #23

Closed andreasdotorg closed 7 years ago

andreasdotorg commented 7 years ago

The everest script doesn't properly check for SCons on Windows, it merely checks whether Python 2.7 is installed. It also doesn't install SCons, even though it installs everything else needed.

sishtiaq commented 7 years ago

Hi @andreas23,

Not the answer to your question but this (manual) process got me scons running on Windows+Cygwin:

  1. Install python 32-bit. Allow the installer to add python to your %PATH% (by default, it doesn't);
  2. Install scons. (If you had installed python 64-bit, scons complains "Python...not found in the registry"; it will only find python if you installed the 32-bit one);

scons.bat should be in Cygwin's $PATH (inherited from Windows %PATH%) now:

$ which scons.bat
/cygdrive/c/Python27/Scripts/scons.bat

Also (to be pedantic!), the script doesn't install python.

sishtiaq commented 7 years ago

Try the version in https://github.com/project-everest/everest/tree/sishtiaq_iss23. If it works, I'll merge it.

andreasdotorg commented 7 years ago

Mhh. I have a hard time testing this, because the script now tries to re-install everything. I'll try to find out what's broken. I've had a look at the patches on this branch though, and they look good enough to me.

sishtiaq commented 7 years ago

Branch merged. The script is no different in terms of install. Maybe you can copy the output from your run of the script, so we can help you debug?

andreasdotorg commented 7 years ago

Kicked a few things (interfering other POSIXy stuff on the path), and it's working just fine now.