Skip to content

Commit

Permalink
updated cmake_platform_opts to always be a list
Browse files Browse the repository at this point in the history
  • Loading branch information
NguyenNhuDi committed Nov 4, 2024
1 parent d6f7ce7 commit 842ffef
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions rmake.py
Original file line number Diff line number Diff line change
Expand Up @@ -114,15 +114,12 @@ def config_cmd():
else:
cmake_executable = "cmake"
toolchain = "toolchain-linux.cmake"
cmake_platform_opts = f"-DROCM_DIR:PATH={rocm_path} -DCPACK_PACKAGING_INSTALL_PREFIX={rocm_path}"
cmake_platform_opts = [f"-DROCM_DIR:PATH={rocm_path}", f"-DCPACK_PACKAGING_INSTALL_PREFIX={rocm_path}"]

tools = f"-DCMAKE_TOOLCHAIN_FILE={toolchain}"
cmake_options.append( tools )

if os.name == 'nt':
cmake_options.extend( cmake_platform_opts)
else:
cmake_options += cmake_platform_opts.split()
cmake_options.extend( cmake_platform_opts)


# build type
Expand Down

0 comments on commit 842ffef

Please sign in to comment.