Skip to content

fix(compose): disable MSYS path conversion in run.sh - #4785

Open
tonbistudio wants to merge 1 commit into
block:mainfrom
tonbistudio:fix/run-sh-msys-pathconv
Open

fix(compose): disable MSYS path conversion in run.sh#4785
tonbistudio wants to merge 1 commit into
block:mainfrom
tonbistudio:fix/run-sh-msys-pathconv

Conversation

@tonbistudio

Copy link
Copy Markdown

Problem

Under Git Bash on Windows, MSYS rewrites absolute container paths in command arguments before docker receives them:

$ ./run.sh add-member <pubkey>
OCI runtime exec failed: exec failed: unable to start container process:
exec: "C:/Program Files/Git/usr/local/bin/buzz-admin":
stat C:/Program Files/Git/usr/local/bin/buzz-admin: no such file or directory

/usr/local/bin/buzz-admin was converted to a Windows Git-installation path inside the container. This breaks add-member, remove-member, and list-members — the documented way to populate a closed relay — on the platform most likely to be running a first local deployment (Windows users must already use Git Bash to run run.sh at all).

Fix

export MSYS_NO_PATHCONV=1 at the top of run.sh. The variable is only interpreted by Git-for-Windows' MSYS layer; it is a no-op in every other shell, so macOS/Linux behavior is unchanged.

Testing

  • Reproduced the failure in Git Bash on Windows 11; with MSYS_NO_PATHCONV=1 exported, add-member/list-members work against a live compose stack.
  • Not re-tested on macOS/Linux; the variable is only read by Git for Windows' MSYS layer, so non-MSYS shells ignore it.

🤖 Generated with Claude Code

Under Git Bash on Windows, MSYS rewrites absolute container paths in
command arguments before docker receives them: `docker compose exec
relay /usr/local/bin/buzz-admin` becomes `C:/Program Files/Git/usr/
local/bin/buzz-admin` inside the container and fails with "OCI runtime
exec failed: no such file or directory". This breaks add-member,
remove-member, and list-members - the documented way to populate a
closed relay - on the platform most likely to be running a first local
deployment.

Export MSYS_NO_PATHCONV=1 at the top of the script; it is a no-op in
every non-MSYS shell.

Co-Authored-By: Claude Fable 5 <[email protected]>
Signed-off-by: ntombisol <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant