The Chromium bots were supposed to have their Python upgraded ages ago. If we ever want to run the LayoutTests on them, they'll need their Python upgraded anyway. I think the correct solution is to fix the bots. Adam On Sat, Aug 14, 2010 at 12:00 AM, Eric Seidel <eric@webkit.org> wrote:
Fixed that one, but errors continue:
1>json 1>Traceback (most recent call last): 1> File "../../WebKitTools/Scripts/create-html-entity-table", line 40, in ? 1> import simplejson 1> File "c:\WebKitBuildSlave\chromium-win-release\build\WebKitTools\Scripts/webkitpy/thirdparty\simplejson\__init__.py", line 95, in ? 1> from decoder import JSONDecoder 1> File "c:\WebKitBuildSlave\chromium-win-release\build\WebKitTools\Scripts/webkitpy/thirdparty\simplejson\decoder.py", line 6 1> from .scanner import Scanner, pattern 1> ^ 1>SyntaxError: invalid syntax
Turs out that simplejson (the version we have in webkitpy/thirdparty) requires 2.5+. The cr-win bot has Python 2.3 (at least in its path).
I'm not yet sure what the right solution is. We may end up rolling out.
-eric
On Fri, Aug 13, 2010 at 10:21 PM, Eric Seidel <eric@webkit.org> wrote:
Sorry for the spam. I've found the error.
2>cygwin - 0 error(s), 0 warning(s) 1>Traceback (most recent call last): 1> File "../../WebKitTools/Scripts/create-html-entity-table", line 34, in ? 1> import webkitpy.thirdparty.simplejson as simplejson 1> File "c:\WebKitBuildSlave\chromium-win-release\build\WebKitTools\Scripts\webkitpy\thirdparty\__init__.py", line 101 1> with codecs.open(readme_path, "w", "ascii") as file: 1> ^ 1>SyntaxError: invalid syntax
I'll figure out how to fix the sys.path to avoid including thirdparty/__init__.py
-eric
On Fri, Aug 13, 2010 at 10:19 PM, Eric Seidel <eric@webkit.org> wrote:
http://build.webkit.org/builders/Chromium%20Win%20Release/builds/10478/steps...
It seems that cr-win for some reason isn't generating the HTMLEntityTable.cpp file like all the others are.
Is it missing python? (Or python from its path?)
Is there a gyp dependency problem?
Any thoughts would be most appreciated.
-eric