[PR #7305] [MERGED] Fix rocm windows build and clean up dependency gathering #43624

Closed
opened 2026-04-24 23:14:01 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/7305
Author: @dhiltgen
Created: 10/21/2024
Status: Merged
Merged: 10/22/2024
Merged by: @dhiltgen

Base: mainHead: fix_rocm_deps


📝 Commits (1)

  • def21a7 Fix rocm windows build and clean up dependency gathering

📊 Changes

3 files changed (+18 additions, -14 deletions)

View changed files

📝 llama/Dockerfile (+2 -2)
📝 llama/make/Makefile.rocm (+12 -8)
📝 llama/make/gpu.make (+4 -4)

📄 Description

On windows ensure windows version define is properly set for rocm. Remove duplicate rocm arch flags.
Resolve wildcards in the targets so parallel builds don't race. Use readlink to resolve rocm dependencies since wildcards omit libelf. Keep windows rocm deps aligned with unified packaging model

Fixes #7279
Fixes #7320


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/ollama/ollama/pull/7305 **Author:** [@dhiltgen](https://github.com/dhiltgen) **Created:** 10/21/2024 **Status:** ✅ Merged **Merged:** 10/22/2024 **Merged by:** [@dhiltgen](https://github.com/dhiltgen) **Base:** `main` ← **Head:** `fix_rocm_deps` --- ### 📝 Commits (1) - [`def21a7`](https://github.com/ollama/ollama/commit/def21a7e5256f3da35342c4dae9f94f0e54e3aa6) Fix rocm windows build and clean up dependency gathering ### 📊 Changes **3 files changed** (+18 additions, -14 deletions) <details> <summary>View changed files</summary> 📝 `llama/Dockerfile` (+2 -2) 📝 `llama/make/Makefile.rocm` (+12 -8) 📝 `llama/make/gpu.make` (+4 -4) </details> ### 📄 Description On windows ensure windows version define is properly set for rocm. Remove duplicate rocm arch flags. Resolve wildcards in the targets so parallel builds don't race. Use readlink to resolve rocm dependencies since wildcards omit libelf. Keep windows rocm deps aligned with unified packaging model Fixes #7279 Fixes #7320 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-24 23:14:01 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/ollama#43624