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

Fix issue 1279: getByToken implementation reveal a problem due to import... #1297

Merged
merged 1 commit into from
Nov 2, 2013

Conversation

acaudron
Copy link

@acaudron acaudron commented Nov 2, 2013

...ing the code to book histograms to the beginRun.
DQMOffline/RecoB
Validation/RecoB

…orting the code to book histograms to the beginRun.
@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 2, 2013

A new Pull Request was created by @acaudron for CMSSW_7_0_X.

Fix issue 1279: getByToken implementation reveal a problem due to import...

It involves the following packages:

DQMOffline/RecoB
Validation/RecoB

@danduggan, @nclopezo, @eliasron, @rovere, @deguio 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.

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 2, 2013

@ktf
Copy link
Contributor

ktf commented Nov 2, 2013

This fixes a problem in the release. I'll approve it and kindly ask @deguio @rovere and @eliasron to complain if this is not correct.

ktf added a commit that referenced this pull request Nov 2, 2013
DQM fix -- Fix issue #1279 related to consumes migration for BTag DQM.
@ktf ktf merged commit 66ebbf1 into cms-sw:CMSSW_7_0_X Nov 2, 2013
@deguio
Copy link
Contributor

deguio commented Nov 3, 2013

thanks for this. I tested on my side also and everything works fine.

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.

4 participants