-
Notifications
You must be signed in to change notification settings - Fork 176
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
Updated root to tip of branch v6-14-00-patches (cmsdist master) #5035
Updated root to tip of branch v6-14-00-patches (cmsdist master) #5035
Conversation
please test |
The tests are being triggered in jenkins. |
-1 Tested at: b24dbe4 You can see the results of the tests here: I found follow errors while testing this PR Failed tests: UnitTests
I found errors in the following unit tests: ---> test ExpressionEvaluatorUnitTest had ERRORS |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+externals |
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_11_0_X/gcc700 IBs (but tests are reportedly failing). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
@fabiocos @Dr15Jones, this udpates root 6.14 for normal IBs. This brings in following changes root-project/root@84a781a...5e217f2 Feel free to merge it |
I don't believe any production jobs make use of TTreeFormula so it seems unlikely to break any production. |
@Dr15Jones @smuzaffar within CMSSW this is just used in macros of HLTriggerOffline/Egamma, and within Fireworks https://cmssdt.cern.ch/lxr/source/Fireworks/Core/src/FWTSelectorToEventList.cc#0006 (not clear to me where this is really used). Is the unit test failure reproducible? |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@fabiocos , we can include this too for next IB |
+1 |
No description provided.