-
-
Notifications
You must be signed in to change notification settings - Fork 30.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test_os
fails on Windows
#118802
Comments
This is the same underlying issue as #118773, isn't it? |
Actually the test should be implemented differently by denying synchronize ( The fast path of |
Same root cause (localised account names), but different code. |
Yeah, this is known - it's why the test checks for either 0 or st_ino/dev, because if we take the faster path we'll get them. So eventually the test becomes redundant, but until then it helps verify that we aren't returning garbage in certain fields (which was the bug that prompted it).
This seems to work fine, PR incoming. |
Sorry, I git blamed to find the PR that added the test, and somehow I mixed up browser tabs and ended up looking at the original code instead of the latest code. I see that you changed it to use checks such as |
Bug report
Bug description:
CPython versions tested on:
CPython main branch
Operating systems tested on:
Windows
Linked PRs
The text was updated successfully, but these errors were encountered: