The requirements download script used for wincairo was having problems on python 3. While working with that, hit error in port/win.py when trying to run check-webkit-style on windows with python 3 and a few more cases that used "is <constant>" that now give SyntaxWarnings.
Created attachment 388946 [details] Patch
Comment on attachment 388946 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=388946&action=review > Tools/Scripts/download-github-release.py:-30 > -import urllib2 We clearly don't have test coverage here. Do you know who uses this script? > Tools/Scripts/webkitpy/common/system/profiler.py:175 > + if exit_code != 0: What sort of linter are you running to find these? Is that a Python 3.8 thing? If so, would you mind run 'test-webkitpy-python3' and fixing everything it flags in a separate patch?
(In reply to Jonathan Bedard from comment #2) > Comment on attachment 388946 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=388946&action=review > > > Tools/Scripts/download-github-release.py:-30 > > -import urllib2 > > We clearly don't have test coverage here. Do you know who uses this script? Wincairo's update-webkit-wincairo-libs.py script is the only current user. > > Tools/Scripts/webkitpy/common/system/profiler.py:175 > > + if exit_code != 0: > > What sort of linter are you running to find these? Is that a Python 3.8 > thing? I think it's a syntax warning (on loading the file?) that was added in 3.8. > If so, would you mind run 'test-webkitpy-python3' and fixing everything it > flags in a separate patch? test-webkitpy-python3 was how I hit most of the "is XXX" ones in this patch. I didn't cover anything that was in thirdparty, but I think I hit all the ones apart from that, but I'll check to see if I missed anything.
(In reply to Stephan Szabo from comment #3) > (In reply to Jonathan Bedard from comment #2) > > Comment on attachment 388946 [details] > > Patch > > > > ... > > > > What sort of linter are you running to find these? Is that a Python 3.8 > > thing? > > I think it's a syntax warning (on loading the file?) that was added in 3.8. > > > If so, would you mind run 'test-webkitpy-python3' and fixing everything it > > flags in a separate patch? > > test-webkitpy-python3 was how I hit most of the "is XXX" ones in this patch. > I didn't cover anything that was in thirdparty, but I think I hit all the > ones apart from that, but I'll check to see if I missed anything. Surprised that there are so few! We might eventually need to update third party libraries to ones that advertise themselves as 3.8 compatible, but that's not an urgent task.
Comment on attachment 388946 [details] Patch Clearing flags on attachment: 388946 Committed r255243: <https://trac.webkit.org/changeset/255243>
All reviewed patches have been landed. Closing bug.
<rdar://problem/58957042>