search

bbatsov / helm-projectile

Helm UI for Projectile
329 stars 71 forks source link

Helm-projectile-ag broken on Windows? #90

Open angrybacon opened 6 years ago

angrybacon commented 6 years ago

Expected behavior

Should update the list of matches for "foo" in my Helm buffer as I progressively type foo.

Actual behavior

sans titre

Both helm-ag and projectile-ag work as intended however.

Steps to reproduce the problem

(require 'package)
(package-initialize)

Open a project file.

M-x helm-projectile-ag foo

Backtraces if necessary (M-x toggle-debug-on-error)

No error.

Environment & version information

angrybacon commented 6 years ago

Ag had jumped to from v0.29.1.1641 to v2.1.0.1. Went back and helm-projectile-ag now works again Let me know what more information I can provide to help.

choco uninstall ag
choco install ag --version 0.29.1.1641
xiongtx commented 6 years ago

helm-projectile-ag just makes use of helm-ag.

Can you check whether helm-ag w/ the newer ag works for you? I'd expect not. If not, you can open an issue there.

angrybacon commented 6 years ago

@xiongtx Like I said, both helm-ag and projectile-ag work as intended.

xiongtx commented 6 years ago

I cannot reproduce on a Windows 8 machine w/ ag 2.1.0.

Can you upgrade to latest helm-projectile and try again?

angrybacon commented 6 years ago 
choco uninstall ag
choco install ag

ag --version

ag version 2.1.0

Features:
  +jit +lzma +zlib

emacs --version

GNU Emacs 25.3.1
Copyright (C) 2017 Free Software Foundation, Inc.
GNU Emacs comes with ABSOLUTELY NO WARRANTY.
You may redistribute copies of GNU Emacs
under the terms of the GNU General Public License.
For more information about these matters, see the file named COPYING.

emacs -q

M-x emacs-version

GNU Emacs 25.3.1 (x86_64-w64-mingw32) of 2017-09-12
(setq package-load-list '((async t)
                          (epl t)
                          (helm t)
                          (helm-ag t)
                          (helm-core t)
                          (helm-projectile t)
                          (pkg-info t)
                          (popup t)
                          (projectile t)))
(package-initialize)
(let (value)
  (dolist (package '(async
                     epl
                     helm
                     helm-ag
                     helm-core
                     helm-projectile
                     pkg-info
                     popup
                     projectile))
    (message "%s: %s" package (pkg-info-package-version package))))
async: (20180119 533)
epl: (20180205 1249)
helm: (20180319 31)
helm-ag: (20170209 745)
helm-core: (20180318 604)
helm-projectile: (20180319 1414)
pkg-info: (20150517 443)
popup: (20160709 729)
projectile: (20180317 2220)

M-x helm-projectile-ag setq

image

xiongtx commented 6 years ago

Could this be a Windows 10 specific problem?

GNU Emacs 25.3.1 (x86_64-w64-mingw32) of 2017-09-26
$ ag --version
ag version 2.1.0

Features:
  +jit +lzma +zlib
async: (20180119 533)
epl: (20180205 1249)
helm: (20180319 31)
helm-ag: (20170209 745)
helm-core: (20180318 604)
helm-projectile: (20180319 1414)
pkg-info: (20150517 443)
popup: (20160709 729)
projectile: (20180317 2220)

helm-projectile-ag

I don't have Windows 10, so if someone else (@bbatsov?) can reproduce, that'd be great.

@angrybacon If you've Windows 8, can you confirm that this works on Windows 8?

angrybacon commented 6 years ago

@xiongtx Unfortunately, I don't have access to Windows 8 easily.

rgoulter commented 5 years ago

Found this issue through Google. I'm also using Windows 10.

M-x emacs-version

GNU Emacs 26.1 (build 1, x86_64-w64-mingw32) of 2018-05-30

ag --version

ag version 2.1.0

Features:
  +jit +lzma +zlib

helm-ag works. (running M-x helm-ag will prompt for pattern, and helm's buffer will update when updating the pattern)

helm-projectile-ag

~\.emacs.d\straight\repos\helm-projectile $ git rev-parse HEAD
8a2dbc973548fac89356c11d70f7f474ea1367a5

(currently the latest commit on master)

angrybacon's workaround of "install an older version of ag" worked for me.