-
Notifications
You must be signed in to change notification settings - Fork 204
Generate cuda.pcm if CUDA is available [15.0.x] #9766
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
Generate cuda.pcm if CUDA is available [15.0.x] #9766
Conversation
|
please test |
|
A new Pull Request was created by @fwyzard for branch IB/CMSSW_15_0_X/master. @cmsbuild, @iarspider, @smuzaffar can you please review it and eventually sign? Thanks.
|
|
cms-bot internal usage |
|
backport #9765 |
|
type bugfix |
|
REMINDER @sextonkennedy, @mandrenguyen, @rappoccio, @antoniovilela: This PR was tested with cms-sw/cmssw#47732, please check if they should be merged together |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c60796/45280/summary.html Comparison SummarySummary:
|
|
hold Because of cms-sw/cmssw#47740 |
|
Pull request has been put on hold by @makortel |
|
@cmsbuild, please test workflow 180.1,181.1,182.1 |
|
-1 Failed Tests: RelVals RelVals |
|
@cmsbuild, please test According to cms-sw/cmssw#47740 (comment) these could work now |
|
REMINDER @sextonkennedy, @antoniovilela, @rappoccio, @mandrenguyen: This PR was tested with cms-sw/cmssw#47732, please check if they should be merged together |
|
-1 Failed Tests: UnitTests Unit TestsI found 2 errors in the following unit tests: ---> test test_DavixReadV_http had ERRORS ---> test test_DavixRead_http had ERRORS Comparison SummarySummary:
|
If CUDA is available, let ROOT generate the
cuda.pcmfile after the build, and include it in the distribution.This fixes the current problem where a missing dictionary for a type that includes a CUDA header leads to a fatal exception.