Skip to content

[mathtext] mv to builtin subdir for consistency#22462

Merged
linev merged 6 commits into
root-project:masterfrom
ferdymercury:submath
Jun 8, 2026
Merged

[mathtext] mv to builtin subdir for consistency#22462
linev merged 6 commits into
root-project:masterfrom
ferdymercury:submath

Conversation

@ferdymercury

@ferdymercury ferdymercury commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

as was done before with hard-coded builtin afterimage

There are some differences from ROOTs version (2012) with the upstream version (2016) but I do not know if it's worth to re-sync or even fully isolate from ROOT_LINKER_LIBRARY and have it as standalone lib.

upstream.diff.txt

@github-actions

github-actions Bot commented Jun 2, 2026

Copy link
Copy Markdown

Test Results

    22 files      22 suites   3d 11h 43m 42s ⏱️
 3 856 tests  3 856 ✅ 0 💤 0 ❌
76 166 runs  76 166 ✅ 0 💤 0 ❌

Results for commit 76d6696.

♻️ This comment has been updated with latest results.

@linev linev self-assigned this Jun 3, 2026
@ferdymercury

ferdymercury commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator Author

An expanded version of this PR that syncs with the 2016 upstream fork is here: #22476 and decouples builtin from ROOT

@linev

linev commented Jun 8, 2026

Copy link
Copy Markdown
Member

There are some differences from ROOTs version (2012) with the upstream version (2016) but I do not know if it's
worth to re-sync or even fully isolate from ROOT_LINKER_LIBRARY and have it as standalone lib.

Difference is significant - one need test it before trying to adopt.

But rest looks fine.

@linev linev merged commit ae81825 into root-project:master Jun 8, 2026
32 of 33 checks passed
@ferdymercury ferdymercury deleted the submath branch June 8, 2026 06:13
@ferdymercury

Copy link
Copy Markdown
Collaborator Author

one need test it before trying to adopt.

Seems it only fails on Windows #22476

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.

2 participants