[guitargeek]

With builtin_mathjax=ON, the mathjax distribution is only unpacked into js/mathjax in the build tree, but nothing installs it: the install rule for JSROOT copies the js directory of the source tree. JSROOT loads mathjax/es5/tex-svg.js relative to its own location (see js/modules/base/latex.mjs), so an installed ROOT always falls back to the CDN, and the builtin_mathjax option has no effect for the installation.

Install the distribution next to JSROOT, like in the build tree.

Fixes the problem that js/mathjax is missing from the install tree:

[I] omen nix-shell ~/c/r/root_build > ls js/
build  changes.md  files  img  index.htm  LICENSE  mathjax  modules  scripts
[I] omen nix-shell ~/c/r/root_build > ls ../root_install/js/
build  changes.md  files  img  index.htm  LICENSE  modules  scripts
[I] omen nix-shell ~/c/r/root_build >

This is another fix when I exercised the nix packaging of the 6.42.00 release candidate.

[linev]

There is PR #23877 which converts jsroot into builtin option.

Probably I should add there install instruction there for:

 install(DIRECTORY ${CMAKE_BINARY_DIR}/js/ DESTINATION ${CMAKE_INSTALL_JSROOTDIR}/ ${DIR_PERMISSIONS})

Then this PR will be not necessary

[guitargeek]

Yep I'm aware of that! But do you really plan to port #23877 at this point? Looks like a fat change for making after the 6.42.00 release candidate is already out. And I think your builtin_jsroot PR will require several packagers to change recipes after the release candidate. That's why I'm suggesting this minimal fix. Your #23877 can always go on top of it

[linev]

But do you really plan to port

Not sure yet.
It makes builtins organization for jsroot, openui5 and mathjax symmetric.

[guitargeek]

Not sure yet.

Okay. In this case, are you fine with merging my PR here, so things work correctly for the release whatever is the fate of #23877?

[linev]

In this case, are you fine with merging my PR here,

If you want to backport it to 6.42 - then let merge yours first and I will rebase my PR

[guitargeek]

/backport to 6.42

[root-project-bot]

Preparing to backport PR #23908 to branch 6.42 requested by guitargeek

[root-project-bot]

This PR has been backported to branch 6.42: #23913