WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
225268
start-local-buildbot-server should fail gracefully for Python 2
https://bugs.webkit.org/show_bug.cgi?id=225268
Summary
start-local-buildbot-server should fail gracefully for Python 2
Aakash Jain
Reported
2021-05-01 06:33:30 PDT
start-local-buildbot-server currently fails with a confusing error when run with python2. [CISupport]$python start-local-buildbot-server --ews Setting up virtualenv at /private/var/folders/k1/w4qtfm5j7_v36jwtf1zg50n00000gn/T/tmpuZB6fl ... ... Process Process-1: Traceback (most recent call last): File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/multiprocessing/process.py", line 267, in _bootstrap self.run() File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/multiprocessing/process.py", line 114, in run self._target(*self._args, **self._kwargs) File "start-local-buildbot-server", line 291, in _start_server if check_tcp_port_open('localhost', self._server_pb_port): File "start-local-buildbot-server", line 79, in check_tcp_port_open except ConnectionRefusedError: NameError: global name 'ConnectionRefusedError' is not defined ConnectionRefusedError is a built-in exception in Python 3, but doesn't work in Python 2. Instead of failing with this exception, we should gracefully exit when the script is invoked with Python 2 and print a proper error message.
Attachments
[fast-cq] Patch
(1.52 KB, patch)
2021-05-01 06:34 PDT
,
Aakash Jain
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Aakash Jain
Comment 1
2021-05-01 06:34:31 PDT
Created
attachment 427496
[details]
[fast-cq] Patch
Aakash Jain
Comment 2
2021-05-01 06:35:48 PDT
This is similar to
https://github.com/WebKit/WebKit/blob/main/Tools/CISupport/ews-build/steps.py#L42
EWS
Comment 3
2021-05-01 18:32:21 PDT
Committed
r276878
(
237224@main
): <
https://commits.webkit.org/237224@main
> All reviewed patches have been landed. Closing bug and clearing flags on
attachment 427496
[details]
.
Radar WebKit Bug Importer
Comment 4
2021-05-01 18:33:13 PDT
<
rdar://problem/77424789
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug