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

GNU sed 4.1.5 doesn't have -E #317

Closed
alef opened this issue Feb 20, 2020 · 1 comment
Closed

GNU sed 4.1.5 doesn't have -E #317

alef opened this issue Feb 20, 2020 · 1 comment
Milestone

Comments

@alef
Copy link

alef commented Feb 20, 2020

Replacing with -r or --regexp-extended works for init/bash_completion

@xdelaruelle
Copy link
Member

Thanks for the report. Option -E is used on purpose to achieve portability across OSes, see b68e44e.

I was not aware this -E option was rather recent on GNU sed.

The quick fix: I will add a check at configure step to detect if sed -E is supported on the platform and fallback to sed -r otherwise.

A more sustainable solution: directly use the output from module avail as input for completion scripts without need to call for external commands like sed.

@xdelaruelle xdelaruelle added this to the 4.5 milestone Feb 22, 2020
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

No branches or pull requests

2 participants