-
Notifications
You must be signed in to change notification settings - Fork 183
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 conifer version #8786
Update conifer version #8786
Conversation
A new Pull Request was created by @thesps (Sioni Summers) for branch IB/CMSSW_13_3_X/master. @iarspider, @cmsbuild, @smuzaffar, @aandvalenzuela can you please review it and eventually sign? Thanks. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8a4e7a/35441/summary.html Comparison SummarySummary:
|
+externals this looks good now, tests cms-sw/cmssw#43125 passed |
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_13_3_X/master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @antoniovilela, @rappoccio, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2) |
The conifer external was introduced in #8780. Compiling the external with
scram build
yielded errors. These have been fixed upstream in the external and a new version tagged. This PR updates the external spec to the new version.I've tested (compiled and executed) this version with a producer in
CMSSW_13_3_X_2023-10-25-2300
that uses the external.