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

[3.10] gh-89452: GHA: Set --with-dbmliborder to avoid issues with homebrew's gdbm 1.24 #125112

Merged
merged 1 commit into from
Oct 8, 2024

Conversation

encukou
Copy link
Member

@encukou encukou commented Oct 8, 2024

Per #89452 (comment), the issue is fixed in configure for 3.11+, and

For older Python versions, the workaround is to build with:

./configure --with-dbmliborder=gdbm:ndbm

We need this workaround in GitHub Actions, otherwise the tests fail.

See also: gh-105304

…th homebrew's gdbm 1.24

Per python#89452 (comment),
the issue is fixed in configure for 3.11+, and

> For older Python versions, the workaround is to build with:
>
>     ./configure --with-dbmliborder=gdbm:ndbm

We need this workaround in GitHub Actions, otherwise the tests fail.
@encukou
Copy link
Member Author

encukou commented Oct 8, 2024

@pablogsal @ambv, given #89452 (comment), does this look reasonable to you, as a workaround for a buildsystem/homebrew incompatibility which was fixed better in 3.11+?

If so, I'll also backport to 3.9 and update the Devguide homebrew instructions.

@ambv
Copy link
Contributor

ambv commented Oct 8, 2024

Makes sense to me.

@ambv ambv merged commit 850189a into python:3.10 Oct 8, 2024
15 checks passed
@encukou encukou added the needs backport to 3.9 only security fixes label Oct 8, 2024
@miss-islington-app
Copy link

Thanks @encukou for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

@encukou encukou deleted the mac-dbmliborder-3.10 branch October 8, 2024 14:46
@miss-islington-app
Copy link

Sorry, @encukou and @ambv, I could not cleanly backport this to 3.9 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 850189a64e7f0b920fe48cb12a5da3e648435680 3.9

@encukou encukou assigned encukou and unassigned ambv Oct 8, 2024
encukou added a commit to encukou/cpython that referenced this pull request Oct 9, 2024
…h homebrew's gdbm 1.24 (pythonGH-125112)

Per https://github.com/python/cpython/issues/89452GH-issuecomment-1116329316,
the issue is fixed in configure for 3.11+, and

> For older Python versions, the workaround is to build with:
>
>     ./configure --with-dbmliborder=gdbm:ndbm

We need this workaround in GitHub Actions, otherwise the tests fail.
(cherry picked from commit 850189a)
@bedevere-app
Copy link

bedevere-app bot commented Oct 9, 2024

GH-125176 is a backport of this pull request to the 3.9 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.9 only security fixes label Oct 9, 2024
ambv added a commit that referenced this pull request Oct 9, 2024
…brew's gdbm 1.24 (GH-125112) (#125176)

Per https://github.com/python/cpython/issues/89452GH-issuecomment-1116329316,
the issue is fixed in configure for 3.11+, and

> For older Python versions, the workaround is to build with:
>
>     ./configure --with-dbmliborder=gdbm:ndbm

We need this workaround in GitHub Actions, otherwise the tests fail.
(cherry picked from commit 850189a)

Co-authored-by: Łukasz Langa <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants