-
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
[ROCM] avoid creating symlinks #7813
Conversation
A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for branch IB/CMSSW_12_4_X/master. @cmsbuild, @smuzaffar, @aandvalenzuela, @iarspider can you please review it and eventually sign? Thanks. |
FYI @fwyzard, I am merging as last night Ibs failed
|
@smuzaffar do you know why the problem did not come up in the PR tests ? |
because we do not build |
Ah... so the problem is when building the RPM, not when building the actual release. |
logs of IBs are available https://cmssdt.cern.ch/SDT/jenkins-artifacts/build-any-ib/CMSSW_12_4_X_2022-04-21-2300/ |
cmssw-tool-conf was build properly but then cmssw failed with error
|
I am afraid we might have to remove |
I guess we cannot see the build area for the failes IBs ? Looking at the installation from the previous tests of #7795 there is a problem with the ROCm external libraries: the symlinks point to paths that no longer exist... Is it possible that by doing the links in |
Mhm, OK, looks like (outside CMSSW) the compilation still works fine without <environment name="LIBDIR" default="$ROCM_LLVM/lib/linux"/>
<environment name="LIBDIR" default="$ROCM_BASE/llvm/lib"/>
<environment name="INCLUDE" default="$ROCM_LLVM"/> |
See #7814. |
If I run |
let me see if I can use https://github.com/cms-sw/cmsdist/blob/IB/CMSSW_12_4_X/master/scram-project-build.file#L293-L304 logic to relocate symlinks for PR tests |
llvm in rocm does not match llvm from cms externals and this causes cmssw to fail. This change should allow to not create symlinks for rocm libs