-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
io: in UpdateFromRule properly handle case of std::string and alias without dictionary #17604
Conversation
Test Results 18 files 18 suites 4d 9h 13m 37s ⏱️ For more details on these failures, see this check. Results for commit a2647e1. ♻️ This comment has been updated with latest results. |
It can be read while being updated by another thread.
This fixes part (b) of the issue seen at: cms-sw/cmsdist#9643 (comment) The issue was due to the source type name not being known at run-time because the type name was an alias which has not dictionary request and thus was not known at the execution the header files have not been loaded (and this is indeed the intent as we strive to run I/O without loading headers)
Please note that
|
@pcanal any updates on this? As you know, this is blocking CMS from updating to the latest |
This fixes the issue seen in cms-sw/cmsdist#9643 (comment)
and is the companion PR of root-project/roottest#1256