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

[Bug]: Snap won't work without git installed locally #19

Closed
soumyaDghosh opened this issue Dec 6, 2023 · 4 comments
Closed

[Bug]: Snap won't work without git installed locally #19

soumyaDghosh opened this issue Dec 6, 2023 · 4 comments
Labels
enhancement New feature or request wontfix This will not be worked on

Comments

@soumyaDghosh
Copy link
Member

What happened?

The app will not work unless and until I have installed git in my local machine.

What should have happened?

As a snap package, it should not be depending on local file/libraries for git.

Output of snap info $snap_name

name:      sublime-merge
summary:   Meet a new Git Client, from the makers of Sublime Text
publisher: Snapcrafters✪
store-url: https://snapcraft.io/sublime-merge
contact:   https://github.com/snapcrafters/sublime-merge/issues
license:   unset
description: |
  Meet a new Git Client, from the makers of Sublime Text.
  
  This snap is maintained by the Snapcrafters community, and is not necessarily endorsed or
  officially maintained by the upstream developers.
commands:
  - sublime-merge
snap-id:      tDX7iejihNrsIdo8vfrxUZ3t4x7fO9VT
tracking:     latest/stable
refresh-date: today at 20:25 IST
channels:
  latest/stable:    2091 2023-10-11 (38) 70MB classic
  latest/candidate: ↑                         
  latest/beta:      2091 2023-10-11 (38) 70MB classic
  latest/edge:      2091 2023-12-06 (40) 70MB classic
installed:          2091            (38) 70MB classic

Output of snap connections $snap_name

Classic snap is not showing it

Output of snap version

snap    2.60.4+23.10.1
snapd   2.60.4+23.10.1
series  16
ubuntu  23.10
kernel  6.5.0-13-generic

Relevant log output

No response

Teminal output of app

No response

@soumyaDghosh soumyaDghosh added the bug Something isn't working label Dec 6, 2023
@popey
Copy link
Contributor

popey commented Dec 6, 2023

I'm conflicted about this. While I agree it's helpful to have whatever the application needs, in order to work, nobody has ever reported this as a problem for the last four years, and there's nearly 8K users. Sublime Merge is very much a developer tool, and I'd expect most developers these days have git installed, or it ships by default.

Keen to hear other points of view though.

@merlijn-sebrechts
Copy link
Member

I agree with Popey. Additionally, distributions make sure their git package integrates perfectly into the desktop environment, with keyrings and SSH agents. Making sure our git does the same thing will take considerable work for almost no gain.

@jnsgruk
Copy link
Member

jnsgruk commented Dec 7, 2023

Yeh, I think I agree with @merlijn-sebrechts and @popey here.

@merlijn-sebrechts merlijn-sebrechts added wontfix This will not be worked on enhancement New feature or request and removed bug Something isn't working labels Dec 7, 2023
@merlijn-sebrechts
Copy link
Member

Closing this as I think we've reached agreement that it's better to use host git.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

4 participants