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
Properly pass strings to Fortran in two generators #37840
Conversation
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37840/29766
|
A new Pull Request was created by @Dr15Jones (Chris Jones) for master. It involves the following packages:
@alberto-sanchez, @SiewYan, @mkirsano, @Saptaparna, @GurpreetSinghChahal can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2ccb46/24496/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Comparison SummarySummary:
|
@smuzaffar , is that a way to trigger test on |
@SiewYan , yes by adding a comment |
please test for el8_aarch64_gcc10 |
test parameters:
|
@SiewYan , our ARM64 node has full disk and we are not able to test/build any aarch64 IBs/PRs. CERN Openlab team is looking in to the issue. |
-1 Failed Tests: UnitTests Unit TestsI found errors in the following unit tests: ---> test test-das-selected-lumis had ERRORS |
workflow 575.0 now passed for aarch64 |
+1 Thanks @smuzaffar! |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
PR validation:
Code compiles and workflow 575.0 runs fine on x86. I do not know what machine to use to test on ARM.
fixes #37799