Skip to content

Conversation

@sigoden
Copy link
Owner

@sigoden sigoden commented Dec 12, 2024

Git bash for windows does not handle slashes correctly, it converts double slashes to single slash for json.

The command to inspect the bug.

PS> & 'C:\Program Files\Git\usr\bin\bash.exe' dump.sh '{"path":"C:\\file"}'
{"path":"C:\file"}

The contect of dump.sh.

printf "%s\n" "$@"

This PR replaces single blackslash to double blackslashs to avoid the bug.

@sigoden sigoden merged commit 7e88e70 into main Dec 12, 2024
3 checks passed
@sigoden sigoden deleted the fix branch December 12, 2024 01:32
matias-ceau pushed a commit to matias-ceau/llm-functions that referenced this pull request Dec 31, 2024
matias-ceau pushed a commit to matias-ceau/llm-functions that referenced this pull request Jan 25, 2025
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.

2 participants