domtronn / spaceline-all-the-icons.el

A Spaceline Mode Line theme using All The Icons for Emacs
MIT License
235 stars 25 forks source link

error during redisplay: void-variable mode-line #100

Closed Vonfry closed 5 years ago

Vonfry commented 5 years ago

Recently, spaceline-all-the-icons cannot work for me. I have tried to track why it is, but failed.

If I disable this package, my emacs can work correctly. I have tried to test only with spaceline and spaceline-all-the-icons by only loading the two packages with emacs -q, and the issue is same.

output by spaceline-all-the-icons--debug-segments

Error during redisplay: (eval (spaceline-ml-all-the-icons)) signaled (void-variable mode-line) [62 times]
spaceline-all-the-icons--debug-segments:  Errors found in Spaceline Segments: [battery-status, buffer-id, buffer-path, buffer-size, eyebrowse-workspace, flycheck-status-info, flycheck-status, git-status, hud, mode-icon, modified, multiple-cursors, narrowed, neotree-close-bracket, neotree-context, neotree-dirs, neotree-files, neotree-index, neotree-open-bracket, nyan-cat, org-clock-current-task, package-updates, paradox-filter, paradox-line-count, paradox-status-installed, paradox-status-new, paradox-status-upgrade, paradox-total, position, process, projectile, region-info, separator-left-active-1, separator-left-active-2, separator-left-active-3, separator-left-active-4, separator-left-extra-1, separator-left-extra-2, separator-left-inactive, separator-minor-mode-left, separator-minor-mode-right, separator-paradox-1, separator-paradox-2, separator-paradox-3, separator-paradox-4, separator-right-active-1, separator-right-active-2, separator-right-inactive, sunrise, sunset, temperature, time, vc-icon, vc-status, weather, which-function, window-number]
flamingbear commented 5 years ago

I'm also having this problem, it started occurring with the latest spaceline release spaceline-20181223.2024. You can work around this by pinning spaceline-2.0.1 melpa-stable.

The problem I'm seeing is that the modeline appears blank and the error Error during redisplay: (eval (spaceline-ml-all-the-icons)) signaled (void-variable mode-line) repeats.

Here is a minimal init.el that will reproduce the error:

(require 'package)
(setq package-enable-at-startup nil)
(add-to-list 'package-archives '("melpa" . "http://melpa.org/packages/"))
(add-to-list 'package-archives '("melpa-stable" . "http://stable.melpa.org/packages/"))
(package-initialize)

(unless (package-installed-p 'use-package)
  (package-refresh-contents)
  (package-install 'use-package))

(use-package all-the-icons :ensure t)
(use-package spaceline :ensure t)
(use-package spaceline-all-the-icons
  :ensure t
  :after spaceline
  :config
  (spaceline-all-the-icons-theme))

Installed emacs 26.1 mac, packages installed from above:

all-the-icons-20180125.1557
archives
bind-key-20180513.430
dash-20180910.1856
gnupg
memoize-20180614.1930
powerline-20180322.248
s-20180406.808
spaceline-20181223.2024
spaceline-all-the-icons-20170829.820
use-package-20181119.2350
ekaschalk commented 5 years ago

Quoting the faces after each :face in the calls to spaceline-compile in spaceline-all-the-icons.el was a sufficient fix for me, besides the highlight-face and default-face.

See this commit here: https://github.com/TheBB/spaceline/commit/ddd4907e4775dd0aa242936f1c8bfbb6bd1afae4.

commandodev commented 5 years ago

This was how I fixed it: https://github.com/syl20bnr/spacemacs/issues/11784#issuecomment-452633754

Hopefully once the fix you mention is deployed I can roll my changes back.

hexmode commented 5 years ago

This issue should be re-opened until a fix is actually merged. I have a fix that doesn't involve pinning that I'm about to create a pull request for.

mjgiarlo commented 5 years ago

@hexmode are you still planning to submit a PR for this? @Vonfry @domtronn are you inclined to consider said PR?

Thanks, all.

hexmode commented 5 years ago

@mjgiarlo thanks for the nudge. I thought I had a pull request already. I think @domtronn has to merge it, though, not @Vonfry.