[PR #60] [CLOSED] fix the problem when split env variables with '=' #161

New issue

Closed

opened 2026-10-18 21:12:19 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-18 21:12:19 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/open-webui/mcpo/pull/60
Author: @vannel
Created: 4/10/2025
Status: ❌ Closed

Base: dev ← Head: bugfix/init-with-env

📝 Commits (2)

8c8a35e Merge pull request #42 from open-webui/dev
853e140 fix(init): wrong variable to split

📊 Changes

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

View changed files

📝 src/mcpo/__init__.py (+1 -1)

📄 Description

When starting with the 'env', like uvx mcpo --env SOME_VAR=somevalue -- mycommand, an error will pop out: AttributeError: 'list' object has no attribute 'split'. Maybe that's because in the __init__, the split function is called on the env variable, instead of var variable.

_{🔄 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/open-webui/mcpo/pull/60 **Author:** [@vannel](https://github.com/vannel) **Created:** 4/10/2025 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `bugfix/init-with-env` --- ### 📝 Commits (2) - [`8c8a35e`](https://github.com/open-webui/mcpo/commit/8c8a35e07ebd910ec600a598d80a9601cd4f6861) Merge pull request #42 from open-webui/dev - [`853e140`](https://github.com/open-webui/mcpo/commit/853e14049469cc8757497accae5c2a6e7a86ae12) fix(init): wrong variable to split ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `src/mcpo/__init__.py` (+1 -1) </details> ### 📄 Description When starting with the 'env', like `uvx mcpo --env SOME_VAR=somevalue -- mycommand`, an error will pop out: `AttributeError: 'list' object has no attribute 'split'`. Maybe that's because in the `__init__`, the split function is called on the `env` variable, instead of `var` variable. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-18 21:12:19 +00:00