[proposal][git] Remove the dugite dependency

[kittaakos]

• We do not ship the executables anyway.

  • Yes, one can argue that the Git executable download script could be useful. I would recommend setting up Git once correctly on the environment, instead of downloading it per Theia backends. We should disable the Git UI is Git is not on the PATH.

• No i18n support for the Git error parsing.

  • We cannot pars the liegt außerhalb des Repositories error message with the fatal: .+: '(.+)' is outside repository regular expression. It means, we do not handle the expected errors correctly. We would be complete if each git command would be prefixed with LC_ALL=C. (cc: @AlexTugarev)

• No wildcard expansion support: [git] "Decorate" files ignored by Git  #1564.

  • We cannot do git check-ignore * because the wildcards are not expanded by dugite.

• We could get rid of the env variables for

  • --no-optional-locks, and
  • the external Git path.

Thoughts?

---

Related: #4400

[gorkem]

What exactly is the proposal here?

[svenefftinge]

What exactly is the proposal here?

to remove the dugite library completely.

[sunix]

What exactly is the proposal here?

to remove the dugite library completely.

Just dugite or the entire the Theia git extension ? (to be replaced with the VSCode extension)

[svenefftinge]

This issue is just about removing the dugite library

[sunix]

We would need to replace it with something else, right ?

[gorkem]

@svenefftinge Is there a second proposal to add something else that I am missing?

[kittaakos]

It seems we can live with dugite. I am closing this, please reopen if required.