Find rocJPEG - Updates (#129)

kiritigowda · web-flow · commit f8f3f423fdec · 2025-03-13T20:23:22.000-04:00
* CMake Config - updates

* Find rocjpeg - updates

* Build Fix

* CMake Prefix - Updates

* CMakeLists - Prefix updates

* CMakeLists - Remove cmake modules from package

* CMakeLists - Fix typo

* Include file - SRC Location updates
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -82,7 +82,7 @@ message("-- ${BoldBlue}rocJPEG Version -- ${VERSION}${ColourReset}")
 message("-- ${BoldBlue}rocJPEG Install Path -- ${CMAKE_INSTALL_PREFIX}${ColourReset}")
 
 list(APPEND CMAKE_MODULE_PATH ${PROJECT_SOURCE_DIR}/cmake)
-list(APPEND CMAKE_PREFIX_PATH ${ROCM_PATH} ${ROCM_PATH}/hip)
+list(APPEND CMAKE_PREFIX_PATH ${ROCM_PATH}/lib/cmake)
 
 # rocJPEG Default Options
 option(BUILD_WITH_AMD_ADVANCE "Build rocJPEG for advanced AMD GPU Architecture" OFF)
@@ -201,7 +201,7 @@ if(HIP_FOUND AND Libva_FOUND)
   install(TARGETS ${PROJECT_NAME} LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR} COMPONENT dev NAMELINK_ONLY)
   install(TARGETS ${PROJECT_NAME} LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR} COMPONENT asan)
   # install rocJPEG include files -- {ROCM_PATH}/include/rocJPEG
-  install(FILES api/rocjpeg.h
+  install(FILES api/rocjpeg/rocjpeg.h
           DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/${PROJECT_NAME} COMPONENT dev)
   install(FILES ${CMAKE_CURRENT_BINARY_DIR}/rocjpeg_version.h
           DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/${PROJECT_NAME} COMPONENT dev)
@@ -210,13 +210,11 @@ if(HIP_FOUND AND Libva_FOUND)
   DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/${PROJECT_NAME}/amd_detail COMPONENT dev)
 
   # install rocJPEG samples -- {ROCM_PATH}/share/rocJPEG
-  install(DIRECTORY cmake DESTINATION ${CMAKE_INSTALL_DATADIR}/${PROJECT_NAME} COMPONENT dev)
-
   install(FILES samples/jpegDecode/CMakeLists.txt samples/jpegDecode/jpegdecode.cpp samples/jpegDecode/README.md DESTINATION ${CMAKE_INSTALL_DATADIR}/${PROJECT_NAME}/samples/jpegDecode COMPONENT dev)
   install(FILES samples/jpegDecodePerf/CMakeLists.txt samples/jpegDecodePerf/jpegdecodeperf.cpp samples/jpegDecodePerf/README.md DESTINATION ${CMAKE_INSTALL_DATADIR}/${PROJECT_NAME}/samples/jpegDecodePerf COMPONENT dev)
   install(FILES samples/jpegDecodeBatched/CMakeLists.txt samples/jpegDecodeBatched/jpegdecodebatched.cpp samples/jpegDecodeBatched/README.md DESTINATION ${CMAKE_INSTALL_DATADIR}/${PROJECT_NAME}/samples/jpegDecodeBatched COMPONENT dev)
   install(FILES samples/rocjpeg_samples_utils.h DESTINATION ${CMAKE_INSTALL_DATADIR}/${PROJECT_NAME}/samples COMPONENT dev)
-  install(DIRECTORY data/images DESTINATION ${CMAKE_INSTALL_DATADIR}/${PROJECT_NAME}/ COMPONENT dev)
+  install(DIRECTORY data/images DESTINATION ${CMAKE_INSTALL_DATADIR}/${PROJECT_NAME} COMPONENT dev)
   # install license information - {ROCM_PATH}/share/doc/rocJPEG
   set(CPACK_RESOURCE_FILE_LICENSE  "${CMAKE_CURRENT_SOURCE_DIR}/LICENSE")
   install(FILES ${CPACK_RESOURCE_FILE_LICENSE} DESTINATION ${CMAKE_INSTALL_DOCDIR} COMPONENT runtime)
diff --git a/api/amd_detail/rocjpeg_api_trace.h b/api/amd_detail/rocjpeg_api_trace.h
@@ -21,7 +21,7 @@ THE SOFTWARE.
 */
 #pragma once
 
-#include "rocjpeg.h"
+#include "rocjpeg/rocjpeg.h"
 
 // Define version macros for the rocJPEG API dispatch table, specifying the MAJOR and STEP versions.
 //
