[issue15217] os.listdir is missing in os.supports_dir_fd

2012-06-29 Thread Larry Hastings
Larry Hastings added the comment: WFM. That is, unless people want to bring up additional only-tangentially-related topics to annoy Hynek ;-) How about those ? They're really having a year of it, aren't they! -- ___ Python tracker

[issue15217] os.listdir is missing in os.supports_dir_fd

2012-06-29 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: > Storchaka: please take it up with Antoine, he's the defender of the realm for > POSIX-functions-are-atomic iirc. I'd be happy with dir_fd for os.listdir, > though it may be too late for 3.3 anyway. I know that this is possible (and relatively easy), but

[issue15217] os.listdir is missing in os.supports_dir_fd

2012-06-28 Thread Hynek Schlawack
Hynek Schlawack added the comment: Just in my defense: Larry suggested in msg164245 to me to check for listdir in supports_dir_fd so I just assumed it's broken as it didn't work. I'm totally fine with the current behavior, the discussion about dir_fd support in listdir isn't what I intended w

[issue15217] os.listdir is missing in os.supports_dir_fd

2012-06-28 Thread Larry Hastings
Larry Hastings added the comment: Storchaka: please take it up with Antoine, he's the defender of the realm for POSIX-functions-are-atomic iirc. I'd be happy with dir_fd for os.listdir, though it may be too late for 3.3 anyway. -- nosy: +pitrou __

[issue15217] os.listdir is missing in os.supports_dir_fd

2012-06-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: > I proposed making os.listdir accept dir_fd, and internally use open and > listdir to emulate the behavior. Hmm, I understood the issue as the request for adding dir_fd argument to os.listdir. openat+fdopendir+readdir+closedir+close or, in Python, def list

[issue15217] os.listdir is missing in os.supports_dir_fd

2012-06-28 Thread Larry Hastings
Larry Hastings added the comment: I'm closing this. os.listdir should not be listed in supports_dir_fd, because it has no dir_fd parameter. For reference, we discussed all this previously in issue #15176. -- resolution: -> invalid stage: needs patch -> committed/rejected status: ope

[issue15217] os.listdir is missing in os.supports_dir_fd

2012-06-28 Thread Hynek Schlawack
Hynek Schlawack added the comment: Ah sorry, that wasn't in the mail (stupid >>>). -- ___ Python tracker ___ ___ Python-bugs-list mai

[issue15217] os.listdir is missing in os.supports_dir_fd

2012-06-28 Thread Larry Hastings
Larry Hastings added the comment: Like I said: use os.supports_fd. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubs

[issue15217] os.listdir is missing in os.supports_dir_fd

2012-06-28 Thread Hynek Schlawack
Hynek Schlawack added the comment: I don't really care about the name but I'd really like some way to check whether listdir supports fds. -- ___ Python tracker ___

[issue15217] os.listdir is missing in os.supports_dir_fd

2012-06-28 Thread Larry Hastings
Larry Hastings added the comment: I want to mark this as wontfix. >>> os.listdir in os.supports_fd True The concept we're struggling with here: is the "fd" you pass in to os.listdir a "dir_fd"? I claim that it isn't. I'm trying to enforce the concept, in both the documentation and the code

[issue15217] os.listdir is missing in os.supports_dir_fd

2012-06-28 Thread Arfrever Frehtes Taifersar Arahesis
Changes by Arfrever Frehtes Taifersar Arahesis : -- nosy: +Arfrever ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscri

[issue15217] os.listdir is missing in os.supports_dir_fd

2012-06-28 Thread Serhiy Storchaka
Changes by Serhiy Storchaka : -- nosy: +storchaka ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.p

[issue15217] os.listdir is missing in os.supports_dir_fd

2012-06-28 Thread Hynek Schlawack
New submission from Hynek Schlawack : >>> n = os.open('/tmp', os.O_RDONLY) >>> l = os.listdir(n) >>> os.listdir in os.supports_dir_fd False -- assignee: larry components: Library (Lib) messages: 164249 nosy: hynek, larry priority: release blocker severity: normal stage: needs patch statu