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

Utilities/StaticAnalyzers: update for Clang 3.7.0 #13161

Merged
merged 1 commit into from Feb 2, 2016
Merged

Utilities/StaticAnalyzers: update for Clang 3.7.0 #13161

merged 1 commit into from Feb 2, 2016

Conversation

davidlt
Copy link
Contributor

@davidlt davidlt commented Feb 2, 2016

The changes are required for Clang 3.7.0 and are not backward compatible
with other revisions.

Tested on CMSSW_8_0_X_2016-01-31-2300 with Clang 3.7.1.

Signed-off-by: David Abdurachmanov David.Abdurachmanov@cern.ch
(cherry picked from commit 047ffea)

Utilities/StaticAnalyzers: update for Clang 3.7.0
7a32558 
The changes are required for Clang 3.7.0 and are not backward compatible
with other revisions.

Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
(cherry picked from commit 047ffea)
@cmsbuild cmsbuild added this to the Next CMSSW_8_0_X milestone Feb 2, 2016
@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 2, 2016

A new Pull Request was created by @davidlt for CMSSW_8_0_X.

It involves the following packages:

Utilities/StaticAnalyzers

@cmsbuild, @smuzaffar, @Dr15Jones, @davidlange6 can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @wddgit this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

davidlt added a commit that referenced this pull request Feb 2, 2016
@davidlt
Merge pull request #13161 from davidlt/Utilities-StaticAnalyzers-clan…
4045497 
…g-3.7

Utilities/StaticAnalyzers: update for Clang 3.7.0
@davidlt davidlt merged commit 4045497 into cms-sw:CMSSW_8_0_X Feb 2, 2016
@smuzaffar
Copy link
Contributor

please test with cms-sw/cmsdist#2119

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 2, 2016

The tests are being triggered in jenkins.
Using externals from cms-sw/cmsdist#2119
https://cmssdt.cern.ch/jenkins/job/cmsdist-cmssw-test-pr/19/console

@cmsbuild cmsbuild mentioned this pull request Feb 2, 2016
Merged
@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 2, 2016

-1

Tested at: 7a32558
I found errors in the following unit tests:

---> test DetectorDescriptionRegressionTestDOMCount had ERRORS
---> test testRecoMETMETProducers had ERRORS
---> test runtestSimCalorimetryHGCalSimProducers had ERRORS
---> test runtestRecoLocalCaloHGCalRecProducers had ERRORS

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

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 2, 2016

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13161/19/summary.html

@slava77 slava77 mentioned this pull request Feb 2, 2016
Merged
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
comparison-available core-pending orp-pending pending-signatures tests-rejected
Projects
None yet
Milestone
CMSSW_8_0_X
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@davidlt @cmsbuild @smuzaffar