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
Fc fix ecal al ca reco producer #21958
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-21958/3038 |
A new Pull Request was created by @fabiocos (Fabio Cossutti) for master. It involves the following packages: Calibration/EcalAlCaRecoProducers @ghellwig, @arunhep, @cerminar, @cmsbuild, @franzoni, @lpernie can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
I'm not very happy with this change since it takes away the purpose of FileInPath. We want more use of FileInPath to help with configurations, not less of them. |
@Dr15Jones this is the only case that I see in which FileInPath is used in a dynamic build of a configuration. I agree this is undoing the good of FileInPath, but unless we want to modify the build procedure is to avoid to create dynamically the configuration. Or do you suggest other solutions? |
Comparison job queued. |
Comparison is ready @slava77 comparisons for the following workflows were not done due to missing matrix map:
Comparison Summary:
|
No more needed after the fix of FilInPath in #21966 |
Replace FileInPath with string in the configuration built at compilation time to avoid problems with the build system (see #21955)