Skip to content

git: Use -z for null separated parsing - #284

Merged
jerry-skydio merged 1 commit into
mainfrom
jerry/revup/main/fixescaping
Jul 27, 2026
Merged

git: Use -z for null separated parsing#284
jerry-skydio merged 1 commit into
mainfrom
jerry/revup/main/fixescaping

Conversation

@jerry-skydio

Copy link
Copy Markdown
Collaborator

Some git commands using paths will escape unicode characters, which
we then don't handle properly.

Fix this by passing -z so git doesn't escape these, and splitting on nulls.

Add a test

@jerry-skydio

jerry-skydio commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator Author

Reviews in this chain:
#284 git: Use -z for null separated parsing

@jerry-skydio

jerry-skydio commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator Author
# head base diff date summary
0 cb5d235a fa73cdf5 diff Jul 27 13:52 PM 3 files changed, 37 insertions(+), 8 deletions(-)
1 4d94381c 258238bd rebase Jul 27 13:54 PM 0 files changed
2 7ac3bf03 792fbe9f diff Jul 27 13:56 PM 0 files changed

@jerry-skydio
jerry-skydio force-pushed the jerry/revup/main/deletelasttouched branch from fa73cdf to 258238b Compare July 27, 2026 20:54
@jerry-skydio
jerry-skydio force-pushed the jerry/revup/main/fixescaping branch from cb5d235 to 4d94381 Compare July 27, 2026 20:54
Base automatically changed from jerry/revup/main/deletelasttouched to main July 27, 2026 20:56
Some git commands using paths will escape unicode characters, which
we then don't handle properly.

Fix this by passing -z so git doesn't escape these, and splitting on nulls.

Add a test
@jerry-skydio
jerry-skydio force-pushed the jerry/revup/main/fixescaping branch from 4d94381 to 7ac3bf0 Compare July 27, 2026 20:56
@jerry-skydio
jerry-skydio merged commit 74e5742 into main Jul 27, 2026
6 checks passed
@jerry-skydio
jerry-skydio deleted the jerry/revup/main/fixescaping branch July 27, 2026 21:01
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