Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork939
Commite07d91a
committed
Drop claim about Cygwin not having git-daemon
On a current Cygwin system with git 2.39.0 (the latest versionoffered by the Cygwin package manager), git-daemon is present, withthe Cygwin path /usr/libexec/git-core/git-daemon.exe.In addition, the cygwin-test.yml workflow does not take any specialsteps to allow git-daemon to work, but all tests pass in it evenwithout skipping or xfailing tests that seem related to git-daemon.The git_daemon_launched function in test/lib/helper.py only invokesgit-daemon directly (rather than through "git daemon") when is_winevaluates true, which only happens on native Windows systems, notCygwin, which is treated the same as (other) Unix-like systems andstill works. So maybe Cygwin git-daemon was never a special case.Whether or not it was, the message about git-daemon needing to befindable in a PATH search is also under an is_win check, and thusis never shown on Cygwin. So I've removed the Cygwin part of thatmessage. (Because the path shown is a MinGW-style path, I have keptthe wording about that being for MinGW-git, even though it is nolonger needed to disambiguate the Cygwin case.)1 parent7d4f6c6 commite07d91a
2 files changed
+7
-11
lines changedLines changed: 1 addition & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
119 | 119 |
| |
120 | 120 |
| |
121 | 121 |
| |
122 |
| - | |
123 |
| - | |
| 122 | + | |
124 | 123 |
| |
125 | 124 |
| |
126 | 125 |
| |
|
Lines changed: 6 additions & 9 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
177 | 177 |
| |
178 | 178 |
| |
179 | 179 |
| |
180 |
| - | |
181 |
| - | |
182 |
| - | |
183 |
| - | |
184 |
| - | |
185 |
| - | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
186 | 184 |
| |
187 | 185 |
| |
188 | 186 |
| |
| |||
217 | 215 |
| |
218 | 216 |
| |
219 | 217 |
| |
220 |
| - | |
| 218 | + | |
221 | 219 |
| |
222 | 220 |
| |
223 | 221 |
| |
224 |
| - | |
225 |
| - | |
| 222 | + | |
226 | 223 |
| |
227 | 224 |
| |
228 | 225 |
| |
|
0 commit comments
Comments
(0)