Title: Add the `.git` subdir as another `safe.directory` on Cygwin CI by EliahKagan · Pull Request #1916 · gitpython-developers/GitPython · GitHub
Open Graph Title: Add the `.git` subdir as another `safe.directory` on Cygwin CI by EliahKagan · Pull Request #1916 · gitpython-developers/GitPython
X Title: Add the `.git` subdir as another `safe.directory` on Cygwin CI by EliahKagan · Pull Request #1916 · gitpython-developers/GitPython
Description: As shown in this run on my fork, the CI test job for Cygwin is failing now. This fixes that. The first commit in this pull request just demonstrates that the failure is due to the upgrade of the Cygwin git package from 2.43.0-1 to 2.45.1-1. Although that commit makes test pass, I recommend against following that approach, mainly because the new version contains multiple security updates (coming in with the upstream 2.45.1 version, not with any downstream patches), but also because the older version will eventually be dropped from the Cygwin repositories. The better workaround is in the second commit here, which adds the .git subdirectory of the cloned GitPython directory as a value of the multi-valued safe.directory Git configuration variable. Its parent directory is already added, which was previously sufficient, but not anymore. I suspect that, rather than this being any bug in the downstream package, this is actually the correct behavior due to one of the several security fixes in the new version of Git, though I have not verified that. So maybe this is really not a workaround, but a permanent fix. I believe the reason there is no need to modify any other workflow is that the other workflows don't need to add any safe.directory paths at all: they either clone the repository with the necessary ownership in the first place or, in the case of the Alpine Linux job, set the ownership with chown.
Open Graph Description: As shown in this run on my fork, the CI test job for Cygwin is failing now. This fixes that. The first commit in this pull request just demonstrates that the failure is due to the upgrade of the Cy...
X Description: As shown in this run on my fork, the CI test job for Cygwin is failing now. This fixes that. The first commit in this pull request just demonstrates that the failure is due to the upgrade of the Cy...
Opengraph URL: https://github.com/gitpython-developers/GitPython/pull/1916
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:33864d30-4e67-4fd1-9637-cc8ed2a07282 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | A12C:13107E:AE792B:EC277B:69686FB3 |
| html-safe-nonce | a62dccf38c740afd2b7bdbf71d1c988ce3c9fb522e69761423b8a688b28e888c |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBMTJDOjEzMTA3RTpBRTc5MkI6RUMyNzdCOjY5Njg2RkIzIiwidmlzaXRvcl9pZCI6IjM0Mzg3MzgyMjU1NDkwNDU2ODMiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 6fb934ddcebc282b1930d4319f311fb97dae85c0323adb1244c4330c80771d22 |
| hovercard-subject-tag | pull_request:1889104424 |
| 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/1916/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 | As shown in this run on my fork, the CI test job for Cygwin is failing now. This fixes that. The first commit in this pull request just demonstrates that the failure is due to the upgrade of the Cy... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 50f46dc2d6192249fd8ebf20e76c800f4f2596d4a5f3ab63dd63a754df154f54 |
| 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 |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | fef287f17234b4529a4b112a3d47fe8551e32ddd |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width