[PR #3681] [MERGED] Support unicode characters in model path #16518

Closed
opened 2026-04-16 05:33:18 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/3681
Author: @jmorganca
Created: 4/16/2024
Status: Merged
Merged: 4/16/2024
Merged by: @jmorganca

Base: mainHead: jmorganca/wmain


📝 Commits (3)

📊 Changes

1 file changed (+31 additions, -2 deletions)

View changed files

📝 llm/ext_server/server.cpp (+31 -2)

📄 Description

When running the c++ subprocess, unicode characters in file names were not being parsed correctly, resulting in an error. This changes server.cpp to use wmain to receive the wide characters and converts them first.

Closes #3273
Fixes #2888
Fixes #3120


🔄 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/3681 **Author:** [@jmorganca](https://github.com/jmorganca) **Created:** 4/16/2024 **Status:** ✅ Merged **Merged:** 4/16/2024 **Merged by:** [@jmorganca](https://github.com/jmorganca) **Base:** `main` ← **Head:** `jmorganca/wmain` --- ### 📝 Commits (3) - [`6e7e1e1`](https://github.com/ollama/ollama/commit/6e7e1e110916dc34f94bcaa611223b0217a663fd) parse wide argv characters on windows - [`da5bf29`](https://github.com/ollama/ollama/commit/da5bf2947562b00d875324b5332c3b5562f00c90) cleanup - [`e6290cd`](https://github.com/ollama/ollama/commit/e6290cdcc9833aca13f15ac3dbcf2fd7c724e7af) move cleanup to end of `main` ### 📊 Changes **1 file changed** (+31 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `llm/ext_server/server.cpp` (+31 -2) </details> ### 📄 Description When running the c++ subprocess, unicode characters in file names were not being parsed correctly, resulting in an error. This changes `server.cpp` to use `wmain` to receive the wide characters and converts them first. Closes #3273 Fixes #2888 Fixes #3120 --- <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 05:33:18 -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#16518