Emacs-wgrep icon indicating copy to clipboard operation
Emacs-wgrep copied to clipboard

Published 20 hours ago • verified publisher icon mhayashi1120

wgrep-helm.el: Add missing Package-Requires for helm

Open sten0 opened this issue 4 years ago • 3 comments

This package requires helm, because it uses helm-grep-split-line from helm-grep.el. See wgrep-helm.el:L47.

helm-grep.el first appeared in Helm 1.4.0, so I've set that as the required version.

sten0 avatar Feb 04 '20 03:02 sten0

Edit was due to an error in my commit message. Fixed!

sten0 avatar Feb 04 '20 03:02 sten0

Your patch does not cause problem right away although, I don't want cyclic reference for helm to avoid loading problem. helm weakly require wgrep-helm. https://github.com/emacs-helm/helm/blob/master/helm-grep.el#L26

helm is a Master, wgrep-helm is a Slave (means optional package for helm user).

This (declare-function helm-grep-split-line "helm-grep") form is added to suppress elisp byte-compiler warnings.

mhayashi1120 avatar Feb 12 '20 11:02 mhayashi1120

Thank you for the tip. Hm, yes, this sounds tricky... Please keep this PR open while I consult the rest of the Debian Emacsen Team, because I don't use Helm, and I'm not yet sure what the correct resolution is.

'hoping to provide a follow-up soon, Nicholas

sten0 avatar Mar 14 '20 19:03 sten0