From bca814cb7c300b15421be1401bf6a39406b4f946 Mon Sep 17 00:00:00 2001 From: Henrik Lissner Date: Tue, 12 Mar 2019 11:47:52 -0400 Subject: [PATCH] Fix listp error when updating #1235 The async process wasn't mapping through doom-package-outdated-p, only filtered. Also ensure no less than two packages per thread are checked. --- core/autoload/packages.el | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/core/autoload/packages.el b/core/autoload/packages.el index 085daadcc..51d510d72 100644 --- a/core/autoload/packages.el +++ b/core/autoload/packages.el @@ -326,7 +326,7 @@ Used by `doom-packages-update'." collect package)) ;; The bottleneck in this process is quelpa's version checks, so check them ;; asynchronously. - (cl-loop with partitions = (/ (length .quelpa) 4) + (cl-loop with partitions = (min 2 (/ (length .quelpa) 4)) for package-list in (seq-partition .quelpa partitions) do (doom-log "New thread for: %s" package-list) collect @@ -348,7 +348,7 @@ Used by `doom-packages-update'." (load ,(expand-file-name "autoload/packages.el" doom-core-dir)) (require 'package) (require 'quelpa) - (cl-remove-if-not #'doom-package-outdated-p ',package-list)))) + (delq nil (mapcar #'doom-package-outdated-p ',package-list))))) into futures finally return (append (delq nil (mapcar #'doom-package-outdated-p .elpa))