Title: Fix typo in _get_exe_extensions PATHEXT fallback by EliahKagan · Pull Request #1890 · gitpython-developers/GitPython · GitHub
Open Graph Title: Fix typo in _get_exe_extensions PATHEXT fallback by EliahKagan · Pull Request #1890 · gitpython-developers/GitPython
X Title: Fix typo in _get_exe_extensions PATHEXT fallback by EliahKagan · Pull Request #1890 · gitpython-developers/GitPython
Description: This applies two minor Cygwin-related clarifications, one of which is theoretically a bugfix though it affects conceptually non-public code that is currently not used in a way that could invoke the bug. Some specific details are in commit messages. The second commit, 988d97b, which fixes COM to .COM in the fallback for the PATHEXT environment variable--whose omission is almost certainly a typo, going back to the function's introduction in e6e23ed (#533)--is theoretically a bugfix. But as noted there, this code is only called when the Python interpreter is a native Windows program, but the function it is a part of is only called when the Python interpreter is not a native Windows program. (Also, it would be pretty unusual for the git executable to have a .com extension.) There are some further changes related to Cygwin detection that may make sense to make in the future, some of which might be best to make after adding relevant tests. The ones that are narrowly related to the change here are that: This value of PATHEXT here is still unusual, because order matters and this lists .BAT before .COM and .EXE, and because it is missing .CMD, which is part of what a minimal value for it should typically contain. I am uncertain if this strange order is intentional. It may also make sense for the list to include .VBS, .JS, .WS, and .MSC for compatibility with the fallback behavior of the Windows cmd.exe shell when PATHEXT is unset, which is what shutil.which does. (I have verified the result of Eryk Sun's debugger check in WinDbg on recent builds of Windows 10 and Windows 11; they are the same.) py_where should really be removed and replaced either with a call to shutil.which, which did not exist at the time py_where was introduced and would almost certainly have been used at that time if it had--or with other logic that works an altogether different way. Which should be done depends on the intent of the public Git.is_cygwin_git class method, which is the only code in GitPython that uses the is_cygwin_git function in git/util.py, for which the py_where helper exists. This PR doesn't try to cover any of that. It's really just applying some clarifications that didn't seem like they fit in any other PR that I would do soon. Regarding the last point about Git.is_cygwin_git, if the goal is to detect Cygwin git even on native Windows systems, then the code it uses will have to be significantly revised. That is because currently it always returns False on such systems. In addition, as noted in the py_where docstring and elsewhere, shell and non-shell path search differ from each other on native Windows, such that neither the nonpublic ad-hoc py_where nor the public general shutil.which will reliably tell us where git is. However, if a path search doesn't need to be done on native Windows, then it can be replaced with shutil.which unless some use has come to depend on (perhaps accidentally introduced) distinctive behavior of py_where. I plan to open an issue about that larger matter, which I am unlikely to work on anytime soon, but that perhaps someone would be interested to take on. (If you know what Git.is_cygwin_git's semantics are supposed to be, then I can make use of that information in the forthcoming issue, but I can still open it even if its intent is currently uncertain.)
Open Graph Description: This applies two minor Cygwin-related clarifications, one of which is theoretically a bugfix though it affects conceptually non-public code that is currently not used in a way that could invoke the...
X Description: This applies two minor Cygwin-related clarifications, one of which is theoretically a bugfix though it affects conceptually non-public code that is currently not used in a way that could invoke the...
Opengraph URL: https://github.com/gitpython-developers/GitPython/pull/1890
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:46f6d372-4f42-54fc-d2fa-b471db7f9078 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 9896:2542A6:168057E:1EBE3D5:69694FAF |
| html-safe-nonce | 987bfc2e19aa4e260883a653c8a972f00a60aae3e82dd29699d90f54ce99e31f |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5ODk2OjI1NDJBNjoxNjgwNTdFOjFFQkUzRDU6Njk2OTRGQUYiLCJ2aXNpdG9yX2lkIjoiMjIzMzY2NjE1NDI5MjQwMDA0NyIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 16775961afeba789cf0c2b3f96e6932999393fbb5eef86bf6257cf3bef9c1ae3 |
| hovercard-subject-tag | pull_request:1799518154 |
| 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/1890/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 applies two minor Cygwin-related clarifications, one of which is theoretically a bugfix though it affects conceptually non-public code that is currently not used in a way that could invoke the... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 54182691a21263b584d2e600b758e081b0ff1d10ffc0d2eefa51cf754b43b51d |
| 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 | d69ac0477df0f87da03b8b06cebd187012d7a930 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width