Add dictionaries to avoid header parsing#50705
Conversation
|
cms-bot internal usage |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50705/48974
|
|
A new Pull Request was created by @makortel for master. It involves the following packages:
@Moanwar, @battibass, @cmsbuild, @ftorrresd, @jfernan2, @lviliani, @mandrenguyen, @mkirsano, @sensrcn, @srimanob, @theofil can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
@cmsbuild, please test |
|
+1 Size: This PR adds an extra 60KB to repository Comparison SummarySummary:
|
|
+1 |
|
+1 |
b64dae9 to
e459dcb
Compare
|
Sorry, one more came up. This is the last update. |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50705/49013
|
|
@cmsbuild, please test |
|
+1 Size: This PR adds an extra 24KB to repository Comparison SummarySummary:
|
|
+1 |
|
@cms-sw/generators-l2 Could you review and sign, please? |
|
+generators |
|
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @ftenchini, @mandrenguyen (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 |
PR description:
This PR adds dictionaries for types that were found to cause header parsing in ROOT master in #50580 (comment) (for TTree IO) and #49949 (comment) (for RNTuple IO)
Resolves cms-sw/framework-team#2153
PR validation:
With these changes the
gInterpreter->Print("autoparsed")(via #50632) no longer listed types causing header parsing that we could easily do something about.