Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit8dc4cb0

Browse files
committed
Use Python 3.9.16 on Cygwin CI
The latest currently packaged version of Python 3.9 for Cygwin is3.9.18 (provided by the Cygwin package python39 at version3.9.18-1). That version, at least as we are using it, has a problemwhere pip has begun to block on some PyPI package downloads.In73ebcfa (EliahKagan#2), I worked around this problem bydowngrading the minor version of Python to 3.8. But it is better touse 3.9 if we can, since it is currently the latest minor versionof Python in the Cygwin repositories, and also because (relating tothat) it is used more often, and thus probably used more often withGitPython, than 3.8.This upgrades Python on Cygwin but not all the way. It upgrades itto the latest (or latest currently available) patch version of 3.9packaged for Cygwin of those that strictly precede 3.9.18 where theproblem occurs. That version is 3.9.16, provided by the Cygwinpackage python39 at version 3.9.16-1.This version may eventually no longer be available for downloadfrom Cygwin's repositories, so hopefully a real solution or betterworkaround will be found by then, or perhaps a future update to thepackage itself will fix the problem.I also tried some more other stuff since finding 3.8 to work in73ebcfa. Changes since then are listed below. They can be examinedin full detail, with individual diffs and CI results, atEliahKagan#3.* Try Python 3.9 with other details the sameChanging it to Python 3.8 worked, but I want to check that it wasactually the use of Python 3.8, rather than other seemingly smallchanges made to support using Python 3.8, that made the difference.* Revert "Try Python 3.9 with other details the same"This reverts commitb55cbfb.* Try 3.9 again, with both python39=3.9.16-1 python39-pip=23.0.1-1* Back to 3.8; try another GitHub ActionPython 3.8 worked with cygwin-install-action, but I want to makethe change to setup-cygwin by itself first before trying it with3.9, in case I am using setup-cygwin incorrectly.* Try 3.9 with this setup-cygwin action* Try pinning with setup-cygwin* Try not pinning, but no -U for PyPA, with setup-cygwinPinning and skipping -U for PyPA packages worked. Let's see if itwas really pinning that made the difference.* Try pinning just python39-pipPinning works, and merely omitting the -U for PyPA package doesn't.Examining the output of runs that used install-cygwin-action andattemped pinning Cygwin package versions shows newer versions wereinstalled, whereas pinning is really happening with setup-cygwin.This tries pinning just the Cygwin package for pip, rather than forPython 3.9. I don't expect this to work.* Try pinning just python39=3.9.16-1And not pip, but this does not add back the -U for PyPA yet.* Add back -U for PyPA packages* Try pinning python39=3.9.16-1 with old action/everythingThis is extremely unlikely to work, I just want to check.* Try just setup-cygwin and pinning python39=3.9.16-1That is, this puts back all the other stuff the way it was on themain branch when the breakage occurred, besides changing fromcygwin-install-action to setup-cygwin to make pinning work andusing it to get version 3.9.16-1 of the Cygwin python39 package.
1 parent73ebcfa commit8dc4cb0

File tree

1 file changed

+16
-12
lines changed

1 file changed

+16
-12
lines changed

‎.github/workflows/cygwin-test.yml

Lines changed: 16 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ jobs:
1515

1616
defaults:
1717
run:
18-
shell:C:\cygwin\bin\bash.exe --login --norc -eo pipefail -o igncr "{0}"
18+
shell:C:\tools\cygwin\bin\bash.exe --login --norc -eo pipefail -o igncr "{0}"
1919

2020
steps:
2121
-name:Force LF line endings
@@ -27,11 +27,10 @@ jobs:
2727
with:
2828
fetch-depth:0
2929

30-
-name:Install Cygwin
31-
uses:cygwin/cygwin-install-action@v4
30+
-name:Set up Cygwin
31+
uses:egor-tensin/setup-cygwin@v4
3232
with:
33-
packages:python38 python38-pip python38-virtualenv git
34-
add-to-path:false# No need to change $PATH outside the Cygwin environment.
33+
packages:python39=3.9.16-1 python39-pip python39-virtualenv git
3534

3635
-name:Arrange for verbose output
3736
run:|
@@ -55,23 +54,28 @@ jobs:
5554
# and cause subsequent tests to fail
5655
cat test/fixtures/.gitconfig >> ~/.gitconfig
5756
57+
-name:Ensure the "pip" command is available
58+
run:|
59+
# This is used unless, and before, an updated pip is installed.
60+
ln -s pip3 /usr/bin/pip
61+
5862
-name:Update PyPA packages
5963
run:|
60-
# Get the latest pip,setuptools, andwheel.
61-
python3.8 -m pip install -U pip setuptools wheel
64+
# Get the latest pip,wheel, andprior to Python 3.12, setuptools.
65+
python -m pip install -U pip$(pip freeze --all | grep -ow ^setuptools) wheel
6266
6367
-name:Install project and test dependencies
6468
run:|
65-
python3.8 -mpip install ".[test]"
69+
pip install ".[test]"
6670
6771
-name:Show version and platform information
6872
run:|
6973
uname -a
70-
command -v gitpython3.8
74+
command -v gitpython
7175
git version
72-
python3.8 --version
73-
python3.8 -c 'import os, sys; print(f"sys.platform={sys.platform!r}, os.name={os.name!r}")'
76+
python --version
77+
python -c 'import os, sys; print(f"sys.platform={sys.platform!r}, os.name={os.name!r}")'
7478
7579
-name:Test with pytest
7680
run:|
77-
python3.8 -mpytest --color=yes -p no:sugar --instafail -vv
81+
pytest --color=yes -p no:sugar --instafail -vv

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp