Skip to content

gh-109136: Fix summarize_stats.py script#109137

Merged
vstinner merged 1 commit intopython:mainfrom
mdboom:fix-summarize-stats
Sep 8, 2023
Merged

gh-109136: Fix summarize_stats.py script#109137
vstinner merged 1 commit intopython:mainfrom
mdboom:fix-summarize-stats

Conversation

@mdboom
Copy link
Contributor

@mdboom mdboom commented Sep 8, 2023

The defines about some stats were moved from Include/pystats.h to Include/cpython/pystats.h, but the path was not updated in this script.

@vstinner
Copy link
Member

vstinner commented Sep 8, 2023

I confirm that the change fix ./python Tools/scripts/summarize_stats.py.

The defines about some stats were moved from Include/pystats.h to Include/cpython/pystats.h, but the path was not updated in this script.

Oh. I'm sure that I made that change, but apparently it got lost when I modified my PR multiple times.

Thanks for the fix @mdboom!

@vstinner vstinner merged commit 52beebc into python:main Sep 8, 2023
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.

4 participants