actions/python-versions
3
0
Fork 0
mirror of https://github.com/actions/python-versions.git synced 2025-04-04 22:39:40 +00:00
Code Actions 2 Packages Releases Activity

Fix for python 3.7.17 macOS (#236)

Browse Source
This commit is contained in:
Dmitry Shibanov
2023-06-23 14:39:13 +02:00
committed by GitHub
parent 5a451d6492
commit c1cf5de988
2 changed files with 14 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
builders/macos-python-builder.psm1
View File

@ -31,6 +31,14 @@ class macOSPythonBuilder : NixPythonBuilder {
.SYNOPSIS
Prepare system environment by installing dependencies and required packages.
#>
if ($this.Version -eq "3.7.17") {
# We have preinstalled ncurses and readLine on the hoster runners. But we need to install bzip2 for
# setting up an environemnt
# If we get any issues realted to ncurses or readline we can try to run this command
# brew install ncurses readline
Execute-Command -Command "brew install bzip2"
}
}
[void] Configure() {
@ -74,7 +82,12 @@ class macOSPythonBuilder : NixPythonBuilder {
if ($this.Version -gt "3.7.12") {
$configureString += " --with-tcltk-includes='-I /usr/local/opt/tcl-tk/include' --with-tcltk-libs='-L/usr/local/opt/tcl-tk/lib -ltcl8.6 -ltk8.6'"
}
}
if ($this.Version -eq "3.7.17") {
$env:LDFLAGS += " -L$(brew --prefix bzip2)/lib -L$(brew --prefix readline)/lib -L$(brew --prefix ncurses)/lib"
$env:CFLAGS += " -I$(brew --prefix bzip2)/include -I$(brew --prefix readline)/include -I$(brew --prefix ncurses)/include"
}
}
### Compile with support of loadable sqlite extensions. Unavailable for Python 2.*

5
tests/sources/python-modules.py
View File

@ -248,11 +248,6 @@ if sys.version_info >= (3, 7):
'dataclasses',
])
if (sys.version_info.major, sys.version_info.minor, sys.version_info.micro) == (3, 7, 17):
standard_library.remove('bz2')
standard_library.remove('curses')
standard_library.remove('readline')
# 'macpath' module has been removed from Python 3.8
if sys.version_info > (3, 7):
standard_library.remove('macpath')