Skip to content

Fix shell completion on FreeBSD #79

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

Closed

Conversation

bendikro
Copy link
Contributor

@bendikro bendikro commented Nov 1, 2023

sed on FreeBSD does not support '\s', and gives the error: "sed: 1: "/^\s*$/d": RE error: trailing backslash ()"

Fix by using POSIX bracket expression '[[:space:]]'

Fixes #65

sed on FreeBSD does not support '\s', and gives the error:
"sed: 1: "/^\s*$/d": RE error: trailing backslash (\)"

Fix by using POSIX bracket expression '[[:space:]]'
Copy link
Contributor

@dhellmann dhellmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@dhellmann
Copy link
Contributor

I'm not sure why the unit test actions haven't run for this PR. I'm investigating whether I can trigger them myself.

@dhellmann
Copy link
Contributor

I rebased and created a new PR (#86) and that one is running the jobs. If those pass, it will merge automatically, so I'll close this PR. You will still receive credit for the contribution in the release notes.

Thank you for your contribution!

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

Successfully merging this pull request may close these issues.

sed error on FreeBSD due to GNU extensions
2 participants