Eryk Sun added the comment:
With increasing use of os.access() in shutil and tempfile, it would be nice to
have a real implementation of os.access() for Windows.
Instead of manually evaluating the security of the file/directory, as
issue2528.2.patch attempts to do, I'd rather just open the
Changes by Paul Moore p.f.mo...@gmail.com:
--
nosy: +paul.moore
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2528
___
___
Python-bugs-list
eryksun added the comment:
In msg243815 you asked me to look over this patch. I hope this helps.
For GetFileSecurity you need to also request LABEL_SECURITY_INFORMATION. To
test this in Vista+, use a file in the root directory of the system drive. This
will inherit a high integrity level w/
Tim Golden added the comment:
Thanks for the very thorough review. This isn't going to make it into
3.5, but I'll rework it in the light of your comments and see if people
are happy with it in the optional argument variation.
--
___
Python tracker
Mark Lawrence added the comment:
The solution proposed here could help resolve #22107.
--
nosy: +BreamoreBoy
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2528
___
Changes by Serhiy Storchaka storch...@gmail.com:
--
nosy: +serhiy.storchaka
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2528
___
___
Changes by Mark Lawrence breamore...@yahoo.co.uk:
--
nosy: -BreamoreBoy
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2528
___
___
Tim Golden added the comment:
Here's an updated patch against trunk with tests doc changes
--
status: languishing - open
Added file: http://bugs.python.org/file31165/issue2528.2.patch
___
Python tracker rep...@bugs.python.org
Tim Golden added the comment:
... and to answer Amaury's question in msg109871 it creates a reasonable
consistency between the results of os.access and the user's actual ability to
read / write a file. eg, you might have no permissions whatsoever on the file
but as long as it wasn't
Changes by Tim Golden m...@timgolden.me.uk:
Removed file: http://bugs.python.org/file9919/os_access-r62091.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2528
___
Christian Heimes added the comment:
Do you want to provide an updated patch for 3.4?
--
nosy: +christian.heimes
status: open - languishing
versions: +Python 3.4 -Python 3.2
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2528
Changes by Tim Golden m...@timgolden.me.uk:
--
assignee: - tim.golden
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2528
___
___
Python-bugs-list
Mark Lawrence breamore...@yahoo.co.uk added the comment:
Tim, do you want more time to think about this, could we close as won't fix
or what?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2528
Mark Lawrence breamore...@yahoo.co.uk added the comment:
A quick look tells me that the patch seems clean. However it involves changes
to posixmodule.c and I don't (yet) want to get involved with doing builds, so
could someone else please give this a try.
--
nosy: +BreamoreBoy
Amaury Forgeot d'Arc amaur...@gmail.com added the comment:
But what are the benefits of this change?
--
nosy: +amaury.forgeotdarc
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2528
___
Tim Golden m...@timgolden.me.uk added the comment:
Although I'm the implementer of the patch (the concept
was discussed way back on c.l.py after a naive poster's
original request) I'm probably +0 myself. It's an attempt
to replace os.access' next-to-useless behaviour on Windows
with something
Changes by Daniel Diniz aja...@gmail.com:
--
components: +Windows
priority: - normal
stage: - patch review
type: - feature request
versions: +Python 2.7, Python 3.2 -Python 2.6
___
Python tracker rep...@bugs.python.org
New submission from Tim Golden [EMAIL PROTECTED]:
At present, os.access under Windows simply calls GetFileAttributes to
determine the readonly attribute (ignoring directories). The patch
attached combines this with the use of the AccessCheck API to compare
the user's permissions with those
18 matches
Mail list logo