Uh oh!
There was an error while loading. Please reload this page.
gh-98894: Fix DTrace test_check_probes for shared builds - #151122
Conversation
When building with --enable-shared, the SystemTap/DTrace notes live in libpython. Add detection logic to be used by readelf.
stratakis
commented
Jun 9, 2026
Easily reproducible traceback by:
This should cover also the install builds, the out of tree builds and their combination. Tested them all and it works. I don't think a news entry is warranted here. cc @pablogsal |
vstinner
commented
Jun 9, 2026
I tested on Fedora 44. I confirm that currently, test_check_probes() of test_dtrace fails. With the change, test_dtrace pass (test_check_probes() pass). But 6 tests are skipped: Note: my locale is French ( |
vstinner
commented
Jun 9, 2026
Ah, I also had to install Linux kernel header files: After these installations, 6 tests are still skipped even if I run test_dtrace as root: |
|
vstinner
left a comment
There was a problem hiding this comment.
LGTM.
The code is a little bit complicated (I expected shorter code), but it works as expected. So I'm fine with it.
Uh oh!
There was an error while loading. Please reload this page.
vstinner
commented
Jun 9, 2026
I also installed I ran the bpftrace test manually as root. I had to stop it with CTRL+C, it does not display anything. |
stratakis
commented
Jun 9, 2026
Locale issue fixed. This shouldn't be backported 3.13 and 3.14 as this actual fixes this is based on is from 3.15+: 9a268e3 |
Uh oh!
There was an error while loading. Please reload this page.
Thanks @stratakis for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.15. |
GH-151235 is a backport of this pull request to the 3.15 branch. |
vstinner
commented
Jun 10, 2026
Merged, thanks for your fix! |
…51122) (#151235) gh-98894: Fix DTrace test_check_probes for shared builds (GH-151122) When building with --enable-shared, the SystemTap/DTrace notes live in libpython. Add detection logic to be used by readelf. Force the C locale on readelf output. (cherry picked from commit 3a08e93) Co-authored-by: stratakis <cstratak@redhat.com>
…n#151122) When building with --enable-shared, the SystemTap/DTrace notes live in libpython. Add detection logic to be used by readelf. Force the C locale on readelf output.
When building with --enable-shared, the SystemTap/DTrace notes live in libpython. Add detection logic to be used by readelf.