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

gh-100086: Add build details to sys.version #100087

Closed
wants to merge 3 commits into from
Closed

gh-100086: Add build details to sys.version #100087

wants to merge 3 commits into from

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Dec 7, 2022

Add more build details about how Python was configured in sys.version, like "debug" or "release" build.

Build details and related configure option:

  • "debug": --with-pydebug, "release" otherwise
  • "+assert": --with-assertions
  • "+tracerefs": --with-trace-refs
  • "+pystats": --enable-pystats
  • "asan": --with-address-sanitizer
  • "msan": --with-memory-sanitizer
  • "ubsan": --with-undefined-behavior-sanitizer
  • "lto+pgo": --with-lto --enable-optimizations
  • "framework": --enable-framework
  • "shared": --enable-shared

@netlify
Copy link

netlify bot commented Dec 7, 2022

Deploy Preview for python-cpython-preview canceled.

Name Link
🔨 Latest commit 5fca7d3
🔍 Latest deploy log https://app.netlify.com/sites/python-cpython-preview/deploys/63911236edff440008be36da

@vstinner
Copy link
Member Author

vstinner commented Dec 7, 2022

cc @erlend-aasland

Add more build details about how Python was configured in
sys.version, like "debug" or "release" build.

Build details and related configure option:

* "debug": --with-pydebug, "release" otherwise
* "+assert": --with-assertions
* "+tracerefs": --with-trace-refs
* "+pystats": --enable-pystats
* "asan": --with-address-sanitizer
* "msan": --with-memory-sanitizer
* "ubsan": --with-undefined-behavior-sanitizer
* "lto+pgo": --with-lto --enable-optimizations
* "framework": --enable-framework
* "shared": --enable-shared
@vstinner
Copy link
Member Author

vstinner commented Dec 7, 2022

Example of "build details" on CI jobs (test.info details):

  • Address sanitizer: "release asan build" (build.NDEBUG: ignore assertions, build.Py_DEBUG: No, sysconfig[PY_CFLAGS]: -fsanitize=address (...))
  • Ubuntu: "debug build" (build.NDEBUG: build assertions, build.Py_DEBUG: Yes)
  • macOS: "debug build" (build.Py_DEBUG: Yes)
  • Windows x64: "debug build" (build.Py_DEBUG: Yes)

@ned-deily
Copy link
Member

I don't think this change should be made. See my comment on the issue.

@vstinner
Copy link
Member Author

vstinner commented Dec 7, 2022

I don't think this change should be made. See #100086 (comment).

Ok, I wrote PR #100093 instead.

@vstinner vstinner closed this Dec 7, 2022
@vstinner vstinner deleted the py_build_str branch December 7, 2022 23:49
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.

3 participants