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

new external starlight added #2114

Merged
merged 2 commits into from Feb 1, 2016
Merged

new external starlight added #2114

merged 2 commits into from Feb 1, 2016

Conversation

smuzaffar
Copy link
Contributor

same as #2049 with updated tag/hash of starlight

@smuzaffar
Copy link
Contributor Author

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 1, 2016

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/cmsdist-test-pr/83/console

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 1, 2016

A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for branch IB/CMSSW_8_0_X/stable.

@cmsbuild, @smuzaffar, @Degano, @iahmad-khan, @davidlange6 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.

make %{makeprocesses} install VERBOSE=1

rm -rf %{i}/lib/archive
rm -rf %{i}/lib/libStarlib.a
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this line, there is no such file.

a/slc6_amd64_gcc493/external/starlight/r193-cms/lib
a/slc6_amd64_gcc493/external/starlight/r193-cms/lib/libStarlib.so
a/slc6_amd64_gcc493/external/starlight/r193-cms/lib/archive
a/slc6_amd64_gcc493/external/starlight/r193-cms/lib/archive/libStarlib.a

Other than that, LGTM.

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 1, 2016

@smuzaffar
Copy link
Contributor Author

merge
looks good

cmsbuild added a commit that referenced this pull request Feb 1, 2016
@cmsbuild cmsbuild merged commit a700bd5 into cms-sw:IB/CMSSW_8_0_X/stable Feb 1, 2016
cmsbuild added a commit that referenced this pull request Feb 1, 2016
@smuzaffar
Copy link
Contributor Author

@KiSooLee , starlight does not build with gcc 5.3.0. this is breaking gcc530 IBs.

In file included from /build/build/muz/sl1/BUILD/slc6_amd64_gcc530/external/starlight/r193/starlight-r193/src/randomgenerator.cpp:39:0:
/build/build/muz/sl1/slc6_amd64_gcc530/external/clhep/2.2.0.4-ikhhed/include/CLHEP/Random/Randomize.h:70:18: error: 'CLHEP::HepRandomGenActive' defined but not used [-Werror=unused-variable]
 static const int HepRandomGenActive = HepRandom::createInstance();
                  ^
cc1plus: all warnings being treated as errors

@davidlt
Copy link
Contributor

davidlt commented Feb 2, 2016

I will take care of this, there is not much be done done (CXXFLAGS = -Wno-error=unused-variable).

@davidlt
Copy link
Contributor

davidlt commented Feb 2, 2016

Solved.

@smuzaffar
Copy link
Contributor Author

Thanks @davidlt, but I mistakenly merged it. This still needs to be discussed in ORP. So I am removing it from cmssw-tool-conf

@davidlt
Copy link
Contributor

davidlt commented Feb 2, 2016

I wouldn't remove it. It does not affect CMSSW in any way. Wait for the decision in ORP.

@smuzaffar
Copy link
Contributor Author

too late, already done :-)

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

3 participants