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
Replace LinkDef files with XML selextion files.in CalibCalorimetry/EcalLaserAnalyzer #11673
Replace LinkDef files with XML selextion files.in CalibCalorimetry/EcalLaserAnalyzer #11673
Conversation
A new Pull Request was created by @wmtan for CMSSW_7_6_X. Replace LinkDef files with XML selextion files.in CalibCalorimetry/EcalLaserAnalyzer It involves the following packages: CalibCalorimetry/EcalLaserAnalyzer @cmsbuild, @mmusich, @diguida, @franzoni, @cerminar can you please review it and eventually sign? Thanks. |
@cmsbuild please test |
The tests are being triggered in jenkins. |
-1 Tested at: c6598c4 cmsDriver.py RelVal -s HLT:50nsGRun,RAW2DIGI,L1Reco,RECO --data --scenario=pp -n 10 --conditions auto:run2_data_50nsGRun --relval 9000,50 --datatier "RAW-HLT-RECO" --eventcontent FEVTDEBUGHLT --customise=HLTrigger/Configuration/CustomConfigs.L1THLT --customise=SLHCUpgradeSimulations/Configuration/postLS1Customs.customisePostLS1_50ns --magField 38T_PostLS1 --processName=HLTRECO --filein file:RelVal_Raw_50nsGRun_DATA.root --fileout file:RelVal_Raw_50nsGRun_DATA_HLT_RECO.root : FAILED - time: date Thu Oct 8 18:39:24 2015-date Thu Oct 8 18:32:59 2015 s - exit: 16640 you can see the results of the tests here: |
@cmsbuild |
@cmsbuild please test |
The tests are being triggered in jenkins. |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_6_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
+1 |
…EcalLaserAnalyzer Replace LinkDef files with XML selextion files.in CalibCalorimetry/EcalLaserAnalyzer
@wmtan I see build failures in CalibCalorimetry/EcalLaserAnalyzer package in CMSSW_7_6_DEVEL_X_2015-10-12-2300 (ROOT 6.04).
|
@davidlt Those messages are non-fatal, and were already there with the LinkDef file, so they were not introduced by this PR. Since I don't own those classes, I hesitate to add a ClassDef(). |
It's fixed in ROOT 6.04 and they are fatal. A number of these classes contain commented out ClassDef lines, which are required. I can make a PR tomorrow. |
RFC: #11787 |
It has been decided to replace LinkDef files with XML selection files.
This PR does this for the LinkDef files that were in CalibCalorimetry/EcalLaserAnalyzer.
A reorganization of existing files was needed because XML selection files are not allowed in plugins.
LinkDef files should not be in plugins, either, but that was never enforced.