Migration to consumes -- CSCSkim

[ptcox]

Add consumes interface to DPGAnalysis/Skims/CSCSkim. This compiles fine and looks OK to me, but is untested since I have no convenient way to do it. I avoid the necessity of modifying 14 associated config files by hard-coding the InputTags for sim and real wire and strip digis required by the consumes interface. (The existing configs only contain InputTags for rechits, segments, trand-alone muons, global muons, tracks.)

[cmsbuild]

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

Consumes to CSCSkim

It involves the following packages:

DPGAnalysis/Skims

@franzoni, @fabiocos, @vlimant 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.

[ktf]

@fabiocos who should sign off for skims nowadays?

[nclopezo]

+1
you can see the logs here:
https://cmssdt.cern.ch/jenkins/job/Pull-Request-Integration/ARCHITECTURE=slc5_amd64_gcc481/494/consoleFull
and the artifacts here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/494

[fabiocos]

As discussed at the ORP today this code is basically unattended as present.
If the technical checks are passed (as D. Mendes is reporting) please take it. Validations will have
to assess the physics correctness of it.

Fabio

On Sep 13, 2013, at 12:31 PM, Giulio Eulisse notifications@github.com wrote:

@fabiocos who should sign off for skims nowadays?

—
Reply to this email directly or view it on GitHub.

[ktf]

Moved to pre5 until we find a maintainer. @cerminar