Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

credential helper invocation on windows #1111

Closed
wants to merge 3 commits into from
Closed

credential helper invocation on windows #1111

wants to merge 3 commits into from

Conversation

Byron
Copy link
Member

@Byron Byron commented Nov 14, 2023

On prompts, typically only git.exe is available, but no shell (i.e. sh), wo we have to prefer going through git.exe to launch credential helpers.

#1103)

Note that for a moment, there was the idea about using `shell-words`
split certain invocations ourselves to avoid having to use a shell.

This is great on windows, where a shell might not be available.
On linux though this also means that some shell-builtins might
not be not be found if this is used, which limits the approach
to windows, and even there it might reduce compatibility.

Thus, let's not do it.
…-prompts on Windows (#1103)

On prompts, typically only `git.exe` is available, but no shell (i.e. `sh`).
Thus, we have to prefer going through `git.exe` to launch credential helpers and possibly split
simple arguments ourselves.
@Byron Byron closed this Nov 14, 2023
@Byron Byron deleted the fix-1103 branch November 14, 2023 19:34
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.

gix fails to read credential store in ~/.git-credentials
1 participant