Issue 35802: os.stat / os.lstat always present, but code checks hastattr(os, 'stat') / hasattr(os, 'lstat') (original) (raw)

This issue has been migrated to GitHub: https://github.com/python/cpython/issues/79983

classification

process

Created on 2019-01-21 21:42 by Anthony Sottile, last changed 2022-04-11 14:59 by admin. This issue is now closed.

Messages (5)
msg334182 - (view)	Author: Anthony Sottile (Anthony Sottile) *	Date: 2019-01-21 21:42
Unless I'm reading incorrectly: https://github.com/python/cpython/blob/7a2368063f25746d4008a74aca0dc0b82f86ff7b/Modules/clinic/posixmodule.c.h#L30-L31 https://github.com/python/cpython/blob/7a2368063f25746d4008a74aca0dc0b82f86ff7b/Modules/clinic/posixmodule.c.h#L68-L69
msg334184 - (view)	Author: Anthony Sottile (Anthony Sottile) *	Date: 2019-01-21 21:50
looks true for os.chmod as well: https://github.com/python/cpython/blob/7a2368063f25746d4008a74aca0dc0b82f86ff7b/Modules/clinic/posixmodule.c.h#L327-L328
msg334226 - (view)	Author: Gregory P. Smith (gregory.p.smith) *	Date: 2019-01-22 18:34
I suspect these conditionals are very old and came from times when we supported some platforms that did not have these APIs. Are they present on all CPython supported platforms today? Windows is probably the only one left to verify. BSD/Linux/Un*xes and modern macOS are POSIX. macOS <= 9 (classic) has long since died.
msg334229 - (view)	Author: Anthony Sottile (Anthony Sottile) *	Date: 2019-01-22 19:23
yep! did my due diligence there, you can check my work on https://github.com/python/cpython/pull/11643 all platforms have these functions since `posixmodule.c` is always compiled and the functions in question are not guarded by preprocessor directives in any way
msg336584 - (view)	Author: Giampaolo Rodola' (giampaolo.rodola) *	Date: 2019-02-26 00:07
A BB failed. Looks like it's unrelated with this change: https://buildbot.python.org/all/#/builders/99/builds/2198