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

[git][electron] Cannot push to a public repository from the bundled electron application #3571

Closed
kittaakos opened this issue Nov 21, 2018 · 4 comments
Labels
bug bugs found in the application electron issues related to the electron target git issues related to git

Comments

@kittaakos
Copy link
Contributor

I go the following error when I tried to push from the UI.

Request push failed with message: git: 'credential-osxkeychain' is not a git command. See 'git --help'.
fatal: could not read Username for 'https://github.com': Device not configured

I can push from the terminal. Also, the credential.helper looks good:

$ git config --global credential.helper
osxkeychain
@kittaakos kittaakos added bug bugs found in the application git issues related to git electron issues related to the electron target labels Nov 21, 2018
@kittaakos kittaakos changed the title [git] Cannot push to a public repository from the bundled electron application [git][electronn] Cannot push to a public repository from the bundled electron application Nov 21, 2018
@kittaakos kittaakos changed the title [git][electronn] Cannot push to a public repository from the bundled electron application [git][electron] Cannot push to a public repository from the bundled electron application Nov 21, 2018
@svenefftinge
Copy link
Contributor

This error means that the executable could not be found. maybe it should be an absolute path?

@kittaakos
Copy link
Contributor Author

This error means that the executable could not be found

I have to disagree. Although I did not check it thoroughly, it seems the git path was OK, but the command was credential-osxkeychain.

maybe it should be an absolute path?

Absolute path of what?

@svenefftinge
Copy link
Contributor

I suggested that because I had the same error message with just another executable a couple of days ago. But I see now that in this context the problem seems to be the embedded git.
I never really understood why that is used when there is a git on the system. Shouldn't we change dugite so that if there is a git on the PATH it uses it?

@kittaakos
Copy link
Contributor Author

Shouldn't we change dugite so that if there is a git on the PATH it uses it?

Yes. I wanted to suggest the same. And if there is no git on the PATH, we can start Theia with an info message (once) and use the embedded Git.

If we could use the system Git, we could also avoid future confusion in the terminal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application electron issues related to the electron target git issues related to git
Projects
None yet
Development

No branches or pull requests

2 participants