Skip to content

Better support for Git Clients#383

Closed
acunniffe wants to merge 3 commits intomainfrom
fix/git-version
Closed

Better support for Git Clients#383
acunniffe wants to merge 3 commits intomainfrom
fix/git-version

Conversation

@acunniffe
Copy link
Copy Markdown
Collaborator

Many devs use Git Clients, which (when being pointed at a git binary often run a --version check and look for libexec

@svarlamov
Copy link
Copy Markdown
Member

➜  testing-git-10 git:(master) ✗ git --exec-path

thread 'main' panicked at src/git/repository.rs:1837:52:
called `Option::unwrap()` on a `None` value
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
➜  testing-git-10 git:(master) ✗ /opt/homebrew/bin/git --exec-path
/opt/homebrew/opt/git/libexec/git-core
➜  testing-git-10 git:(master) ✗

@acunniffe
Copy link
Copy Markdown
Collaborator Author

acunniffe commented Jan 21, 2026

Spoke to Sasha - move symlink logic into the install-hooks, and fix panic.
Verify if Fork uses the exec symlinks before doing a pass-through / alias wrapper

Copy link
Copy Markdown
Member

@svarlamov svarlamov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wdyt of doing the symlink logic in rust in the install flow? Just hoping to keep complexity out of the install scripts themselves. Everything else looks great!

@acunniffe acunniffe closed this Jan 24, 2026
@svarlamov svarlamov deleted the fix/git-version branch April 4, 2026 20:22
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