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

Commands executed in the terminal fail to honor found Git path #2660

Closed
anzz1 opened this issue May 1, 2023 · 3 comments
Closed

Commands executed in the terminal fail to honor found Git path #2660

anzz1 opened this issue May 1, 2023 · 3 comments
Assignees
Labels
bug Something isn't working verified ✔ Verified
Milestone

Comments

@anzz1
Copy link

anzz1 commented May 1, 2023

Description

  1. On Windows, "git.exe" is not on PATH, yes this is intended.
  2. settings.json has "git.path": "C:\\sys\\git\\bin\\git.exe"
  3. GitLens still tries to execute "git" from PATH

Same as #132

However, this is no problem as there is a gitlens.advanced.path configuration option which lets you set the path to the binary ! Problem solved ?

Oh wait, no . Some rocket scientist thought it ought to be removed, as "it is no longer needed". So this problem was unsolved in the time between that issue and now.

https://newreleases.io/project/github/gitkraken/vscode-gitlens/release/v9.0.0 , bottom of page

GitLens Version

12.2.2

VS Code Version

Version: 1.70.2 (user setup)
Commit: e4503b30fc78200f846c62cf8091b76ff5547662
Date: 2022-08-16T05:35:13.448Z
Electron: 18.3.5
Chromium: 100.0.4896.160
Node.js: 16.13.2
V8: 10.0.139.17-electron.0
OS: Windows_NT x64 6.1.7601

Git Version

No response

Logs, Screenshots, Screen Captures, etc

Tries to use git at PATH:
image

Path setting was removed:
image

image

image

@anzz1 anzz1 added bug Something isn't working triage Needs to be looked at labels May 1, 2023
@eamodio eamodio removed the triage Needs to be looked at label May 1, 2023
@eamodio eamodio self-assigned this May 1, 2023
@eamodio eamodio added this to the 13.6 milestone May 1, 2023
@eamodio eamodio changed the title "Removed gitlens.advanced.git setting as it is no longer needed" Commands executed in the terminal fail to honor found Git path May 1, 2023
@eamodio
Copy link
Member

eamodio commented May 1, 2023

The issue here has nothing to do with gitlens.advanced.path being removed -- because it is NOT needed anymore. It was added before the Git extension had the git.path setting. The real issue here is that commands executed in the terminal are not honoring that location -- and instead relying on Git to be on the path.

@eamodio eamodio closed this as completed in 039d611 May 1, 2023
@eamodio eamodio added pending-release Resolved but not yet released to the stable edition needs-verification Request for verification labels May 1, 2023
@eamodio
Copy link
Member

eamodio commented May 1, 2023

Can you please verify this fix in tomorrow's (v2023.5.02xx) pre-release edition of GitLens? Thank you so much!

You can switch to the pre-release edition of GitLens, by clicking on the "Switch to Pre-Release version of this extension" from the Extensions view.

image

@eamodio eamodio added verified ✔ Verified and removed pending-release Resolved but not yet released to the stable edition needs-verification Request for verification labels May 13, 2023
@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working verified ✔ Verified
Projects
None yet
Development

No branches or pull requests

2 participants