diff --git a/api/rocjpeg/rocjpeg.h b/api/rocjpeg/rocjpeg.h
diff --git a/cmake/FindrocJPEG.cmake b/cmake/FindrocJPEG.cmake
diff --git a/cmake_modules/rocjpeg-config.cmake.in b/cmake_modules/rocjpeg-config.cmake.in
@@ -1,17 +1,22 @@
 # - Config file for the rocjpeg package
 # It defines the following variables
-#  ROCJPEG_INCLUDE_DIR - include directories for rocjpeg
-#  ROCJPEG_LIB_DIR     - libraries to link against
-#  ROCM_PATH     - Install Base Location for ROCM.
+#  ROCJPEG_INCLUDE_DIR - include directory for rocjpeg
+#  ROCJPEG_LIB_DIR     - library directory for rocjpeg
+#  ROCJPEG_LIBRARY     - library to link against
 
 @PACKAGE_INIT@
 
 # Compute paths
 set_and_check(rocjpeg_INCLUDE_DIR "@PACKAGE_INCLUDE_INSTALL_DIR@")
 set_and_check(ROCJPEG_INCLUDE_DIR "@PACKAGE_INCLUDE_INSTALL_DIR@")
+set_and_check(rocJPEG_INCLUDE_DIR "@PACKAGE_INCLUDE_INSTALL_DIR@")
 set_and_check(rocjpeg_LIB_DIR "@PACKAGE_LIB_INSTALL_DIR@")
 set_and_check(ROCJPEG_LIB_DIR "@PACKAGE_LIB_INSTALL_DIR@")
-set_and_check(ROCM_PATH "${PACKAGE_PREFIX_DIR}")
+set_and_check(rocJPEG_LIB_DIR "@PACKAGE_LIB_INSTALL_DIR@")
+set_and_check(rocjpeg_LIBRARY "@PACKAGE_LIB_INSTALL_DIR@/librocjpeg.so")
+set_and_check(ROCJPEG_LIBRARY "@PACKAGE_LIB_INSTALL_DIR@/librocjpeg.so")
+set_and_check(rocJPEG_LIBRARY "@PACKAGE_LIB_INSTALL_DIR@/librocjpeg.so")
+
 
 get_filename_component(ROCJPEG_CMAKE_DIR "${CMAKE_CURRENT_LIST_FILE}" PATH)
 include("${ROCJPEG_CMAKE_DIR}/rocjpeg-targets.cmake")
diff --git a/docs/doxygen/Doxyfile b/docs/doxygen/Doxyfile
@@ -830,7 +830,7 @@ WARN_LOGFILE           =
 # Note: If this tag is empty the current directory is searched.
 
 INPUT                  = ../reference/index.md \
-                                ../../api/rocjpeg.h
+                                ../../api/rocjpeg/rocjpeg.h
 
 # This tag can be used to specify the character encoding of the source files
 # that doxygen parses. Internally doxygen uses the UTF-8 encoding. Doxygen uses
diff --git a/samples/jpegDecode/CMakeLists.txt b/samples/jpegDecode/CMakeLists.txt
@@ -41,11 +41,10 @@ project(jpegdecode)
 set(CMAKE_CXX_STANDARD 17)
 set(CMAKE_CXX_STANDARD_REQUIRED On)
 
-list(APPEND CMAKE_MODULE_PATH ${PROJECT_SOURCE_DIR}/../../cmake)
-list(APPEND CMAKE_PREFIX_PATH ${ROCM_PATH}/hip ${ROCM_PATH})
+list(APPEND CMAKE_PREFIX_PATH ${ROCM_PATH}/lib/cmake)
 
 find_package(HIP QUIET)
-find_package(rocJPEG QUIET)
+find_package(ROCJPEG QUIET)
 find_package(rocprofiler-register QUIET)
 
 if(HIP_FOUND AND ROCJPEG_FOUND AND rocprofiler-register_FOUND)
diff --git a/samples/jpegDecodeBatched/CMakeLists.txt b/samples/jpegDecodeBatched/CMakeLists.txt
@@ -41,11 +41,10 @@ project(jpegdecodebatched)
 set(CMAKE_CXX_STANDARD 17)
 set(CMAKE_CXX_STANDARD_REQUIRED On)
 
-list(APPEND CMAKE_MODULE_PATH ${PROJECT_SOURCE_DIR}/../../cmake)
-list(APPEND CMAKE_PREFIX_PATH ${ROCM_PATH}/hip ${ROCM_PATH})
+list(APPEND CMAKE_PREFIX_PATH ${ROCM_PATH}/lib/cmake)
 
 find_package(HIP QUIET)
-find_package(rocJPEG QUIET)
+find_package(ROCJPEG QUIET)
 find_package(rocprofiler-register QUIET)
 
 if(HIP_FOUND AND ROCJPEG_FOUND AND rocprofiler-register_FOUND)
