Skip to content

Comments

python@3.8: make test run on ARM#64872

Closed
claui wants to merge 1 commit intoHomebrew:masterfrom
claui:arm-test-python3.8
Closed

python@3.8: make test run on ARM#64872
claui wants to merge 1 commit intoHomebrew:masterfrom
claui:arm-test-python3.8

Conversation

@claui
Copy link
Contributor

@claui claui commented Nov 15, 2020

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

Same thing as #64869 but for python@3.8.

The test block uses the _gdbm module as an example for an external
loadable module. That module is not part of the natively built ARM
Python though.

Replace _gdbm with _dbm, which is present in either architecture and
should serve just the same purpose.
@claui claui added 11 Big Sur is specifically affected CI-syntax-only Change only affects brew syntax, not the install. Only run syntax CI. labels Nov 15, 2020
@BrewTestBot BrewTestBot added the legacy Relates to a versioned @ formula label Nov 15, 2020
@BrewTestBot
Copy link
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@BrewTestBot BrewTestBot added the stale No recent activity label Dec 7, 2020
@fxcoudert fxcoudert closed this Dec 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

11 Big Sur is specifically affected CI-syntax-only Change only affects brew syntax, not the install. Only run syntax CI. legacy Relates to a versioned @ formula stale No recent activity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants