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
Remove use of boost::mpl from Ref.h #30876
Conversation
Using C++17 features allowed implementing the same traits check in a relatively simple manner.
The code-checks are being triggered in jenkins. |
@davidlange6 FYI |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-30876/17250
|
A new Pull Request was created by @Dr15Jones (Chris Jones) for master. It involves the following packages: DataFormats/Common @makortel, @smuzaffar, @cmsbuild, @Dr15Jones can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins.
|
-1 Tested at: 7622e30 CMSSW: CMSSW_11_2_X_2020-07-22-1200 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: You can see the results of the tests here: I found follow errors while testing this PR Failed tests: Build HeaderConsistency
I found compilation error when building: >> Compiling /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_2_X_2020-07-22-1200/src/FastSimulation/Event/src/FSimVertex.cc >> Compiling /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_2_X_2020-07-22-1200/src/FastSimulation/Event/src/KineParticleFilter.cc In file included from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_2_X_2020-07-22-1200/src/DataFormats/HepMCCandidate/interface/GenParticleFwd.h:6, from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_2_X_2020-07-22-1200/src/FastSimulation/Event/interface/FSimEvent.h:6, from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_2_X_2020-07-22-1200/src/FastSimulation/Event/src/FSimEvent.cc:2: /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_2_X_2020-07-22-1200/src/DataFormats/Common/interface/FwdRef.h:129:29: error: 'remove_cv' in namespace 'boost' does not name a template type typedef typename boost::remove_cv::type>::type key_type; ^~~~~~~~~ /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_2_X_2020-07-22-1200/src/DataFormats/Common/interface/FwdRef.h:129:38: error: expected unqualified-id before '<' token typedef typename boost::remove_cv::type>::type key_type; ^ The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: |
Comparison not run due to Build errors (RelVals and Igprof tests were also skipped) |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 |
yes @Dr15Jones , |
ping @cms-sw/l1-l2 the change in your area is just the inclusion of a missing header. |
merge |
+1 |
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 be automatically merged. |
PR description:
Using C++17 features allowed implementing the same traits check in a relatively simple manner.
This was done based on a discussion in the Core meeting about the difficulty boost::mpl poses for ROOT modules.
PR validation:
The code compiles. Checks were done on the
edm::has_key_compare
implementation using compiler explorer.