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

counsel.el (counsel-locate-cmd-es): Match path #1633

Closed
wants to merge 1 commit into from

Conversation

@et2010
Copy link

commented Jun 18, 2018

No description provided.

@basil-conto

This comment has been minimized.

Copy link
Collaborator

commented Jun 18, 2018

Just out of curiosity, because I'm not a Windows user: is the ".exe" suffix required to locate/execute executables?

@et2010

This comment has been minimized.

Copy link
Author

commented Jun 18, 2018

It seems you can execute without the suffix. I didn't notice any differences.

@abo-abo

This comment has been minimized.

Copy link
Owner

commented Jun 18, 2018

Could you please explain what "match path" means? More precisely, what is the change in behavior when adding the -p flag?

@et2010

This comment has been minimized.

Copy link
Author

commented Jun 18, 2018

Sorry for the confusing description. With this flag, the parent folders containing the searched string will also be matched

image

For comparison, here's when this flag was removed:

image

This could be helpful when one forget the exact file name but know where to find the file.

Edit:

Another example: If I want to find a zip file in my Download directory, I could type:

image

@abo-abo abo-abo closed this in 80f05a7 Jun 18, 2018

@abo-abo

This comment has been minimized.

Copy link
Owner

commented Jun 18, 2018

Thanks, I assume here that all users will want -p flag. If there's an objection later on, we can extract the command into a customizable variable.

@et2010 et2010 deleted the et2010:match-path branch Jun 19, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.