Remove declaration of dupicate dictionary for SMatrix

[Dr15Jones]

The dictionaries for SMatrix<double,2> is already declared in libSMatrix
for ROOT 6 and ROOT gives a warning if it also appears in libDataFormatsMath.

[Dr15Jones]

This should fix some of the unit test failures.

[cmsbuild]

A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_7_4_ROOT6_X.

Remove declaration of dupicate dictionary for SMatrix

It involves the following packages:

DataFormats/Math

@cmsbuild, @cvuosalo, @nclopezo, @slava77 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.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.

[slava77]

this single line was added in #7866 https://github.com/cms-sw/cmssw/pull/7866/files
and it solved a problem there with a missing dictionary.
Where is the other dictionary definition?

[slava77]

is it just the ROOT6, or will this work for ROOT5?

[Dr15Jones]

ROOT 6 declares the dictionary in libSMatrix.so but evidently ROOT 5 did not have it.

[slava77]

Should we have

#if ROOTVERSION>=6 ? or something like that

On 2/25/15 4:28 PM, Chris Jones wrote:

ROOT 6 declares the dictionary in libSMatrix.so but evidently ROOT 5 did
not have it.

—
Reply to this email directly or view it on GitHub
#7955 (comment).

---

Vyacheslav (Slava) Krutelyov
TAMU: Physics Dept Texas A&M MS4242, College Station, TX 77843-4242
CERN: 42-R-027
AIM/Skype: siava16 googleTalk: slava77@gmail.com
(630) 291-5128 Cell (US) +41 76 275 7116 Cell (CERN)

---

[Dr15Jones]

It was submitted only to the ROOT 6 branch

[cvuosalo]

@cmsbuild please test

[cmsbuild]

+1
Tested at: ed4a7fc
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7955/2943/summary.html

[Dr15Jones]

@ktf please merge to fix ROOT6 problems.