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

[BUG] --strip-cwd-prefix docs out of date #1243

Closed
1 task done
PatrickF1 opened this issue Jan 15, 2023 · 1 comment · Fixed by #1445
Closed
1 task done

[BUG] --strip-cwd-prefix docs out of date #1243

PatrickF1 opened this issue Jan 15, 2023 · 1 comment · Fixed by #1445

Comments

@PatrickF1
Copy link

Checks

  • I have read the troubleshooting section and still think this is a bug.

Describe the bug you encountered:

man fd's docs for --strip-cwd-prefix says

By default, relative paths are prefixed with './' when the output goes to a non interactive terminal (TTY). Use this flag to disable this behaviour.

Actually, 8.5.0 removed that behavior: No leading ./ prefix for non-interactive results.

Describe what you expected to happen:

The docs for that option should say

Relative paths are prefixed with './' when --exec/-x, --exec-batch/-X, or --print0/-0 are used. Use this flag to disable this behaviour.

What version of fd are you using?

8.6.0

Which operating system / distribution are you on?

Darwin 22.2.0 arm64
@sharkdp
Copy link
Owner

sharkdp commented Jan 15, 2023

Thank you. This should be fixed.

tmccombs added a commit to tmccombs/fd that referenced this issue Dec 6, 2023
tmccombs added a commit to tmccombs/fd that referenced this issue Dec 9, 2023
tmccombs added a commit to tmccombs/fd that referenced this issue Dec 11, 2023
tmccombs added a commit to tmccombs/fd that referenced this issue Apr 28, 2024
tmccombs added a commit to tmccombs/fd that referenced this issue May 5, 2024
tmccombs added a commit to tmccombs/fd that referenced this issue May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants