Skip to content

Commit

Permalink
Revert "CMakeLists: Fix libdrm-amdgpu dependencies"
Browse files Browse the repository at this point in the history
This reverts commit 9529bf7.

Reason for revert: Infra still not ready for it yet

Change-Id: I03e043c1ca7924264e3e70e3e82c73b4efc2ae75
  • Loading branch information
kentrussell committed Nov 12, 2021
1 parent 9529bf7 commit e842d7f
Showing 1 changed file with 13 additions and 5 deletions.
18 changes: 13 additions & 5 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -351,13 +351,21 @@ set(CPACK_PACKAGE_VERSION "${PACKAGE_VERSION_STR}")

# Setting devel package dependendent version
set(CPACK_DEBIAN_PACKAGE_DEPENDS "libdrm-dev, rocm-core")
set(CPACK_DEBIAN_PACKAGE_RECOMMENDS "libdrm-amdgpu-amdgpu1")
set(CPACK_DEBIAN_PACKAGE_SUGGESTS "libdrm-amdgpu-amdgpu1")
set(CPACK_DEBIAN_PACKAGE_REPLACES "hsakmt-roct")

# Certain versions of rpm can't handle SUGGESTS or RECOMMENDS
# Also, CPACK_RPM_PACKAGE_RECOMMENDS isn't supported by
# CPack, so just set libdrm-amdgpu as a requirement for RPMs
set(CPACK_RPM_PACKAGE_REQUIRES "libdrm-devel, rocm-core, libdrm-amdgpu")
set(CPACK_RPM_PACKAGE_REQUIRES "libdrm-devel, rocm-core")

# NOTE: CentOS/RHEL <8 and SUSE <10 have no concept of "Suggests"
# so a "suggests" field will just cause rpm installation to fail
# If the HSAKMT_REQUIRES_LIBDRM variable is true in the build
# environment, we use REQUIRES. Otherwise we use SUGGESTS
# For the distros above, this variable should be set to true
if(DEFINED HSAKMT_REQUIRES_LIBDRM)
string ( APPEND CPACK_RPM_PACKAGE_REQUIRES ", libdrm-amdgpu" )
else()
set(CPACK_RPM_PACKAGE_SUGGESTS "libdrm-amdgpu" )
endif()

# Set the names now using CPACK utility
set(CPACK_DEBIAN_FILE_NAME "DEB-DEFAULT")
Expand Down

0 comments on commit e842d7f

Please sign in to comment.