search

waymondo / apropospriate-theme

A colorful, low-contrast, light & dark theme set for Emacs with a fun name
310 stars 19 forks source link

Error when loading the theme: (void-variable variant) #3

Closed dgutov closed 9 years ago

dgutov commented 9 years ago 
Debugger entered--Lisp error: (void-variable variant)
  (eq variant (quote light))
  (if (eq variant (quote light)) (quote (("#FF5722" . 0) ("#FFA000" . 10) ("#FFD600" . 30) ("#f2f2f2" . 60) ("#FAFAFA" . 80))) (quote (("#F8BBD0" . 0) ("#EC407A" . 10) ("#B388FF" . 30) ("#f2f2f2" . 60) ("#FAFAFA" . 80))))
  eval((if (eq variant (quote light)) (quote (("#FF5722" . 0) ("#FFA000" . 10) ("#FFD600" . 30) ("#f2f2f2" . 60) ("#FAFAFA" . 80))) (quote (("#F8BBD0" . 0) ("#EC407A" . 10) ("#B388FF" . 30) ("#f2f2f2" . 60) ("#FAFAFA" . 80)))))
  custom-theme-recalc-variable(highlight-tail-colors)
waymondo commented 9 years ago

how are you loading the theme? trying to reproduce this.

dgutov commented 9 years ago

Install the theme from MELPA, then, in the same session, M-x load-theme, and pick one of the two variants of this theme.

dgutov commented 9 years ago

Guess I should also mention that I'm using the Emacs master.

waymondo commented 9 years ago

Odd - I just removed the package, restarted, and did M-x package-install then M-x load-theme and chose each version and it loaded correctly. I think I'm on a very recent version of Emacs. When I do C-h C-a I get GNU Emacs 25.0.50.1 (x86_64-apple-darwin14.3.0, NS appkit-1347.57 Version 10.10.3 (Build 14D130a))

If you eval (load-theme 'apropospriate-light t) afterwards does it load?

dgutov commented 9 years ago

If you eval (load-theme 'apropospriate-light t) afterwards does it load?

That makes no difference.

The error is in highlight-tail-colors value expression. If I replace variant with ',variant, that gets rid of the error. Could it be that you're not seeing the error because you don't have highlight-tail installed?

waymondo commented 9 years ago

Yes, you are the man. I have highlight-tail disabled currently but that was definitely the issue. Fixed and pushed.

dgutov commented 9 years ago

Yup, it's fixed. Thanks.