| Summary: | Python 3: Update windows scripts and uses of is constant | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Stephan Szabo <stephan.szabo> | ||||
| Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | commit-queue, ews-watchlist, glenn, jbedard, webkit-bug-importer | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | WebKit Nightly Build | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Attachments: |
|
||||||
|
Description
Stephan Szabo
2020-01-27 17:16:23 PST
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. |