Commit fea8f98
committed
minor symfony#58153 [Filesystem] Add a warning about
This PR was submitted for the 7.2 branch but it was merged into the 5.4 branch instead.
Discussion
----------
[Filesystem] Add a warning about `chown()` and `chgrp()` on Windows
| Q | A
| ------------- | ---
| Branch? | 5.4
| Bug fix? | no
| New feature? | no
| Deprecations? | no
| Issues | -
| License | MIT
`chmod()` and `chgrp()` are not supported on Windows (see [this](https://github.com/php/php-src/blob/0d616d1b82ca7609a7ece762cb8688d9bed74370/ext/standard/tests/file/chgrp.phpt#L6) and [this](https://github.com/php/php-src/blob/0d616d1b82ca7609a7ece762cb8688d9bed74370/ext/standard/tests/file/chown.phpt#L6)). I propose to skip the logic of these methods when running on Windows.
Commits
-------
fa3fd9d [Filesystem] Add a warning about `chown()` and `chgrp()` on Windowschown() and chgrp() on Windows (alexandre-daubois)1 file changed
+6
-0
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
233 | 233 | | |
234 | 234 | | |
235 | 235 | | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
236 | 239 | | |
237 | 240 | | |
238 | 241 | | |
| |||
260 | 263 | | |
261 | 264 | | |
262 | 265 | | |
| 266 | + | |
| 267 | + | |
| 268 | + | |
263 | 269 | | |
264 | 270 | | |
265 | 271 | | |
| |||
0 commit comments