Fix critical: initialize _amd_gpu_radeon before case block
_amd_gpu_radeon was only set inside the */rocm*) case arm, so on NVIDIA/CPU/macOS paths where TORCH_INDEX_URL does not contain "rocm", the variable was unbound. With set -u (nounset) enabled, this crashes the installer for every non-AMD user. Move initialization to before the case block so it is always defined.
This commit is contained in:
parent
543e721121
commit
84a9c55e1b
1 changed files with 1 additions and 1 deletions
|
|
@ -1159,9 +1159,9 @@ TORCH_INDEX_URL=$(get_torch_index_url)
|
|||
# Auto-detect GPU for AMD ROCm based
|
||||
# get_torch_index_url must have chosen */rocm*
|
||||
# (gfx in rocminfo or amd-smi list). Then require rocminfo "Marketing Name:.*Radeon".
|
||||
_amd_gpu_radeon=false
|
||||
case "$TORCH_INDEX_URL" in
|
||||
*/rocm*)
|
||||
_amd_gpu_radeon=false
|
||||
if _has_amd_rocm_gpu && command -v rocminfo >/dev/null 2>&1 && \
|
||||
rocminfo 2>/dev/null | grep -q 'Marketing Name:.*Radeon'; then
|
||||
_amd_gpu_radeon=true
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue