Skip to content

bpo-38527: fix configure script for Solaris #16845

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Apr 1, 2020
Merged

bpo-38527: fix configure script for Solaris #16845

merged 3 commits into from
Apr 1, 2020

Conversation

wiggin15
Copy link
Contributor

@wiggin15 wiggin15 commented Oct 19, 2019 •
edited by miss-islington
Loading

On Solaris, the regular "grep" command may be an old version that fails to search a binary file. We need to use the correct command (ggrep, in our case), which is found by the configure script earlier.

https://bugs.python.org/issue38527

Automerge-Triggered-By: @pablogsal

@isidentical isidentical requested a review from jcea February 29, 2020 16:33
@isidentical
Copy link
Member

Tested on SunOS gcc-solaris11 5.11 11.3 sun4u sparc SUNW,SPARC-Enterprise, works well.

@inorton
Copy link
Contributor

inorton commented Mar 23, 2020

Hi, I just hit this on Solaris 11.3. the fix looks ok to me. @wiggin15 I think you only need to add the news item

@isidentical isidentical requested a review from pablogsal April 1, 2020 14:05
@isidentical
Copy link
Member

This is a really short change but that is needed in every time I had to compile CPython on solaris so it would be super cool to get this merged.

@miss-islington
Copy link
Contributor

@wiggin15: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit 5dd8360 into python:master Apr 1, 2020
@miss-islington
Copy link
Contributor

Thanks @wiggin15 for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7, 3.8.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Apr 1, 2020
@wiggin15 @miss-islington
bpo-38527: fix configure script for Solaris (pythonGH-16845)
7cd50fe 
On Solaris, the regular "grep" command may be an old version that fails to search a binary file. We need to use the correct command (ggrep, in our case), which is found by the configure script earlier.

Automerge-Triggered-By: @pablogsal
(cherry picked from commit 5dd8360)

Co-authored-by: Arnon Yaari <[email protected]>
@bedevere-bot
Copy link

GH-19279 is a backport of this pull request to the 3.8 branch.

@miss-islington
Copy link
Contributor

Sorry, @wiggin15, I could not cleanly backport this to 3.7 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 5dd836030e0e399b21ab0865ae0d93934bdb3930 3.7

@miss-islington miss-islington self-assigned this Apr 1, 2020
@miss-islington
Copy link
Contributor

Thanks @wiggin15 for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Sorry @wiggin15, I had trouble checking out the 3.7 backport branch.
Please backport using cherry_picker on command line.
cherry_picker 5dd836030e0e399b21ab0865ae0d93934bdb3930 3.7

@pablogsal
Copy link
Member

@wiggin15 @isidentical Could you do the backport to 3.7 using cherry_picker?

miss-islington added a commit that referenced this pull request Apr 1, 2020
@miss-islington @wiggin15
bpo-38527: fix configure script for Solaris (GH-16845)
fc03640 
On Solaris, the regular "grep" command may be an old version that fails to search a binary file. We need to use the correct command (ggrep, in our case), which is found by the configure script earlier.

Automerge-Triggered-By: @pablogsal
(cherry picked from commit 5dd8360)

Co-authored-by: Arnon Yaari <[email protected]>
@wiggin15
Copy link
Contributor Author

wiggin15 commented Apr 1, 2020

@pablogsal Thanks for merging this! The file was added in 3.8. No need to backport.

@pablogsal
Copy link
Member

Great, thanks for the PR!

@fanisk21
Copy link

fanisk21 commented Dec 1, 2022

Hi Guys, why this got lost at bpo-43617: Check autoconf-archive package in configure.ac (GH-25016) ?

@wiggin15
Copy link
Contributor Author

wiggin15 commented Dec 3, 2022

Hi @fanisk21 nice catch. Please comment on the linked issue (GH-25016) about this regression.
I'm afraid I don't have a Solaris machine available to test a fix. Any chance you can open a pull request to re-fix this?
Thanks.

@fanisk21 fanisk21 mentioned this pull request Dec 5, 2022
Merged
DanielTOsborne referenced this pull request Feb 29, 2024
@tiran
[3.9] bpo-43617: Check autoconf-archive package in configure.ac (GH-2…
064bc07 
…5016) (GH-25034)

Signed-off-by: Christian Heimes <[email protected]>.
(cherry picked from commit 5d6e8c1)

Co-authored-by: Christian Heimes <[email protected]>
@DanielTOsborne
Copy link

Oops, I spent time tracking down the same issue, and commented on the commit itself, before checking here for updated comments.
Assuming you'll still accept a PR for this, I can submit one.

I specifically needed to build 3.9 though, so I haven't actually checked the main branch yet if it's still an issue there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pablogsal pablogsal pablogsal approved these changes

@isidentical isidentical isidentical approved these changes

@jcea jcea Awaiting requested review from jcea

Assignees

@miss-islington miss-islington

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@wiggin15 @isidentical @inorton @miss-islington @bedevere-bot @pablogsal @fanisk21 @DanielTOsborne @the-knights-who-say-ni