salt icon indicating copy to clipboard operation
salt copied to clipboard

[master] Stop using egrep

Open Rudd-O opened this issue 5 months ago • 3 comments

What does this PR do?

When egrep is used, a nasty warning from grep saying egrep should be replaced by grep -E appears on the executed command. This leads to failure matching returned values from semanage fcontext.

This fix fixes it.

What issues does this PR fix or reference?

Have not noticed any bugs reported on this.

Previous Behavior

Failure running selinux.fcontext_policy_present.

New Behavior

It works.

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

  • [ ] Docs
  • [ ] Changelog - https://docs.saltproject.io/en/master/topics/development/changelog.html
  • [ ] Tests written/updated

Commits signed with GPG?

No

Please review Salt's Contributing Guide for best practices, including the PR Guidelines.

See GitHub's page on GPG signing for more information about signing commits with GPG.

Rudd-O avatar Aug 26 '24 09:08 Rudd-O