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

Support ripgrep's --path-separator option #428

Closed
mookid opened this issue Apr 10, 2019 · 6 comments
Closed

Support ripgrep's --path-separator option #428

mookid opened this issue Apr 10, 2019 · 6 comments

Comments

@mookid
Copy link
Contributor

mookid commented Apr 10, 2019

On windows (both cygwin and cmd.exe shells) the output is with backslashes as path separator. However, one sometimes would want to interop with windows port of unix tools and get slash-separated paths.

When using ripgrep, one can override the path separator with the --path-separator option. I think it would be great to support it in fd as well.

This should also allow working around issues like #153.

reference: the ripgrep discussion BurntSushi/ripgrep#275

@mookid mookid mentioned this issue Apr 10, 2019
3 tasks
@mookid mookid changed the title Support ripgrep's --file-separator option Support ripgrep's --path-separator option Apr 10, 2019
@vvuk
Copy link

vvuk commented Aug 7, 2019

What happened to this in #429? I see that PR got closed without comment.

@mookid
Copy link
Contributor Author

mookid commented Aug 9, 2019

I did not see a great interest for it. I would be happy to see this merged, but I don't want to keep pinging.

When I need the feature, I use my fork.

@sharkdp
Copy link
Owner

sharkdp commented Aug 10, 2019

I think there is some interest for it (see also the discussion in #153).

If you closed the PR because there was no response from my side, let me assure you that it was only a lack of time, not lack of interest 😄

@sharkdp
Copy link
Owner

sharkdp commented Sep 15, 2019

Fixed in #429 by @mookid

@rajeshduggal
Copy link

Great! When can we expect to see a windows release which includes this? https://github.com/sharkdp/fd/releases

@sharkdp
Copy link
Owner

sharkdp commented Sep 15, 2019

Released in v7.4.0.

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

No branches or pull requests

4 participants