bpo-42874: Remove grep -qE options for Solaris 10 compatibility #24200
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
According to bpo-42874, some versions of grep do not support the
-q
and-E
options. Although both options are used elsewhere in the configure script, this particular bit of validation can be achieved without them,so there's no real harm in using a grep call with no flags.
Would be good to get some people taking advantage of the
--with-tzpath
arguments in the wild to try this out.. Local testing seems to indicate that this does the same thing, but I don't know that we have any buildbots using this option. Maybe @pablogsal?bpo-42874: https://bugs.python.org/issue42874
https://bugs.python.org/issue42874
Automerge-Triggered-By: GH:pablogsal