-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Fixed thread-safety issues with GeometricDetExtra #2806
Fixed thread-safety issues with GeometricDetExtra #2806
Conversation
The unused method setGeomgraphicalId was const and was used to change a mutable. By changing the method to non-const we could get rid of the mutable and solve the thread-safety issue. No other code is affected by this change.
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_7_1_X. Fixed thread-safety issues with GeometricDetExtra It involves the following packages: Geometry/TrackerNumberingBuilder @civanch, @Dr15Jones, @ianna, @mdhildreth, @cmsbuild, @nclopezo, @Degano, @ktf can you please review it and eventually sign? Thanks. |
+1 |
-1 from /afs/cern.ch/cms/sw/ReleaseCandidates/vol0/slc6_amd64_gcc481/cms/cmssw/CMSSW_7_1_X_2014-03-08-1400/src/DataFormats/Common/interface/ProductData.h:11, from /afs/cern.ch/cms/sw/ReleaseCandidates/vol0/slc6_amd64_gcc481/cms/cmssw/CMSSW_7_1_X_2014-03-08-1400/src/DataFormats/Common/interface/BasicHandle.h:28, from /afs/cern.ch/cms/sw/ReleaseCandidates/vol0/slc6_amd64_gcc481/cms/cmssw/CMSSW_7_1_X_2014-03-08-1400/src/FWCore/Framework/interface/Event.h:20, from /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_7_1_X-slc6_amd64_gcc481/CMSSW_7_1_X_2014-03-08-1400/src/SimTracker/TrackerHitAssociation/test/TestAssociator.h:12, from /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_7_1_X-slc6_amd64_gcc481/CMSSW_7_1_X_2014-03-08-1400/src/SimTracker/TrackerHitAssociation/test/TestAssociator.cc:11: /afs/cern.ch/cms/sw/ReleaseCandidates/vol0/slc6_amd64_gcc481/cms/cmssw/CMSSW_7_1_X_2014-03-08-1400/src/FWCore/Utilities/interface/TypeWithDict.h:20:23: fatal error: TDataType.h: No such file or directory #include "TDataType.h" ^ compilation terminated. >> Compiling edm plugin /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_7_1_X-slc6_amd64_gcc481/CMSSW_7_1_X_2014-03-08-1400/src/SimTracker/SiStripDigitizer/plugins/DigiSimLinkPileUpSignals.cc In file included from /afs/cern.ch/cms/sw/ReleaseCandidates/vol0/slc6_amd64_gcc481/cms/cmssw/CMSSW_7_1_X_2014-03-08-1400/src/DataFormats/Provenance/interface/BranchDescription.h:16:0, you can see the results of the tests here: |
you are building against a too old IB (i.e. 08/03/2014). This pull request is on top of the present head of CMSSW_7_1_X. |
Ok, I will restart the tests. I was using the latest available IB. |
Geometry/TrackerNumberingBuilder -- Fixed thread-safety issues with GeometricDetExtra
The unused method setGeomgraphicalId was const and was used to change
a mutable. By changing the method to non-const we could get rid of the
mutable and solve the thread-safety issue. No other code is affected
by this change.