Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Installation from MELPA doesn't work #1

Closed
gvol opened this issue Dec 2, 2019 · 2 comments
Closed

Installation from MELPA doesn't work #1

gvol opened this issue Dec 2, 2019 · 2 comments

Comments

@gvol
Copy link

gvol commented Dec 2, 2019

I tried to install via MELPA. I didn't previously have counsel installed, and it failed to install it. It seems to depend on the ivy package, but on the swiper file which is (presumably) in the swiper package which was not installed automatically.

On the other hand, the swiper package depends on ivy, so it seems that this should depend on swiper instead. And perhaps the counsel package as well (or maybe the name is wrong)?

Compiling file /Users/iandrus/.emacs.d/elpa/counsel-jq-20191202.846/counsel-jq.el at Mon Dec  2 14:09:55 2019
Entering directory ‘/Users/iandrus/.emacs.d/elpa/counsel-jq-20191202.846/’
counsel-jq.el:12:1:Error: Cannot open load file: No such file or directory, swiper
@To1ne
Copy link

To1ne commented Dec 3, 2019

ivy, swiper, and counsel live in the same codebase: https://github.com/abo-abo/swiper. It's sometimes confusing.

Loading swiper was changed not so long ago: c940636. Although I don't really understand why this change was made. I would suggest to revert it.

@munen munen closed this as completed in b14dfc5 Dec 7, 2019
@munen
Copy link
Contributor

munen commented Dec 7, 2019

Thank you for the issue, @gvol and thank you for the comment @To1ne 🙏

I added swiper as a direct dependency. After the next build of Melpa packages, installation should work without pre-installed Swiper.

I apologize for the mistake. I removed it due to a linter problem that said swiper wasn't installable. However, after a package refresh, it becomes installable..

Thank you again and enjoy using counsel-jq(;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants