-
Notifications
You must be signed in to change notification settings - Fork 3
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
seattleinstaller fails on Windows #146
Comments
Here's the raw results for a Windows 8.1 Pro 32 bit machine (German locale). I didn't ever get an empty string for
|
Here's the raw results for a Windows 7 64 bit machine. I find that platform.platform() returned windows-32 bit although it is a 64bit machine. But architecture is (64bit, windowsPE). platform.release() is empty. Results are unchanged for cmd.exe and Windows PowerShell.
|
I run Seattle installer on windows 7 pro 64bit with python2.5 or python2.7. Python 2.5 cannot work but python2.7 works. So updating python version is a solution to fix this issue. Here are my test results. This is running on python2.5
This is running on python2.7
|
Hi Tested the above ticket for pythn 2.5.1, 2.5.2, 2.5.4 and 2.7.5 and it works well in call cases. below is the trace:- C:\Users\swap\Desktop\seattle_win\seattle>python
C:\Users\swap>python
C:\Users\swap\Desktop\seattle_win\seattle>python
|
I tested repyV1 before and it works for python2.7. But when I test repyV2 for python2.5 or 2.7, they can't work. Python2.5 result is the same as repyV1. Here is python2.7 result:
|
Okay, thanks for testing. What does |
installer_benchmark.log contains:
|
With "repyV1" and "repyV2" you mean installers from seattleclearinghouse.poly.edu and sensibilityclearinghouse.poly.edu, right? |
I download from https://seattleclearinghouse.poly.edu/download/flibble/. It is repyV1. And I download repyV2 from https://github.com/SeattleTestbed/repy_v2. |
Use an installer from https://sensibilityclearinghouse.poly.edu/geni/download/altruistic/ to test RepyV2 installation. |
Okay. I have tested repyV2. It works on windows 7 pro 64bit with python2.7. We don't need to modify seattleinstaller.py when updating to python2.7
|
If you check platform.release() it returns value as '8' for windows 8 in latest python version... |
Thanks! I don't test it on windows 8 machine. So it can't work fine in windows 8? Maybe we should add release == 8 to support window 8 system. |
@yyzhuang reports that the Seattle installer failed on Windows. The debug message shows that
platform.architecture()
returned 32 bits andWindowsPE
, although it's a 64 bit machine.platform.release
is empty altogether.Given we include a very old version of Python in the Win installer (see #59), it's likely that the old platform detection code gets the OS string wrong, and maybe just reports that the Python binary is 32 bit instead of checking the
/proc/cpuinfo
equivalent of Windows.In addition to the fix, the Win/Python version matrix in SeattleTestbed/attic#1059 needs to be amended.
The text was updated successfully, but these errors were encountered: