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

Final clang fix. #4096

Merged
merged 1 commit into from Jun 13, 2014
Merged

Final clang fix. #4096

merged 1 commit into from Jun 13, 2014

Conversation

ktf
Copy link
Contributor

@ktf ktf commented Jun 3, 2014

Fixes complain about:

.../xr.cc: error: explicit specialization of 'fgIsA' after instantiation
template<> atomic_TClass_ptr TFWLiteSelector<XXXX>::fgIsA(0);
                                             ^
.../xr.cc:111:9: note: implicit instantiation first required here

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 3, 2014

A new Pull Request was created by @ktf (Giulio Eulisse) for CMSSW_7_2_X.

Final clang fix.

It involves the following packages:

FWCore/TFWLiteSelectorTest
PhysicsTools/ParallelAnalysis

@nclopezo, @monttj, @cmsbuild, @Dr15Jones, @vadler, @Degano, @ktf can you please review it and eventually sign? Thanks.
@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.

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 3, 2014

-1
I found an error when building:

Entering library rule at FWCore/Version
>> Compiling  /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_7_2_X-slc6_amd64_gcc481/CMSSW_7_2_X_2014-06-03-0200/src/FWCore/TFWLiteSelectorTest/src/ThingsTSelector.cc 
>> Compiling  /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_7_2_X-slc6_amd64_gcc481/CMSSW_7_2_X_2014-06-03-0200/src/FWCore/TFWLiteSelectorTest/src/ThingsTSelector2.cc 
>> Building LCG reflex dict from header file src/FWCore/TFWLiteSelectorTest/src/classes.h
In file included from /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_7_2_X-slc6_amd64_gcc481/CMSSW_7_2_X_2014-06-03-0200/src/PhysicsTools/ParallelAnalysis/src/TrackAnalysisAlgorithm.cc:1:0:
/build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_7_2_X-slc6_amd64_gcc481/CMSSW_7_2_X_2014-06-03-0200/src/PhysicsTools/ParallelAnalysis/interface/TrackAnalysisAlgorithm.h:39:12: error: 'atomic_TClass_ptr' does not name a type
 template<> atomic_TClass_ptr TFWLiteSelectorexamples::TrackAnalysisAlgorithm::fgIsA;
            ^
--->> genreflex: INFO: Parsing file src/FWCore/TFWLiteSelectorTest/src/classes.h with GCC_XML OK
--->> genreflex: INFO: Generating Reflex Dictionary
class TFWLiteSelectortfwliteselectortest::ThingsWorker


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

@cmsbuild
Copy link
Contributor

Pull request #4096 was updated. @nclopezo, @monttj, @cmsbuild, @Dr15Jones, @vadler, @Degano, @ktf can you please check and sign again.

@cmsbuild
Copy link
Contributor

@ktf
Copy link
Contributor Author

ktf commented Jun 13, 2014

+1

Bypassing AT signature. Complain if not ok.

ktf added a commit that referenced this pull request Jun 13, 2014
@ktf ktf merged commit 8e2c508 into cms-sw:CMSSW_7_2_X Jun 13, 2014
@ktf ktf deleted the final-clang-fix branch June 13, 2014 20:17
@cmsbuild
Copy link
Contributor

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

2 participants