Skip to content
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

Move FWEveDigitSetScalableMarker delcaration in header file. Related to #332 #578

Merged
merged 1 commit into from Aug 21, 2013

Conversation

alja
Copy link
Contributor

@alja alja commented Aug 20, 2013

In pull request #332 were problems with dictionary since Fireworks/Core/src/FWEveDigitSetScalableMarker.cc file was included in classes.h.

I have split the code into source and header so that classes.h include only header files. As I understand a workaround for problem in #332 has already been made, but not included in the official cmssw repository.

@ghost ghost assigned nclopezo Aug 21, 2013
@cmsbuild
Copy link
Contributor

+1

@ktf
Copy link
Contributor

ktf commented Aug 21, 2013

ciao @alja I'll use this pull request as a test, pardon extra emails.

@ktf
Copy link
Contributor

ktf commented Aug 21, 2013

+1

@ktf
Copy link
Contributor

ktf commented Aug 21, 2013

-1

@ktf
Copy link
Contributor

ktf commented Aug 21, 2013

+1

@ktf
Copy link
Contributor

ktf commented Aug 21, 2013

hold

@ktf
Copy link
Contributor

ktf commented Aug 21, 2013

+1

@ktf
Copy link
Contributor

ktf commented Aug 21, 2013

@nclopezo please test.

@nclopezo
Copy link
Contributor

Hi,

Anyway, I took these changes on top of CMSSW_7_0_X_2013-08-21-0200, I ran the unit tests and RelVals, all tests passed.

You can see the logs here:

https://cmssdt.cern.ch/jenkins/job/Pull-Request-Integration/ARCHITECTURE=slc5_amd64_gcc472/202/consoleFull

ktf added a commit that referenced this pull request Aug 21, 2013
Move FWEveDigitSetScalableMarker delcaration in header file. Related to #332
@ktf ktf merged commit 5a5ae00 into cms-sw:CMSSW_7_0_X Aug 21, 2013
@alja alja deleted the devel_70 branch April 15, 2014 21:35
nclopezo pushed a commit to nclopezo/cmssw that referenced this pull request Apr 30, 2014
Use root 5.34.17 in stable branch
gpetruc pushed a commit to gpetruc/cmssw that referenced this pull request Jan 4, 2016
Include the possibility to use the conversion-safe electron veto in the PhotonAnalyzer
cmsbuild pushed a commit that referenced this pull request Nov 27, 2020
Move SiPixelFedCablingMapGPU code from RecoLocalTracker/SiPixelClusterizer/ to
  - CondFormats/SiPixelObjects/
  - CalibTracker/SiPixelESProducers/
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants