Bash completions file from modules/tools/contrib not installed into installation prefix

[mhuijgen]

Describe the bug
When installing into a prefix with make DESTDIR=/my/custom/path install the bash completion script will still be copied to the systems root /etc/bash_completion.d instead of $DESTDIR/etc/bash_completion.d
This fails the install because the system is not writable.

To Reproduce
Steps to reproduce the behavior. For example:
Build and then run make install with DESTDIR set to a writeable path.

Expected behavior
The bash script should be installedinto the installation prefix

Output/Screenshots

-- Installing: /etc/bash_completion.d/ifm3d_completions
CMake Error at modules/tools/contrib/cmake_install.cmake:54 (file):
  file INSTALL cannot copy file
  "/home/builder/workspace/ifm-ifm3d/build-debian-bullseye-x86_64/ifm3d-prefix/src/ifm3d-build/modules/tools/ifm3d_completions"
  to "/etc/bash_completion.d/ifm3d_completions": Permission denied.
Call Stack (most recent call first):
  modules/tools/cmake_install.cmake:52 (include)
  cmake_install.cmake:60 (include)

Proposed Solution
Per cmake docs https://cmake.org/cmake/help/latest/command/install.html a relative path will be interpreted relative to the installation prefix.

On line

ifm3d/modules/tools/contrib/CMakeLists.txt

Line 14 in 70971db

DESTINATION /etc/bash_completion.d

the leading / from the DESTINATION path /etc/bash_completion.d should be removed.

[inbangsa]

@mhuijgen Thanks for this, I confirm the issue and will be resolved in next release.

[github-actions]

This issue is stale because it has been open for 30 days with no activity.

[github-actions]

This issue was closed because it has been inactive for 14 days since being marked as stale.