-
Notifications
You must be signed in to change notification settings - Fork 181
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
Trigger warning when trying to check hidden file status on PyPy #238
Conversation
48074c1
to
4082eb4
Compare
4082eb4
to
38d5b0c
Compare
6ded5f3
to
a061c5a
Compare
@bollwyvl (or others) - can you review this? |
isn't this going to solve the problem ? https://foss.heptapod.net/pypy/pypy/-/issues/3469 |
It looks like it will in the next pypy release. Having this warning in will still help people that are on older releases, though. |
@@ -275,6 +275,7 @@ def is_file_hidden_win(abs_path, stat_res=None): | |||
# allow AttributeError on PyPy for Windows | |||
# 'stat_result' object has no attribute 'st_file_attributes' | |||
# https://foss.heptapod.net/pypy/pypy/-/issues/3469 | |||
warnings.warn("hidden files are not detectable on this system, so no file will be marked as hidden.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe we have/open a dedicated issue and link to it from here?
LGTM, modulo the small docs items... but since we anticipate it being fixed upstream soon anyway... |
3eb006a
to
c79b7e0
Compare
Thanks everyone for your comments and reviews! |
As discussed in #235, perhaps the best thing to do to give best-effort support for pypy and help users to understand the current limitations is to trigger a warning when we know we cannot give good results.