Skip to content

Commit

Permalink
Update the containers to populate the HIP libraries to ldconfig.
Browse files Browse the repository at this point in the history
  • Loading branch information
tcojean committed Dec 11, 2019
1 parent 0ac84dc commit 92291c8
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
4 changes: 4 additions & 0 deletions dev_tools/containers/ginkgo-cuda-base.py
Original file line number Diff line number Diff line change
Expand Up @@ -124,3 +124,7 @@
Stage0 += shell(commands=['cd /var/tmp/hipSPARSE', 'mkdir build', 'cd build',
'cmake -DBUILD_CUDA=on ..', 'make install'])
Stage0 += shell(commands=['rm -rf /var/tmp/hipSPARSE'])

# Populate the HIP environment variables
Stage0 += environment(variables={'PATH': '$PATH:/opt/rocm/bin'})
Stage0 += shell(commands=['echo "/opt/rocm/lib" > /etc/ld.so.conf.d/hip.conf && ldconfig'])
2 changes: 1 addition & 1 deletion dev_tools/containers/ginkgo-nocuda-base.py
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@
'fancyvrb multirow hanging adjustbox xkeyval '
'collectbox stackengine etoolbox listofitems ulem '
'wasysym sectsty tocloft newunicodechar caption etoc '
'pgf ec helvetic courier wasy'])
'pgf ec helvetic courier wasy letltxmacro'])

# Copy PAPI libs
add_papi = USERARG.get('papi', 'False')
Expand Down
2 changes: 1 addition & 1 deletion include/ginkgo/core/stop/criterion.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,7 @@ using EnableDefaultCriterionFactory =
* @internal For some abstract reason, nvcc compilation through HIP does not
* properly take into account the `using` declaration to inherit
* constructors. In addition, the default initialization `{}` for
* `_parameters_name##type $parameters` also does not work, which
* `_parameters_name##type parameters` also does not work, which
* means the current form is probably the only correct one.
*
* @ingroup stop
Expand Down

0 comments on commit 92291c8

Please sign in to comment.