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

How does it compare to helm-swoop ? #3

Closed
syl20bnr opened this issue Mar 11, 2015 · 7 comments
Closed

How does it compare to helm-swoop ? #3

syl20bnr opened this issue Mar 11, 2015 · 7 comments
Labels

Comments

@syl20bnr
Copy link

Question in title + what's your vision about the future feature of swiper ?

helm-swoop: https://github.com/ShingoFukuyama/helm-swoop

@abo-abo
Copy link
Owner

abo-abo commented Mar 11, 2015

  • it's more pretty, using my definition of pretty
  • it's more usable, using my definition of usable
  • it's more simple, using my definition of simple

Both packages set out to accomplish the same task, so they can't be too different.
You can use either one as a matter of taste.

The plan is to provide a back end similar to ido eventually. Helm is used initially to get a
reference implementation.

@abo-abo abo-abo closed this as completed Mar 11, 2015
@syl20bnr
Copy link
Author

That's fine with me but I hoped for something more constructive. I don't see much difference in "prettyness" between the 2 packages, faces are customizable.

Do you plan to add editing feature and multi buffer search ?

There is also another implementation of helm-swoop without helm [1] which might be interesting for you.

[1] https://github.com/ShingoFukuyama/emacs-swoop

@abo-abo
Copy link
Owner

abo-abo commented Mar 11, 2015

That's fine with me but I hoped for something more constructive. I don't see much difference in
"prettyness" between the 2 packages, faces are customizable.

Just try completing words separated by spaces with both packages, you'll see the difference: it's
totally unrelated to faces color. The matching is different as well.

Do you plan to add editing feature and multi buffer search ?

I'm not currently interested in that, I just want a re-search-forward replacement.

There is also another implementation of helm-swoop without helm [1] which might be interesting for you.

[1] https://github.com/ShingoFukuyama/emacs-swoop

Thanks, I saw this link earlier today. It looks to me like helm-swoop is a straight upgrade over
emacs-swoop. I'm looking at ido to get something less obtrusive than helm.

@syl20bnr
Copy link
Author

@abo-abo helm-swoop is not an upgrade, emacs-swoop is a rewrite of helm-swoop without helm, but as you see the helm variant is the more popular one, although emacs-swoop has cool unique features.

Reference: emacsorphanage/swoop#3

Since you already have multiple backend I would suggest to put helm in front and then in a second README section you can introduce ivy (which must be fully optional). This is a better strategy for adoption but as all my advices are **** for you... ;-)

@abo-abo
Copy link
Owner

abo-abo commented Mar 14, 2015

Since you already have multiple backend I would suggest to put helm in front and then in a second
README section you can introduce ivy (which must be fully optional).

I want to downplay the helm back end eventually. The only reason why it's the default currently is
that the ivy part of swiper isn't completed yet.

swiper's key point will be unobtrusiveness, and that can only be done with ivy (maybe also ido
with ido-vertical, but it's hard to make ido work with swiper).

Unobtrusiveness is that both helm-swoop and swoop don't have.

This is a better strategy for adoption but as all my advices are **** for you... ;-)

Well, there hasn't been any good ones yet. But keep trying, if you like.

@syl20bnr
Copy link
Author

That's ok with me, I truly think that there is benefit at some point in all advices, be good or bad.

Don't be mistaken, Helm can be unobtrusive, it is just a matter of configuration and in your case for swiper it is just a matter of documentation. There is a great benefit to have a whole configuration consistently relying on Helm instead of several different interfaces, users like consistency a lot. Helm is one of the most powerful package on MELPA, don't downplay it.

@abo-abo
Copy link
Owner

abo-abo commented Mar 15, 2015

Helm is one of the most powerful package on MELPA, don't downplay it.

Helm is only a package for completion, just like ido or ivy. It
may be easy to use, once someone has gone through the trouble of
setting it up, but it's hard to get it to do exactly what you want.
Some people are fine with black boxes, as long as they give them nice
things, I'm not.

abo-abo pushed a commit that referenced this issue Sep 19, 2018
`start-process-shell-command` doesn't seem to wait for
`counsel-locate-action-extern` to finish under Ubuntu. [Gist to
reproduce the
issue](https://gist.github.com/yiufung/38be4c3a6832334f36f2441550cf0b12).

Related
discussion: ([#1](https://askubuntu.com/questions/646631/emacs-doesnot-work-with-xdg-open),
[#2](https://emacs.stackexchange.com/questions/19344/why-does-xdg-open-not-work-in-eshell),
[#3](http://emacs.1067599.n8.nabble.com/emacs-and-xdg-open-td106892.html))
suggests to:

1. Set `process-connection-type` as nil temporarily
2. Called with `setsid -w` to wait for the process to finish.

To avoid interfering with other platforms and applications, I adopted
the latter approach which uses `lsb_release` to check distribution
type with fall back as `xdg-open`. Currently only check for Ubuntu,
and to be extended in the future.

Re #863
Re #1401
Fixes #1759
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants