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.

Sign up for GitHub

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

Jump to bottom

L1 trigger make edm one producer #2132

Merged
merged 3 commits into from
Jan 24, 2014
Merged

L1 trigger make edm one producer #2132

merged 3 commits into from
Jan 24, 2014

Conversation

gartung
Copy link
Member

@gartung gartung commented Jan 22, 2014

Make various L1trigger producer classes inherit from edm::one::producer

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @gartung (Patrick Gartung) for CMSSW_7_1_X.

L1 trigger make edm one producer

It involves the following packages:

L1Trigger/CSCTrackFinder
L1Trigger/DTTrackFinder
L1Trigger/GlobalMuonTrigger

@cmsbuild, @Degano, @mulhearn, @nclopezo can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@cmsbuild
Copy link
Contributor

+1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2132/33/summary.html

@mulhearn
Copy link
Contributor

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_1_X IBs unless changes (tests are also fine). @ktf can you please take care of it?

ktf added a commit that referenced this pull request Jan 24, 2014
@ktf
Merge pull request #2132 from gartung/L1Trigger-make-edm-one-producer
04c5d21 
Multithreading fixes -- L1 trigger make edm one producer
@ktf ktf merged commit 04c5d21 into cms-sw:CMSSW_7_1_X Jan 24, 2014
@gartung gartung deleted the L1Trigger-make-edm-one-producer branch January 30, 2014 20:36
@nclopezo nclopezo modified the milestones: CMSSW_7_1_0_pre3, CMSSW_7_1_0_pre2 Feb 5, 2014
@nclopezo nclopezo modified the milestones: CMSSW_7_1_0_pre4, CMSSW_7_1_0_pre3 Feb 24, 2014
@nclopezo nclopezo modified the milestones: CMSSW_7_1_0_pre5, CMSSW_7_1_0_pre4 Mar 10, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
fully-signed l1-approved tests-approved
Projects
None yet
Milestone
CMSSW_7_1_0_pre5
Development

Successfully merging this pull request may close these issues.
5 participants
@gartung @cmsbuild @mulhearn @ktf @nclopezo