Title: Add missing pip in $PATH on Cygwin CI by EliahKagan · Pull Request #1784 · gitpython-developers/GitPython · GitHub
Open Graph Title: Add missing pip in $PATH on Cygwin CI by EliahKagan · Pull Request #1784 · gitpython-developers/GitPython
X Title: Add missing pip in $PATH on Cygwin CI by EliahKagan · Pull Request #1784 · gitpython-developers/GitPython
Description: This makes a pip -> pip3 symlink in /usr/bin in a new step prior to the first step that runs the pip command. Using pip3, pip3.9, or a command like python -m pip would work, but this allows the Cygwin workflow to continue using the same installation commands as the main testing workflow. Adding this fixes two problems: When the pip version installed by the python39-pip Cygwin package is current, so that upgrading pip doesn't install any new command, no pip command is created in /usr/local. This has happened not to be the case for a long time, which is why the Cygwin workflow was able to pass. (That the recent failures started at the merge of #1783 turns out to be a coincidence: rerunning jobs on prior commits has the failure, as does experimentally reverting it.) Even when the pip version installed by python39-pip is behind the latest available version, pip is still used before being upgraded to check if setuptools is installed, to decide whether to upgrade it. This is to keep similar steps in the two testing workflows similar, since the Cygwin workflow only uses Python 3.9, which always has setuptools. Because pip was never in $PATH in that step, the Cygwin workflow wrongly refrained from trying to upgrade setuptools. When the "Update PyPA packages" step does find a newer version of pip to upgrade to, it installs it in /usr/local/bin, which we have in $PATH before /usr/bin, so the upgraded version, when present, will still be preferred in subsequent commands, as before. Running pip on Cygwin when it may not be in $PATH -- and, for one step, never is -- was a bug introduced in e8956e5 (#1709). Before that, pip still was not always available, but it was not used. This change fixes the bug by making sure pip is always available. This pull request also changes setuptools detection to be more precise. It's fairly unlikely that the old way would have misdetected setuptools, but on Cygwin it was passing multiple setuptools arguments to pip install, which made no behavioral difference but made the GitHub Actions log confusing.
Open Graph Description: This makes a pip -> pip3 symlink in /usr/bin in a new step prior to the first step that runs the pip command. Using pip3, pip3.9, or a command like python -m pip would work, but this allows the ...
X Description: This makes a pip -> pip3 symlink in /usr/bin in a new step prior to the first step that runs the pip command. Using pip3, pip3.9, or a command like python -m pip would work, but this allows ...
Opengraph URL: https://github.com/gitpython-developers/GitPython/pull/1784
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:09de1744-9f89-764b-330a-c906b5ecd94f |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 8BBC:6A14:3544B59:35EFAA0:69685013 |
| html-safe-nonce | e849b48a6a8d0933ec9f0f4209d650fe1a7851ca38c7b76aceeb65be913fcbd8 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4QkJDOjZBMTQ6MzU0NEI1OTozNUVGQUEwOjY5Njg1MDEzIiwidmlzaXRvcl9pZCI6IjYzNjk1NzYxMzExMjMyMzY4ODMiLCJyZWdpb25fZWRnZSI6InNlYSIsInJlZ2lvbl9yZW5kZXIiOiJzZWEifQ== |
| visitor-hmac | d535e91b93db0f54d3a2c4973a2114adb07b6278e4ad5048c56c4e449e68481f |
| hovercard-subject-tag | pull_request:1655847002 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/gitpython-developers/GitPython/pull/1784/files |
| twitter:image | https://avatars.githubusercontent.com/u/1771172?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/1771172?s=400&v=4 |
| og:image:alt | This makes a pip -> pip3 symlink in /usr/bin in a new step prior to the first step that runs the pip command. Using pip3, pip3.9, or a command like python -m pip would work, but this allows the ... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | f16c57f41ed243e5b4dfe9b9bcd6828bd83080b1b6dbb4ff239bbe9745f12e0c |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/gitpython-developers/GitPython git https://github.com/gitpython-developers/GitPython.git |
| octolytics-dimension-user_id | 503709 |
| octolytics-dimension-user_login | gitpython-developers |
| octolytics-dimension-repository_id | 1126087 |
| octolytics-dimension-repository_nwo | gitpython-developers/GitPython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 1126087 |
| octolytics-dimension-repository_network_root_nwo | gitpython-developers/GitPython |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | cfa7062cc6d4fe8fcb156bd33f4c97bd3b2470af |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width