classification
Title: running configure on Solaris 10 gives grep "illegal option" errors
Type: enhancement Stage: patch review
Components: Build Versions: Python 3.10, Python 3.9
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: christian.heimes, martin.wheatley.home, miss-islington, p-ganssle
Priority: normal Keywords: patch

Created on 2021-01-09 12:14 by martin.wheatley.home, last changed 2021-01-12 18:18 by miss-islington.

Pull Requests
URL Status Linked Edit
PR 24200 merged p-ganssle, 2021-01-12 15:04
Messages (6)
msg384720 - (view) Author: Martin Wheatley (martin.wheatley.home) Date: 2021-01-09 12:14
I'm installin Python 3.91. on a Solaris 10 system (I known it's 'old' but I have a legacy installation to support).

Running ./configure the following errors are seen...

checking PROFILE_TASK... -m test --pgo
checking for --with-lto... no
checking for llvm-profdata... no
grep: illegal option -- q
Usage: grep -hblcnsviw pattern file . . .
checking for -Wextra... yes
checking whether /usr/sfw/bin/gcc accepts and needs -fno-strict-aliasing... yes
checking if we can turn off /usr/sfw/bin/gcc unused result warning... no
checking if we can turn off /usr/sfw/bin/gcc unused parameter warning... yes


and


checking aligned memory access is required... yes
checking for --with-hash-algorithm... default
checking for --with-tzpath... grep: illegal option -- q
grep: illegal option -- E
Usage: grep -hblcnsviw pattern file . . .
"/usr/share/zoneinfo:/usr/lib/zoneinfo:/usr/share/lib/zoneinfo:/etc/zoneinfo"
checking for --with-address-sanitizer... no
checking for --with-memory-sanitizer... no
checking for --with-undefined-behavior-sanitizer... no
checking for t_open in -lnsl... yes
checking for socket in -lsocket... yes
msg384811 - (view) Author: Christian Heimes (christian.heimes) * (Python committer) Date: 2021-01-11 12:17
Paul, please take a look. You added the code in commit 62972d9d73e.
msg384951 - (view) Author: Paul Ganssle (p-ganssle) * (Python committer) Date: 2021-01-12 15:11
This particular grep statement is used to validate the `tzpath` variable. Apparently it is easy enough to achieve what I was going for using vanilla grep with no options, so I've created GH-24200 to fix the issue.

I notice that there are other uses of `-q` and `-E` in the configure file, but presumably those are on more optional paths.

@martin.wheatley.home: Can you check to see if GH-24200 fixes your issue?

Also, can you clarify whether this happens with a plain `./configure` invocation, or are you specifying `./configure --with-tzpath='/usr/share/zoneinfo:/usr/lib/zoneinfo:/usr/share/lib/zoneinfo:/etc/zoneinfo'` yourself?
msg384953 - (view) Author: Martin Wheatley (martin.wheatley.home) Date: 2021-01-12 15:28
I'll do the test

it happens with a plain ./configure

Many thanks
Martin

On Tue, 12 Jan 2021 at 15:11, Paul Ganssle <report@bugs.python.org> wrote:

>
> Paul Ganssle <p.ganssle@gmail.com> added the comment:
>
> This particular grep statement is used to validate the `tzpath` variable.
> Apparently it is easy enough to achieve what I was going for using vanilla
> grep with no options, so I've created GH-24200 to fix the issue.
>
> I notice that there are other uses of `-q` and `-E` in the configure file,
> but presumably those are on more optional paths.
>
> @martin.wheatley.home: Can you check to see if GH-24200 fixes your issue?
>
> Also, can you clarify whether this happens with a plain `./configure`
> invocation, or are you specifying `./configure
> --with-tzpath='/usr/share/zoneinfo:/usr/lib/zoneinfo:/usr/share/lib/zoneinfo:/etc/zoneinfo'`
> yourself?
>
> ----------
> versions: +Python 3.10
>
> _______________________________________
> Python tracker <report@bugs.python.org>
> <https://bugs.python.org/issue42874>
> _______________________________________
>
msg384955 - (view) Author: Martin Wheatley (martin.wheatley.home) Date: 2021-01-12 15:31
I'll do the test

it happens with a plain ./configure

Many thanks
Martin

On Tue, 12 Jan 2021 at 15:11, Paul Ganssle <report@bugs.python.org> wrote:

>
> Paul Ganssle <p.ganssle@gmail.com> added the comment:
>
> This particular grep statement is used to validate the `tzpath` variable.
> Apparently it is easy enough to achieve what I was going for using vanilla
> grep with no options, so I've created GH-24200 to fix the issue.
>
> I notice that there are other uses of `-q` and `-E` in the configure file,
> but presumably those are on more optional paths.
>
> @martin.wheatley.home: Can you check to see if GH-24200 fixes your issue?
>
> Also, can you clarify whether this happens with a plain `./configure`
> invocation, or are you specifying `./configure
> --with-tzpath='/usr/share/zoneinfo:/usr/lib/zoneinfo:/usr/share/lib/zoneinfo:/etc/zoneinfo'`
> yourself?
>
> ----------
> versions: +Python 3.10
>
> _______________________________________
> Python tracker <report@bugs.python.org>
> <https://bugs.python.org/issue42874>
> _______________________________________
>
msg384972 - (view) Author: miss-islington (miss-islington) Date: 2021-01-12 18:18
New changeset 0f66498fd8ee8644be6df963b86a1523f6069ddd by Paul Ganssle in branch 'master':
bpo-42874: Remove grep -qE options for Solaris 10 compatibility (GH-24200)
https://github.com/python/cpython/commit/0f66498fd8ee8644be6df963b86a1523f6069ddd
History
Date User Action Args
2021-01-12 18:18:04miss-islingtonsetnosy: + miss-islington
messages: + msg384972
2021-01-12 15:31:38martin.wheatley.homesetmessages: + msg384955
2021-01-12 15:28:54martin.wheatley.homesetmessages: + msg384953
2021-01-12 15:11:12p-gansslesetmessages: + msg384951
versions: + Python 3.10
2021-01-12 15:04:45p-gansslesetkeywords: + patch
stage: patch review
pull_requests: + pull_request23025
2021-01-11 12:17:54christian.heimessetnosy: + christian.heimes, p-ganssle
messages: + msg384811
2021-01-10 18:30:55ned.deilysetcomponents: + Build, - Installation
2021-01-10 18:30:29ned.deilysettitle: configure errors -> running configure on Solaris 10 gives grep "illegal option" errors
2021-01-09 12:14:13martin.wheatley.homecreate