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

Resurrect needed cfi files for HLT ConfDB parsing (each plugin needs a c... #2346

Merged
merged 1 commit into from Feb 7, 2014

Conversation

Martin-Grunewald
Copy link
Contributor

...fi)

Resurrect needed cfi files for HLT ConfDB parsing (each plugin needs a cfi)

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 7, 2014

A new Pull Request was created by @Martin-Grunewald (Martin Grunewald) for CMSSW_7_0_X.

Resurrect needed cfi files for HLT ConfDB parsing (each plugin needs a c...

It involves the following packages:

RecoTauTag/RecoTau

@nclopezo, @cmsbuild, @anton-a, @thspeer, @slava77, @Degano 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.
@nclopezo, @ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@ktf
Copy link
Contributor

ktf commented Feb 7, 2014

Bypassing this since it's critical for HLT. Complain if not ok.

ktf added a commit that referenced this pull request Feb 7, 2014
HLT fixes -- Resurrect needed cfi files for HLT ConfDB parsing (each plugin needs a c...
@ktf ktf merged commit e4a0c1a into cms-sw:CMSSW_7_0_X Feb 7, 2014
@slava77
Copy link
Contributor

slava77 commented Feb 7, 2014

Martin,
Is there a way (script or unit tests?) to catch some problems earlier?
These files were removed with the 2014 tau reco updates about two weeks ago.

@Martin-Grunewald
Copy link
Contributor Author

@Martin-Grunewald Martin-Grunewald deleted the RecoTauTagRecoTauFix branch April 1, 2014 05:19
ggovi pushed a commit to ggovi/cmssw that referenced this pull request Jan 11, 2017
davix-toolfile: use include/davix for INCLUDE
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

5 participants