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

Incompatible with (setq vc-display-status nil) #246

Closed
jaredr opened this issue Oct 30, 2019 · 0 comments
Closed

Incompatible with (setq vc-display-status nil) #246

jaredr opened this issue Oct 30, 2019 · 0 comments
Labels
bug Something isn't working

Comments

@jaredr
Copy link

jaredr commented Oct 30, 2019

Was receiving errors like the following:
Error during redisplay: (eval (doom-modeline-segment--vcs)) signaled (args-out-of-range " Git" 5 nil)

It appears that I'd customized the vc-display-status variable to nil, and this causes vc-mode to not include branch information. doom-modeline-update-vcs-text tries to substring the branch name out of vc-mode and ends up out of range.

The existing check against vc-mode (and buffer-file-name) is inadequate because vc-mode isn't nil, it's something like " Git".

Steps to reproduce the behavior:
On emacs 26.3:
eval (setq vc-display-status nil), then save or visit a file under git version control.

@seagle0128 seagle0128 added the bug Something isn't working label Oct 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants