skeeto / skewer-mode

Live web development in Emacs
The Unlicense
1.09k stars 57 forks source link

Warning: cl package required at runtime #21

Closed pashinin closed 11 years ago

pashinin commented 11 years ago

Just warnings, but can be fixed:

skewer-bower.el:28:1:Warning: cl package required at runtime
skewer-html.el:19:1:Warning: cl package required at runtime
skewer-mode.el:108:1:Warning: cl package required at runtime

Maybe something like this will help:

(eval-when-compile
  (require 'cl))

Also some more warnings:

skewer-bower.el:89:40:Warning: reference to free variable
    `skewer-bower-git-executable'
skewer-bower.el:192:1:Warning: Unused lexical variable `bower'
skewer-bower.el:192:1:Warning: Unused lexical variable `skewer'

Emacs 24.3.50.2

skeeto commented 11 years ago

I just fixed the two unused variable warnings in a829fb7. The free variable warning is a false positive. The compiler just isn't noticing the definition. It's there.

As for "cl required at runtime," that's the strange vendetta the core Emacs devs have against the cl package. That warning carries no real weight. I'm using the sort* function from cl so eval-when-compile isn't an option. The current "solution" to this as of Emacs 24.3 is to use cl-lib instead. However, since it will be a couple of years before most people are updated to 24.3 or higher, and cl will be around much, much longer than that, I'm not doing that right now.