FlatPU

[tomcornelis]

Also pull request for CMSSW_7_0_X branch

[cmsbuild]

A new Pull Request was created by @UAEDF-tomc (Tom Cornelis) for CMSSW_7_0_X.

FlatPU

It involves the following packages:

Configuration/StandardSequences
SimGeneral/MixingModule

@civanch, @nclopezo, @vlimant, @mdhildreth, @cmsbuild, @franzoni, @Degano, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @cerati, @GiacomoSguazzoni, @rovere, @wmtan this is something you requested to watch as well.
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.

[cmsbuild]

-1
I found an error when building:

Compiling src/SimGeneral/MixingModule/python/mix_flat_0_10_cfi.py ...
Sorry: IndentationError: ('unexpected indent', ('src/SimGeneral/MixingModule/python/mix_flat_0_10_cfi.py', 49, 4, '    mixObjects = cms.PSet(theMixObjects)\n'))
gmake[1]: **\* [CompilePython] Error 1
gmake[1]: Leaving directory `/build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_7_0_X-slc6_amd64_gcc481/CMSSW_7_0_X_2014-06-03-0200'
gmake: **\* [src] Error 2
gmake: **\* [There are compilation/build errors. Please see the detail log above.] Error 2


you can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4102/187/summary.html

[cmsbuild]

Pull request #4102 was updated. @civanch, @nclopezo, @vlimant, @mdhildreth, @cmsbuild, @franzoni, @Degano, @davidlange6 can you please check and sign again.

[cmsbuild]

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

[tomcornelis]

Could this pull request be signed, so that it could go into the next CMSSW_7_0_X release? We need this PU scenario soon.

[srimanob]

@davidlange6
I forgot to ask this in the ORP. The additional PU scenario is needed, so we need to have it in 7_0_6.

Thanks,
Phat

[davidlange6]

trivial - so bypassing..