-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Fix Dependency violation in muon geometry builders #32256
Conversation
@cms-sw/externals-l2 actually I'm not sure why we need |
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-32256/20022
|
The tests are being triggered in jenkins.
|
A new Pull Request was created by @silviodonato (Silvio Donato) for master. It involves the following packages: Geometry/CSCGeometryBuilder @civanch, @Dr15Jones, @makortel, @cvuosalo, @ianna, @mdhildreth, @cmsbuild, @kpedro88 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
+1 |
Comparison job queued. |
I'm a bit confused how exactly moving the I'm also confused what exactly is being violated here. |
Thanks @makortel , the origin of the dependency is here
It was introduced with #29111. @slomeo would it be possible to remove the dependency on Fireworks in CSCGeometryValidate.cc ?
I'm confused as well, it might be just a bug of |
Comparison is ready Comparison Summary:
|
PR description:
This PR solves the following dependency errors:
See https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/slc7_amd64_gcc820/CMSSW_11_2_X_2020-11-23-2300/depViolationLogs/Geometry/CSCGeometryBuilder
PR validation:
I tested
ReleaseDepsChecks.pl
and the errors disappeared after the fix .