diff --git a/samples/jpegDecodePerf/CMakeLists.txt b/samples/jpegDecodePerf/CMakeLists.txt
@@ -41,11 +41,10 @@ project(jpegdecodeperf)
 set(CMAKE_CXX_STANDARD 17)
 set(CMAKE_CXX_STANDARD_REQUIRED On)
 
-list(APPEND CMAKE_MODULE_PATH ${PROJECT_SOURCE_DIR}/../../cmake)
-list(APPEND CMAKE_PREFIX_PATH ${ROCM_PATH}/hip ${ROCM_PATH})
+list(APPEND CMAKE_PREFIX_PATH ${ROCM_PATH}/lib/cmake)
 
 find_package(HIP QUIET)
-find_package(rocJPEG QUIET)
+find_package(ROCJPEG QUIET)
 find_package(rocprofiler-register QUIET)
 
 # threads
diff --git a/samples/rocjpeg_samples_utils.h b/samples/rocjpeg_samples_utils.h
@@ -42,7 +42,7 @@ THE SOFTWARE.
     namespace fs = std::experimental::filesystem;
 #endif
 #include <chrono>
-#include "rocjpeg.h"
+#include "rocjpeg/rocjpeg.h"
 
 #define CHECK_ROCJPEG(call) {                                             \
     RocJpegStatus rocjpeg_status = (call);                                \
diff --git a/src/rocjpeg_decoder.h b/src/rocjpeg_decoder.h
@@ -27,7 +27,7 @@ THE SOFTWARE.
 #include <vector>
 #include <mutex>
 #include <queue>
-#include "../api/rocjpeg.h"
+#include "../api/rocjpeg/rocjpeg.h"
 #include "rocjpeg_api_stream_handle.h"
 #include "rocjpeg_parser.h"
 #include "rocjpeg_commons.h"
diff --git a/src/rocjpeg_vaapi_decoder.h b/src/rocjpeg_vaapi_decoder.h
@@ -43,7 +43,7 @@ THE SOFTWARE.
 #include <va/va_drmcommon.h>
 #include "rocjpeg_commons.h"
 #include "rocjpeg_parser.h"
-#include "../api/rocjpeg.h"
+#include "../api/rocjpeg/rocjpeg.h"
 
 /**
  * @brief Enumeration representing the compute partition for the MI300+ family of GPUs.
diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt
@@ -47,21 +47,32 @@ endif()
 set(CMAKE_CXX_STANDARD 17)
 set(CMAKE_CXX_STANDARD_REQUIRED On)
 
-project(ROCJPEG-test)
+project(rocjpeg-test)
 
 # make test with CTest
 enable_testing()
 include(CTest)
 
-# add find modules
-list(APPEND CMAKE_MODULE_PATH ${ROCM_PATH}/share/rocjpeg/cmake)
+# add ROCm find Config location
+list(APPEND CMAKE_PREFIX_PATH ${ROCM_PATH}/lib/cmake)
 
 # find rocDecode
-find_package(rocJPEG QUIET)
+find_package(ROCJPEG QUIET)
+
+if(ROCJPEG_FOUND)
+  message("-- ${White}${PROJECT_NAME}: rocJPEG found with find_package(ROCJPEG QUIET)${ColourReset}")
+  message("-- \t${White}ROCJPEG_INCLUDE_DIR -- ${ROCJPEG_INCLUDE_DIR}${ColourReset}")
+  message("-- \t${White}ROCJPEG_LIB_DIR -- ${ROCJPEG_LIB_DIR}${ColourReset}")
+  message("-- \t${White}ROCJPEG_LIBRARY -- ${ROCJPEG_LIBRARY}${ColourReset}")
+  message("-- \t${White}ROCJPEG_FOUND -- ${ROCJPEG_FOUND}${ColourReset}")
+  message("-- \t${White}ROCJPEG_VERSION -- ${ROCJPEG_VERSION}${ColourReset}")
+  message("-- \t${White}ROCJPEG_VERSION_MAJOR -- ${ROCJPEG_VERSION_MAJOR}${ColourReset}")
+  message("-- \t${White}ROCJPEG_VERSION_MINOR -- ${ROCJPEG_VERSION_MINOR}${ColourReset}")
+  message("-- \t${White}ROCJPEG_VERSION_PATCH -- ${ROCJPEG_VERSION_PATCH}${ColourReset}")
+else()
+  message("-- ${Yellow}${PROJECT_NAME} requires rocJPEG. Install rocJPEG before running CTests")
+endif(ROCJPEG_FOUND)
 
-if(NOT ROCJPEG_FOUND)
-    message("-- ${Yellow}${PROJECT_NAME} requires rocJPEG. Install rocJPEG before running CTests")
-endif()
 
 add_test(
   NAME
