[PR #3350] [MERGED] Remove need for $VSINSTALLDIR since build will fail if ninja.exe cannot be found #37000

Closed
opened 2026-04-22 21:39:59 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/3350
Author: @jmorganca
Created: 3/25/2024
Status: Merged
Merged: 3/26/2024
Merged by: @jmorganca

Base: mainHead: jmorganca/windows-build


📝 Commits (1)

  • cf2e13e remove need for $VSINSTALLDIR since build will fail if ninja cannot be found

📊 Changes

2 files changed (+10 additions, -15 deletions)

View changed files

📝 docs/development.md (+9 -8)
📝 llm/generate/gen_windows.ps1 (+1 -7)

📄 Description

No description provided


🔄 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/3350 **Author:** [@jmorganca](https://github.com/jmorganca) **Created:** 3/25/2024 **Status:** ✅ Merged **Merged:** 3/26/2024 **Merged by:** [@jmorganca](https://github.com/jmorganca) **Base:** `main` ← **Head:** `jmorganca/windows-build` --- ### 📝 Commits (1) - [`cf2e13e`](https://github.com/ollama/ollama/commit/cf2e13e6d8fe401745b90df393d46ec04f7e8d4e) remove need for `$VSINSTALLDIR` since build will fail if `ninja` cannot be found ### 📊 Changes **2 files changed** (+10 additions, -15 deletions) <details> <summary>View changed files</summary> 📝 `docs/development.md` (+9 -8) 📝 `llm/generate/gen_windows.ps1` (+1 -7) </details> ### 📄 Description _No description provided_ --- <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-22 21:39:59 -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#37000