Skip to content

Conversation

@guitargeek
Copy link
Contributor

We should give the user a helpful error message if the ROOT and Python interpreter versions don't match. Otherwise, one will get confusing undefined symbol error or other unclear behavior at runtime.

This came up sometimes in forum posts or GitHub issues, so the clear error would have helped.

We should give the user a helpful error message if the ROOT and Python
interpreter versions don't match. Otherwise, one will get confusing
`undefined symbol` error or other unclear behavior at runtime.

This came up sometimes in forum posts or GitHub issues, so the clear
error would have helped.
@github-actions
Copy link

github-actions bot commented Feb 8, 2026

Test Results

    22 files      22 suites   3d 16h 18m 12s ⏱️
 3 788 tests  3 788 ✅ 0 💤 0 ❌
76 171 runs  76 171 ✅ 0 💤 0 ❌

Results for commit 4230e66.

♻️ This comment has been updated with latest results.

Copy link
Member

@dpiparo dpiparo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for this useful improvement.

Copy link
Member

@vepadulano vepadulano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the record, the proper/more established approach would be to develop CPython extensions that only use Python limited API and thus can be built to be ABI compatible across all Python versions supported. After discussion with @guitargeek , it's clear that this is currently not possible for cppyy, so this approach is the next best thing 👍

@guitargeek guitargeek merged commit 2401d1f into root-project:master Feb 9, 2026
53 of 56 checks passed
@guitargeek guitargeek deleted the python_version_check branch February 9, 2026 08:32
@dpiparo
Copy link
Member

dpiparo commented Feb 9, 2026

/backport to 6.38

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