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.

Sign up for GitHub

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

Jump to bottom

Can't build C extensions with --enable-pystats builds #121583

Closed
mdboom opened this issue Jul 10, 2024 · 0 comments
Closed

Can't build C extensions with --enable-pystats builds #121583

mdboom opened this issue Jul 10, 2024 · 0 comments
Assignees
@mdboom
Labels
3.13 bugs and security fixes type-bug An unexpected behavior, bug, or error

Comments

@mdboom
Copy link
Contributor

mdboom commented Jul 10, 2024 •
edited by bedevere-app bot
Loading

Bug report

Bug description:

After building with --enable-pystats, you can't build any C extensions because Python.h includes pycode_uop_ids.h (indirectly).

      In file included from /home/mdboom/Work/builds/cpython/Include/pystats.h:16,
                       from /home/mdboom/Work/builds/cpython/Include/Python.h:65,
                       from src/_imagingmorph.c:14:
      /home/mdboom/Work/builds/cpython/Include/cpython/pystats.h:22:10: fatal error: pycore_uop_ids.h: No such file or directory
         22 | #include "pycore_uop_ids.h"
            |          ^~~~~~~~~~~~~~~~~~
      compilation terminated.

CPython versions tested on:

CPython main branch

Operating systems tested on:

Linux

Linked PRs
@mdboom mdboom added the type-bug An unexpected behavior, bug, or error label Jul 10, 2024
@mdboom mdboom self-assigned this Jul 10, 2024
@mdboom mdboom added the 3.13 bugs and security fixes label Jul 10, 2024
mdboom added a commit to mdboom/cpython that referenced this issue Jul 10, 2024
@mdboom
pythongh-121583: Remove dependency from pystats.h to internal header …
3976d10 
…file
@bedevere-app bedevere-app bot mentioned this issue Jul 10, 2024
Merged
brandtbucher pushed a commit that referenced this issue Jul 16, 2024
@mdboom @ZeroIntensity
GH-121583: Remove dependency from pystats.h to internal header file (G…
f036a46 
…H-121587)

Co-authored-by: Peter Bierma <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jul 16, 2024
@mdboom @ZeroIntensity @miss-islington
pythonGH-121583: Remove dependency from pystats.h to internal header …
6f5842e 
…file (pythonGH-121587)

(cherry picked from commit f036a46)

Co-authored-by: Michael Droettboom <[email protected]>
Co-authored-by: Peter Bierma <[email protected]>
@bedevere-app bedevere-app bot mentioned this issue Jul 16, 2024
Merged
brandtbucher pushed a commit that referenced this issue Jul 16, 2024
@miss-islington
[3.13] GH-121583: Remove dependency from pystats.h to internal header…
f888636 
… file (GH-121880)

(cherry picked from commit f036a46)
Co-authored-by: Michael Droettboom <[email protected]>
Co-authored-by: Peter Bierma <[email protected]>
@brandtbucher brandtbucher mentioned this issue Jul 17, 2024
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mdboom mdboom

Labels
3.13 bugs and security fixes type-bug An unexpected behavior, bug, or error
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mdboom @brandtbucher