[issue2466] os.path.ismount doesn't work for mounts the user doesn't have permission to see

2016-06-03 Thread Robin Roth
Robin Roth added the comment: Based on the review by SilentGhost I removed all refactoring-changes and only left the one line needed for the fix. -- Added file: http://bugs.python.org/file43140/minimal_fix_ismount_directory_not_readable.patch

[issue2466] os.path.ismount doesn't work for mounts the user doesn't have permission to see

2016-04-22 Thread Robin Roth
Robin Roth added the comment: Any progress on merging this? The fix is simple, there is a test; anything else I can do to help? Ansible integrated the patch posted here as a workaround of an issue this caused. So there was some external review of the fix. See https://github.com/ansible/ansible

[issue2466] os.path.ismount doesn't work for mounts the user doesn't have permission to see

2015-12-17 Thread Robin Roth
Robin Roth added the comment: any comments/updates on merging this? -- ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue2466> ___ ___

[issue2466] os.path.ismount doesn't work for mounts the user doesn't have permission to see

2015-11-14 Thread Robin Roth
Robin Roth added the comment: Antoine's suggestion does not work, because "dirname" does not cover enough cases (for example trailing slash, possibly more). As suggested by him I now use realpath (instead of abspath). I can't come up with a symlink-situation that is broken with th

[issue2466] os.path.ismount doesn't work for mounts the user doesn't have permission to see

2015-10-01 Thread Robin Roth
Robin Roth added the comment: This bug is still present and annoying. For me it appeared when running ismount on a nfs-mounted directory, when the user who runs python has no read permission inside the directory. Therefore the lstat on /mntdir/.. fails. Attached a patch that fixes