jesseduffield / jesseduffield/lazygit

read-only config conflicts with pop-up dismissal

Open
#818 4 comments 0 reactions 0 assignees View on GitHub
bug hacktoberfest
Dominant language
Go
Stars
82.4k
Forks
3k
Avg merge
2d 18h
Merged PRs (30d)
19

Description

**Describe the bug**
I version my dotfiles with nix and home-manager, which puts immutable versions of all my config files into the right places for me. I've seen a lot of benefits from doing this, like being able to set my MBP up again in record time after Apple wipes it every time they have to replace the keyboard 🙄

Anyway, it turns out that this doesn't work for lazygit because it expects to be able to write the config file.

**To Reproduce**
Make the config file read-only (`chmod -w config.yml` oughta do it) then try and dismiss the tracking or news popups.

**Expected behavior**
The popups are dismissed and I can use `lazygit` normally.

**Actual behavior**

```
2020/04/27 14:48:52 An error occurred! Please create an issue at https://github.com/jesseduffield/lazygit/issues

*os.PathError open /Users/brianhicks/Library/Application Support/jesseduffield/lazygit/config.yml: permission denied
/private/tmp/nix-build-lazygit-0.18.drv-0/go/src/github.com/jesseduffield/lazygit/main.go:74 (0x16a1079)
/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-go-1.14/share/go/src/runtime/proc.go:203 (0x1036c52)
/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-go-1.14/share/go/src/runtime/asm_amd64.s:1373 (0x1064631)
```

**Desktop (please complete the following information):**
- OS: Mac
- Lazygit Version: **sigh** looks like that's not in the Nix version. 😦 I'm building from nixpkgs `20.09pre221814.10100a97c89`

**Other Information**
I can get around this by setting `reporting` and `startupPopupVersion` in my config, which I've done, but I'm guessing that this'll come up everytime that version needs to increment.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.