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

Update DD4Hep to the most recent version #6824

Closed
civanch opened this issue Apr 20, 2021 · 5 comments · Fixed by #6826
Closed

Update DD4Hep to the most recent version #6824

civanch opened this issue Apr 20, 2021 · 5 comments · Fixed by #6826

Comments

@civanch
Copy link
Contributor

civanch commented Apr 20, 2021

In special IB CMSSW_12_0_DD4_X_2021-04-19-2300 it was shown that with the recent PRs for ECAL geometry is fine. We need to update DD4Hep to this version in the master thread first, later in 11_3_X.

@cmsbuild
Copy link
Contributor

A new Issue was created by @civanch Vladimir Ivantchenko.

@Dr15Jones, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@civanch
Copy link
Contributor Author

civanch commented Apr 20, 2021

FYI @bsunanda, @ianna, @mrodozov, this is needed to merge cms-sw/cmssw#33339

@mrodozov
Copy link
Contributor

so you want it in 12_0 and after in 11_3 ?

@civanch
Copy link
Contributor Author

civanch commented Apr 20, 2021

Yes, exactly.

mrodozov added a commit that referenced this issue Apr 20, 2021
mrodozov added a commit that referenced this issue Apr 20, 2021
use latest on master also for 11.3, see #6824
@civanch
Copy link
Contributor Author

civanch commented Apr 21, 2021

@mrodozov , Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants