root icon indicating copy to clipboard operation
root copied to clipboard

[rootx] Turn `root` executable into symbolic link to `root.exe`

Open guitargeek opened this issue 2 months ago • 5 comments

There is no need to two separate executables anymore, but for backwards compatibility we keep root.exe and then root is a symbolic link to it.

guitargeek avatar Sep 29 '25 17:09 guitargeek

Test Results

    2 files      2 suites   9h 32m 52s ⏱️ 3 458 tests 3 363 ✅ 95 💤 0 ❌ 6 903 runs  6 903 ✅  0 💤 0 ❌

Results for commit 0ce4d20d.

:recycle: This comment has been updated with latest results.

github-actions[bot] avatar Sep 29 '25 19:09 github-actions[bot]

is a symbolic link to it.

Rather than this technical choice, we may want to just copy/duplicate. We just replaced the symbolic link genreflex/rootcling by a copy.

pcanal avatar Nov 03 '25 21:11 pcanal

is a symbolic link to it.

Rather than this technical choice, we may want to just copy/duplicate. We just replaced the symbolic link genreflex/rootcling by a copy.

With debug symbols, they are < 100k, so this would be a viable choice.

hageboeck avatar Nov 04 '25 09:11 hageboeck

About how ROOTSYS could still be used, I'm suggesting to not set ROOTSYS anymore in our tests: https://github.com/root-project/root/pull/20408

Like this, our tests have a higher change of spotting problems if setting ROOTSYS in the wrapping root executable is missing.

guitargeek avatar Nov 13 '25 12:11 guitargeek

roottest/root/rint/CMakeLists.txt: COMMAND $<TARGET_FILE:root> -b -q -l -e ".libraries" might need root.exe instead of root?

ferdymercury avatar Dec 08 '25 08:12 ferdymercury