[PR #8125] [MERGED] darwin: restore multiple runners for x86 #12636

Closed
opened 2026-04-13 00:05:25 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/8125
Author: @dhiltgen
Created: 12/17/2024
Status: Merged
Merged: 12/17/2024
Merged by: @jmorganca

Base: mainHead: mac_intel


📝 Commits (1)

  • 98faf35 darwin: restore multiple runners for x86

📊 Changes

4 files changed (+12 additions, -5 deletions)

View changed files

📝 Makefile (+0 -2)
📝 macapp/forge.config.ts (+2 -1)
📝 runners/common.go (+1 -0)
📝 scripts/build_darwin.sh (+9 -2)

📄 Description

In 0.5.2 we simplified packaging to have avx only for macos x86. It looks like there may still be some non-AVX systems out there, so this puts back the prior logic of building no-AVX for the primary binary, and now 2 runners for avx and avx2. These will be packaged in the App bundle only, so the stand-alone binary will now be without AVX support on macos. On arm, we'll also see these runners reported as available in the log, but they're dormant and will never be used at runtime.


🔄 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/8125 **Author:** [@dhiltgen](https://github.com/dhiltgen) **Created:** 12/17/2024 **Status:** ✅ Merged **Merged:** 12/17/2024 **Merged by:** [@jmorganca](https://github.com/jmorganca) **Base:** `main` ← **Head:** `mac_intel` --- ### 📝 Commits (1) - [`98faf35`](https://github.com/ollama/ollama/commit/98faf357611ffc29fc1f2021b1c9eb9a2705558b) darwin: restore multiple runners for x86 ### 📊 Changes **4 files changed** (+12 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `Makefile` (+0 -2) 📝 `macapp/forge.config.ts` (+2 -1) 📝 `runners/common.go` (+1 -0) 📝 `scripts/build_darwin.sh` (+9 -2) </details> ### 📄 Description In 0.5.2 we simplified packaging to have avx only for macos x86. It looks like there may still be some non-AVX systems out there, so this puts back the prior logic of building no-AVX for the primary binary, and now 2 runners for avx and avx2. These will be packaged in the App bundle only, so the stand-alone binary will now be without AVX support on macos. On arm, we'll also see these runners reported as available in the log, but they're dormant and will never be used at runtime. --- <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-13 00:05:25 -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#12636