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

Copying commit hash crashes Lazygit #1114

Open
MuratUrsavas opened this issue Dec 14, 2020 · 9 comments
Open

Copying commit hash crashes Lazygit #1114

MuratUrsavas opened this issue Dec 14, 2020 · 9 comments
Labels
bug Something isn't working hacktoberfest

Comments

@MuratUrsavas
Copy link

MuratUrsavas commented Dec 14, 2020

I've tried to copy the hash from the reflog window via "CTRL + o" but got this message:

2020/12/14 14:21:13 An error occurred! Please create an issue at https://github.com/jesseduffield/lazygit/issues

*errors.errorString No clipboard utilities available. Please install xsel, xclip, wl-clipboard or Termux:API add-on for termux-clipboard-get/set.
github.com/jesseduffield/lazygit/main.go:133 (0xa09eb3)
runtime/proc.go:203 (0x43149e)
runtime/asm_amd64.s:1357 (0x45cec1)

Got fixed after installing xsel. I guess xsel or xclip should be a dependency.

Environment:

  • OS: Ubuntu 20.04
  • Lazygit Version : build date=2020-10-14, build source=debian, version=0.23.6, os=linux, arch=amd64
@MuratUrsavas MuratUrsavas added the bug Something isn't working label Dec 14, 2020
@MuratUrsavas MuratUrsavas changed the title Copying commit has crashes Lazygit Copying commit hash crashes Lazygit Dec 14, 2020
@jesseduffield
Copy link
Owner

I think that error message is appropriate, but we should have it appear in an error panel rather than have it crash the app. This should be simple enough to implement so I'll put a hacktoberfest label on it.

@MuratUrsavas
Copy link
Author

I think adding 'xsel' to the dependencies could be a good idea. It would get rid of this error in the first place.

@jesseduffield
Copy link
Owner

when you say dependencies are you referring specifically to the ubuntu installation method? I am not super across how that's set up :/

@MuratUrsavas
Copy link
Author

Yes, exactly. It's done in the deb configuration. Not a big deal. I've learned it recently but need some refresh as some time passed since. Will check it out and return back.

@MuratUrsavas
Copy link
Author

Got back: You just need to add xsel into the control file of debian package into the "Build-Depends:" section. This way apt would know it should install xsel before lazygit.

@jesseduffield
Copy link
Owner

Now we simply need to work out how to access that control file haha

@MuratUrsavas
Copy link
Author

:)

Who's running the launchpad PPA?

@jesseduffield
Copy link
Owner

I believe it was @dawidd6 at some point. I should really just get a registry of people in charge of each installation method

@Tej12345
Copy link

Also if you get status -1 when trying to copy commit hash via ssh.
use -X flag when connecting to your server.

ssh -X my_ssh_server

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working hacktoberfest
Projects
None yet
Development

No branches or pull requests

3 participants