-
Notifications
You must be signed in to change notification settings - Fork 184
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
Update DD4Hep to latest on master #6594
Update DD4Hep to latest on master #6594
Conversation
A new Pull Request was created by @mrodozov (Mircho Rodozov) for branch IB/CMSSW_11_3_X/master. @cmsbuild, @smuzaffar, @mrodozov can you please review it and eventually sign? Thanks. |
@cvuosalo I started an IB CMSSW_11_3_X_2021-01-26-0100 that includes the changes from this PR |
@cvuosalo @ianna If you want to build the external and have it locally installed, so you can also patch it you can do:
This takes few hours to build the external and all related to it changes in CMSSW |
@mrodozov @ianna With the special IB, I ran DD4hep tests. Once again, the
|
please test |
@cvuosalo I'm runing the tests to have the logs and search for the missing symbols mentioned yesterday. |
-1 Failed Tests: UnitTests RelVals Unit TestsI found errors in the following unit tests: ---> test GeometryDTGeometryBuilderTestDriver had ERRORS RelVals
|
@cvuosalo there is something mentioned here: |
@mrodozov Markus Frank suggests that the problem with
Are either of these possible? |
The |
I'll see it later when I'm finished with other things to be done would you elaborate which Library-path (to cmssw generated lib, dd4hep lib path etc) and which ROOT map. |
@mrodozov I was just repeating suggestons from Markus. I don't have more to add about them. |
closing in favor of #6612 6612 |
Previous pr was merged to make an IB for debugging a unit test
Deps on tbb removed (not used)