emenel/doomemacs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

fix(vertico): relax doctor tests

Browse source 
consult--grep-lookahead-p throws an error if argv[0] can't be found, and
so will require if consult isn't installed (which would be redundant
with the package checks the doctor already does). To prevent misleading
backtraces here, I've suppressed the latter issue, but the former will
need attention later.
This commit is contained in:
Henrik Lissner 2022-09-16 17:33:44 +02:00
parent 9121100bb4
commit 6bd8cbf903
No known key found for this signature in database
GPG key ID: B60957CA074D39A3
1 changed files with 9 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

17
modules/completion/vertico/doctor.el
View file

@ -1,11 +1,12 @@
;;; completion/vertico/doctor.el -*- lexical-binding: t; -*- ;;; completion/vertico/doctor.el -*- lexical-binding: t; -*-
(require 'consult) (when (require 'consult nil t)
;; FIXME: This throws an error if grep is missing.
(unless (consult--grep-lookahead-p "grep" "-P")
(warn! "The installed grep binary was not built with support for PCRE lookaheads")
(explain! "Some advanced consult filtering features will not work as a result, see the module readme."))
(unless (consult--grep-lookahead-p "grep" "-P") ;; TODO: Move this to core in v3.0
(warn! "The installed grep binary was not built with support for PCRE lookaheads. (unless (consult--grep-lookahead-p "rg" "-P")
Some advanced consult filtering features will not work as a result, see the module readme.")) (warn! "The installed ripgrep binary was not built with support for PCRE lookaheads.")
(explain! "Some advanced consult filtering features will not work as a result, see the module readme.")))
(unless (consult--grep-lookahead-p "rg" "-P")
(warn! "The installed ripgrep binary was not built with support for PCRE lookaheads.
Some advanced consult filtering features will not work as a result, see the module readme."))