Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

replace match statements with if statements in gdb debug script #18226

Merged
merged 1 commit into from
Apr 2, 2025

Conversation

lcarilla
Copy link
Contributor

@lcarilla lcarilla commented Apr 1, 2025

so it works with lower GDB versions, had a lot of pain with this.
match is a really new python feature, the most recent LTS ubuntu version (24.04) ships gdb 15.x which comes with python3.8 and therefore doesnt work with this script. lets make it more available to people developing without the latest gdb/python version

@iluuu1994
Copy link
Member

Hi @lcarilla. You also need to run the scripts/gdb/debug_gdb_scripts_gen.php script, I believe.

@iluuu1994 iluuu1994 requested a review from arnaud-lb April 1, 2025 21:32
@lcarilla lcarilla force-pushed the gdb-scripts-compatibility branch from 5bbd23c to 8581c69 Compare April 1, 2025 21:37
@lcarilla lcarilla requested a review from bukka as a code owner April 1, 2025 21:37
@linuseing
Copy link

Had the same issue

@arnaud-lb arnaud-lb merged commit 02d58fa into php:master Apr 2, 2025
9 checks passed
@arnaud-lb
Copy link
Member

Thank you @lcarilla!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants