[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [elpa] master 39d163a: load-dir: fix CL and free variable warnings
From: |
Ted Zlatanov |
Subject: |
Re: [elpa] master 39d163a: load-dir: fix CL and free variable warnings |
Date: |
Mon, 20 Mar 2017 10:00:44 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.0.50 (gnu/linux) |
On Mon, 20 Mar 2017 09:51:07 -0400 Stefan Monnier <address@hidden> wrote:
SM> Either use `some` instead of `cl-some`, or use `cl-lib` (and add
SM> a corresponding `Package-Requires: ((cl-lib "0.5"))` in the file's
SM> header).
BTW, I noticed ELPAs' lmc/lmc.el has "Package-Requires" without the ":"
afterwards.
SM> Stefan "who favors the `cl-lib` solution"
Like this? I'll commit if you agree.
Thanks for the help
Ted
diff --git a/packages/load-dir/load-dir.el b/packages/load-dir/load-dir.el
index ab88ea7..b2231a5 100644
--- a/packages/load-dir/load-dir.el
+++ b/packages/load-dir/load-dir.el
@@ -9,6 +9,7 @@
;; Maintainer: Teodor Zlatanov <address@hidden>
;; Version: 0.0.5
;; Keywords: lisp, files, convenience
+;; Package-Requires: ((cl-lib "0.5"))
;; This file is part of GNU Emacs.
@@ -43,7 +44,7 @@
;;; Code:
-(eval-when-compile (require 'cl-extra))
+(eval-when-compile (require 'cl-lib))
(defgroup load-dir nil
"Automatically load all Emacs Lisp files in given directories."