Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

PEP 739 / build-details.json: c_api.include is used instead of c_api.headers #134455

Copy link
Copy link
Closed
@mgorny

Description

@mgorny
Issue body actions

Bug report

Bug description:

PEP 739 specifies that the CPython header directory is included in the c_api.headers key. However, it seems that the implemented that landed in #130069 uses c_api.include instead:

{
  ...
  "c_api": {
    "include": "/usr/include",
    "pkgconfig_path": "/usr/lib64/pkgconfig"
  }
}

I don't see any evidence that the name was changed deliberately. In fact, the code seems to be mixing both include and headers names:

data['c_api']['include'] = INCLUDEDIR

PATH_KEYS = [
'base_interpreter',
'libpython.dynamic',
'libpython.dynamic_stableabi',
'libpython.static',
'c_api.headers',
'c_api.pkgconfig_path',
]

CPython versions tested on:

3.14, CPython main branch

Operating systems tested on:

Linux

Linked PRs

Metadata

Metadata

Assignees

No one assigned

    Labels

    type-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      Morty Proxy This is a proxified and sanitized view of the page, visit original site.