Skip to content

[build] Do not strip away symbols from libraries and executables#22917

Open
dpiparo wants to merge 1 commit into
root-project:masterfrom
dpiparo:doNotStrip
Open

[build] Do not strip away symbols from libraries and executables#22917
dpiparo wants to merge 1 commit into
root-project:masterfrom
dpiparo:doNotStrip

Conversation

@dpiparo

@dpiparo dpiparo commented Jul 25, 2026

Copy link
Copy Markdown
Member

By default, scikit-build triggers the stripping of debugging and other symbols, in order to optimize for space.
However, the stripping is too aggressive on macOS. This has undesired effects. For example, also the rootcling symbol usedToIdentifyRootClingByDlSym is stripped. Its presence changes the behaviour of some routines some ROOT libraries, e.g. when generating dictionaries.

By default, scikit-build triggers the stripping of debugging and
other symbols, in order to optimize for space.
However, the stripping is too aggressive on macOS. This has
undesired effects. For example, also the rootcling symbol
`usedToIdentifyRootClingByDlSym` is stripped. Its presence
changes the behaviour of some routines some ROOT libraries, e.g.
when generating dictionaries.
@dpiparo
dpiparo requested a review from siliataider July 25, 2026 14:24
@dpiparo dpiparo self-assigned this Jul 25, 2026
@dpiparo dpiparo added the build-python-wheels Run the workflow to build Python wheels label Jul 25, 2026
@github-actions

Copy link
Copy Markdown

Test Results

    23 files      23 suites   3d 15h 38m 48s ⏱️
 3 874 tests  3 874 ✅ 0 💤 0 ❌
79 793 runs  79 793 ✅ 0 💤 0 ❌

Results for commit 1695340.

@siliataider

Copy link
Copy Markdown
Contributor

Very nice! this fixed the tests that relied on ACLiC with the experimental macOs wheels, except for python 3.14 ...
https://github.com/root-project/root/actions/runs/30168741943/job/89712798280?pr=22886

@dpiparo
dpiparo marked this pull request as ready for review July 26, 2026 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build-python-wheels Run the workflow to build Python wheels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants