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

src: use grep -E instead of egrep #147

Merged
merged 1 commit into from Jan 11, 2023
Merged

src: use grep -E instead of egrep #147

merged 1 commit into from Jan 11, 2023

Conversation

lzaoral
Copy link
Contributor

@lzaoral lzaoral commented Jan 11, 2023

egrep was deprecated in GNU grep 3.8. This change was done using

$ find src -name '*.sh' -exec sed -i 's/egrep/grep -E/g' {} \;

Fixes:

egrep: warning: egrep is obsolescent; using grep -E

Can be seen e.g. at the beginning of the following test log on Fedora Rawhide:
https://artifacts.dev.testing-farm.io/d85b93c1-fffe-44f4-8d53-99896290648f/work-tests.ymle821oxz5/tests-ojftz554/PASS-file-tests-err.log

... which was deprecated in GNU grep 3.8 using
$ find src -name '*.sh' -exec sed -i 's/egrep/grep -E/g' {} \;

Fixes:
egrep: warning: egrep is obsolescent; using grep -E
@hegerj
Copy link
Collaborator

hegerj commented Jan 11, 2023

LGTM

@hegerj hegerj merged commit a707b39 into beakerlib:master Jan 11, 2023
@lzaoral lzaoral deleted the replace-egrep branch January 11, 2023 15:11
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.

None yet

2 participants