Skip to content

add Windows support#2

Open
camalolo wants to merge 1 commit into
raine:mainfrom
camalolo:windows
Open

add Windows support#2
camalolo wants to merge 1 commit into
raine:mainfrom
camalolo:windows

Conversation

@camalolo
Copy link
Copy Markdown

Hello, for your consideration, if you want to provide your tool to windows users. Just used it to perform a multi commit per hunks, no issues arose, but of course you should really test it if this feature interests you. If not, you are welcome to disregard/close this PR. Thanks a lot for the tool you made, I love it !

git-surgeon now compiles and runs on Windows. Key changes:

  • Replace sed -i.bak with _edit-todo self-invocation (edit-mark mode) to eliminate Unix shell dependency in split command
  • Normalize backslash paths to forward slashes in GIT_SEQUENCE_EDITOR for Git for Windows compatibility
  • Guard Unix-only PermissionsExt behind #[cfg(unix)]
  • Add certutil SHA-256 path for checksum verification on Windows
  • Use dirs::cache_dir() instead of hardcoded ~/.cache
  • Add windows-amd64/arm64 platform targets
  • Gate update command on Windows (no release artifacts yet)
  • Document Windows support in README

💘 Generated with Krush

Assisted-by: GLM-5 Turbo via Krush krush@camalolo.com

git-surgeon now compiles and runs on Windows. Key changes:

- Replace sed -i.bak with _edit-todo self-invocation (edit-mark mode)
  to eliminate Unix shell dependency in split command
- Normalize backslash paths to forward slashes in GIT_SEQUENCE_EDITOR
  for Git for Windows compatibility
- Guard Unix-only PermissionsExt behind #[cfg(unix)]
- Add certutil SHA-256 path for checksum verification on Windows
- Use dirs::cache_dir() instead of hardcoded ~/.cache
- Add windows-amd64/arm64 platform targets
- Gate update command on Windows (no release artifacts yet)
- Document Windows support in README

💘 Generated with Krush

Assisted-by: GLM-5 Turbo via Krush <krush@camalolo.com>
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