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

CI: Install git on Cygwin CI runner #211

Merged
merged 4 commits into from
Jan 7, 2024
Merged

Conversation

DWesl
Copy link
Contributor

@DWesl DWesl commented Aug 6, 2023

Cygwin pip now has a chance to resolve everything on the command line.

Git commands will fail because the repository was created by Windows git, but this should work until the above problems are solved.

Cygwin pip now has a chance to resolve everything on the command line.  

It won't be able to resolve dependencies, due to something pulling in Rust, but it'll get to the point where pip points out that it is not pip's fault that CI doesn't have Rust compilers for Cygwin
jaraco.text depends on inflect; inflect>=6.0.0 depends on Rust.
Add an additional rule installing a version of the dependency that will actually install.
@DWesl
Copy link
Contributor Author

DWesl commented Aug 6, 2023

It's possible to fix the actions/checkout post-run failure, but that doesn't seem to be affecting anything, so I'm leaving it unless someone tells me otherwise.

@DWesl
Copy link
Contributor Author

DWesl commented Aug 6, 2023

I don't understand why 3.7 fails to install the pytest branch, but I suspect that's unrelated.

@DWesl
Copy link
Contributor Author

DWesl commented Aug 13, 2023

It seems pytest dropped support for Python 3.7 in pytest-dev/pytest#11152 before the fix for the MRO bug went in:
https://github.com/RonnyPfannschmidt/pytest/blob/fix-10447-maker-mro-order-needs-reverse/setup.cfg#L53
Should that batch of tests get bumped to using Python 3.8 instead?

@DWesl
Copy link
Contributor Author

DWesl commented Aug 17, 2023

The banner at the top of https://docs.python.org/3.7/ suggests that 3.7 is EOL, which also suggests moving the failing CI runs to 3.8.

@jaraco jaraco force-pushed the main branch 3 times, most recently from 12769e5 to d23e28a Compare November 5, 2023 15:55
@jaraco
Copy link
Member

jaraco commented Jan 7, 2024

Unfortunately, cygwin got disabled due to incompatibility with Rust dependencies. I was hopeful that the pinning of inflect might help, so I've updated it per the suggestion above, but it looks like the additional reliance on ruff is probably preventing re-enabling Cygwin.

@jaraco jaraco merged commit 53bfdb2 into pypa:main Jan 7, 2024
16 checks passed
@DWesl DWesl deleted the fix-cygwin-ci branch March 29, 2024 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants