Bug 226617 – [Python-3] Change shebangs in JSC/inspector/scripts to Python 3

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: Tools / Tests

[hysu]

Breaking up an effort to make Python scripts in the JSC build process use Python 3 by default into multiple patches. This patch addresses the ./inspector/scripts/ directory.

[hysu]

Created attachment 430530 [details]
Patch

[Radar WebKit Bug Importer]

<rdar://problem/78848866>

[EWS Watchlist]

This patch modifies the inspector protocol generator. Please ensure that you have rebaselined any generator test results (i.e., by running `Tools/Scripts/run-inspector-generator-tests --reset-results`)

[hysu]

(Submitting this patch first as --no-review to see if it gets past EWS)

[Jonathan Bedard]

Comment on attachment 430530 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=430530&action=review

> Source/JavaScriptCore/inspector/scripts/codegen/cpp_generator.py:1
> +#!/usr/bin/env python3

Make sure that we cover any callers of these too, which may be calling via "python Source...."

[Jonathan Bedard]

Comment on attachment 430530 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=430530&action=review

> Source/JavaScriptCore/inspector/scripts/generate-inspector-protocol-bindings.py:1
> +#!/usr/bin/env python3

Looks like this is the root script, we need to make sure it's callers are invoking it with Python3

[Sam Sneddon [:gsnedders]]

*** This bug has been marked as a duplicate of bug 231587 ***