[PR #7141] [MERGED] Fix build leakages #17597

Closed
opened 2026-04-16 06:07:58 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/7141
Author: @dhiltgen
Created: 10/8/2024
Status: Merged
Merged: 10/8/2024
Merged by: @dhiltgen

Base: mainHead: build_leaks


📝 Commits (1)

📊 Changes

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

View changed files

📝 llama/Makefile (+1 -1)
📝 llm/generate/gen_common.sh (+1 -9)

📄 Description

The recent change to applying patches leaves the submodule dirty based on "new commits" being present. This ensures we clean up so the tree no longer reports dirty after a go generate ./... run.

The Makefile was being a bit too aggressive in cleaning things up and would result in deleting the placeholder files which someone might accidentally commit.


🔄 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/7141 **Author:** [@dhiltgen](https://github.com/dhiltgen) **Created:** 10/8/2024 **Status:** ✅ Merged **Merged:** 10/8/2024 **Merged by:** [@dhiltgen](https://github.com/dhiltgen) **Base:** `main` ← **Head:** `build_leaks` --- ### 📝 Commits (1) - [`d0708cf`](https://github.com/ollama/ollama/commit/d0708cf8c3d33493e492721ec43f36a07783e816) Fix build leakages ### 📊 Changes **2 files changed** (+2 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `llama/Makefile` (+1 -1) 📝 `llm/generate/gen_common.sh` (+1 -9) </details> ### 📄 Description The recent change to applying patches leaves the submodule dirty based on "new commits" being present. This ensures we clean up so the tree no longer reports dirty after a `go generate ./...` run. The Makefile was being a bit too aggressive in cleaning things up and would result in deleting the placeholder files which someone might accidentally commit. --- <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-16 06:07:58 -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#17597