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

[ROOT6] Revert change for dd4hep ROOT master until fixed #5593

Merged
merged 1 commit into from
Feb 24, 2020

Conversation

mrodozov
Copy link
Contributor

please test
for some reason in ROOT6 and ROOT620 branches DD4 cmake picks system python3 instead of the python2 given by the recipe

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 24, 2020

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/4835/console Started: 2020/02/24 13:03

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @mrodozov (Mircho Rodozov) for branch IB/CMSSW_11_1_X/rootmaster.

@cmsbuild, @smuzaffar, @mrodozov, @tulamor can you please review it and eventually sign? Thanks.
cms-bot commands are listed here

@mrodozov
Copy link
Contributor Author

merging this to restart the IBs , dd4 bulilds fine with the old dd4 version

@mrodozov mrodozov merged commit 57acd3a into cms-sw:IB/CMSSW_11_1_X/rootmaster Feb 24, 2020
@cmsbuild
Copy link
Contributor

-1

Tested at: efab0c8

CMSSW: CMSSW_11_1_ROOT6_X_2020-02-21-2300
SCRAM_ARCH: slc7_amd64_gcc820
You can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8ab6ad/4835/summary.html

I found follow errors while testing this PR

Failed tests: UnitTests

  • Unit Tests:

I found errors in the following unit tests:

---> test testDD4hepFilteredView had ERRORS
---> test DetectorDescriptionDDCMSTestDriver had ERRORS

@cmsbuild
Copy link
Contributor

Comparison job queued.

@mrodozov
Copy link
Contributor Author

right, this commit didn't worked as well :/

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.

2 participants