root icon indicating copy to clipboard operation
root copied to clipboard

Remove dictionaries for raw file

Open jblomer opened this issue 1 month ago • 5 comments

jblomer avatar Nov 07 '25 22:11 jblomer

Test Results

    20 files      20 suites   3d 15h 52m 39s ⏱️  3 786 tests  3 778 ✅ 0 💤 8 ❌ 73 867 runs  73 859 ✅ 0 💤 8 ❌

For more details on these failures, see this check.

Results for commit bfa08284.

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

github-actions[bot] avatar Nov 08 '25 03:11 github-actions[bot]

It seems this breaks the plugin mechanism for RRawFileDavix without runtime_cxxmodules, probably because ROOT cannot locate which library to load. Maybe we can get around it with #pragma link C++ class ROOT::Internal::RRawFileDavix-;, pretending to have a custom streamer?

hahnjo avatar Nov 08 '25 09:11 hahnjo

It seems this breaks the plugin mechanism for RRawFileDavix without runtime_cxxmodules, probably because ROOT cannot locate which library to load. Maybe we can get around it with #pragma link C++ class ROOT::Internal::RRawFileDavix-;, pretending to have a custom streamer?

Strange. In the plug-in description, we specify the library. Also, why does it work then for the XRootD raw file that doesn't have a dictionary.

jblomer avatar Nov 08 '25 20:11 jblomer

Also, why does it work then for the XRootD raw file that doesn't have a dictionary.

Hm, are we testing the plugin support for the root:// protocol? net/netxng/test/RRawFileNetXNG.cxx constructs RRawFileNetXNG directly...

hahnjo avatar Nov 09 '25 21:11 hahnjo

The motivation of the failing mac-beta build should be addressed by https://github.com/root-project/root/pull/20389

hahnjo avatar Nov 12 '25 15:11 hahnjo

Since the motivation is gone and this PR causes problems, I'll close it.

jblomer avatar Dec 16 '25 13:12 jblomer