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
Macros need to compile in ROOT6 (Reco) #8715
Conversation
A new Pull Request was created by @wmtan for CMSSW_7_5_X. Macros need to compile in ROOT6 (Reco) It involves the following packages: RecoParticleFlow/PFClusterTools @cmsbuild, @cvuosalo, @nclopezo, @slava77 can you please review it and eventually sign? Thanks. |
@cmsbuild please test |
The tests are being triggered in jenkins. |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @ktf, @smuzaffar |
Since Slava is correct that it appears these changes are harmless for ROOT5, I removed the request to not forward these to 7_5_ROOT5_X from the original comment. |
+1 |
Macros need to compile in ROOT6 (Reco)
These changes are harmless for ROOT5. |
In ROOT6 macros are processed by cling, rather than CINT. Over 500 CMSSW macros do not compile in ROOT6. Since that is too many macros to be fixed centrally, it was decided by David Lange to centrally fix only those 45 macros with compilation errors that have been modified since the switch over to git, since those are the ones most likely to be used. A few of these 45 macros are in the Reconstruction L2 category. This pull request partially fixes the compilation errors. I could not fully fix RecoParticleFlow/PFClusterTools/test/src/testWrappers.C, because I could not find any place in CMSSW where type "SingleParticleWrapper" was declared. I chose not to fix RecoParticleFlow/PFClusterTools/test/src/clusterCalibrate.C because its only problem is that a constructor that it needs, pftools::PFClusterCalibration(pftools::IO *) has been commented out. Since it may have been commented out for a good reason, I left this alone. To fix this macro, simply uncomment the constructor.