[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Brian Curtin added the comment: Closed. I created #9332 for the remaining side issues. -- status: pending -> closed ___ Python tracker ___

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Amaury Forgeot d'Arc added the comment: All patches seems already applied. Should we close this issue? -- status: open -> pending ___ Python tracker ___ __

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Brian Curtin added the comment: Committed Jason's patch with Antoine's twist as r82743 after running on Arch Linux. Thanks for catching and looking into this stuff. -- ___ Python tracker ___

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Antoine Pitrou added the comment: > Hm, the patch could be more "pythonic". Something like: > > symlink_exception = (AttributeError,) > try: > symlink_exception += (NotImplementedError, WindowsError) > except NameError: > pass Probably better as: symlink_exception = (AttributeError, N

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Amaury Forgeot d'Arc added the comment: Hm, the patch could be more "pythonic". Something like: symlink_exception = (AttributeError,) try: symlink_exception += (NotImplementedError, WindowsError) except NameError: pass try: ... except symlink_exception: ... -- ___

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Changes by Jason R. Coombs : Removed file: http://bugs.python.org/file17920/smime.p7s ___ Python tracker ___ ___ Python-bugs-list mailing li

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Jason R. Coombs added the comment: Here's a patch to address the posix failures: - test_posixpath seems to have lost the creation of one test file. - WindowsError doesn't exist on other platforms, so it can't be caught directly (in tarfile.py). I've written a work-around, but I don't particula

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Amaury Forgeot d'Arc added the comment: Also, GetFinalPathNameByHandle() is called 5 times with VOLUME_NAME_DOS, and once with VOLUME_NAME_NT. This one looks suspect to me. [I noticed this because these symbols are not defined with the SDK shipped with VS8.0. I'll propose another patch for t

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Jason R. Coombs added the comment: I'm building now and will report back shortly... -- Added file: http://bugs.python.org/file17920/smime.p7s ___ Python tracker ___ smime.p7s

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Antoine Pitrou added the comment: This has broken all non-Windows buildbots. Can you take a look? -- nosy: +pitrou ___ Python tracker ___ _

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Brian Curtin added the comment: Committed in r82659. I'm leaving this open until a few other issues are fleshed out. 1. Document privilege escalation and/or expose some method to do so. 2. Test execution, e.g., buildbots Once I get a few more things off my plate I should be able to finish set

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Jason R. Coombs added the comment: It seems issue7443 discusses the cause of the transient failures. -- ___ Python tracker ___ ___

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Changes by Jason R. Coombs : Removed file: http://bugs.python.org/file16203/admin results.zip ___ Python tracker ___ ___ Python-bugs-list ma

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Changes by Jason R. Coombs : Removed file: http://bugs.python.org/file16202/guest results.zip ___ Python tracker ___ ___ Python-bugs-list ma

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Changes by Jason R. Coombs : Added file: http://bugs.python.org/file16835/guest results - patch 29.zip ___ Python tracker ___ ___ Python-bug

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Jason R. Coombs added the comment: With patch 29, I believe all patch-related test failures are corrected. The remaining failures in some modules (namely test_tarfile, test_glob, test_mailbox, test_warnings) can be demonstrated to occur in unpatched builds. Since some failures are transient,

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Changes by Jason R. Coombs : Added file: http://bugs.python.org/file16834/admin results - patch 29.zip ___ Python tracker ___ ___ Python-bug

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Changes by Jason R. Coombs : Added file: http://bugs.python.org/file16833/admin results - no patch.zip ___ Python tracker ___ ___ Python-bug

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Changes by Jason R. Coombs : Removed file: http://bugs.python.org/file16824/windows symlink draft 28.patch ___ Python tracker ___ ___ Python

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Changes by Jason R. Coombs : Removed file: http://bugs.python.org/file16821/windows symlink draft 28.patch ___ Python tracker ___ ___ Python

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Changes by Jason R. Coombs : Removed file: http://bugs.python.org/file16809/windows symlink draft 27.patch ___ Python tracker ___ ___ Python

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Changes by Jason R. Coombs : Added file: http://bugs.python.org/file16825/windows symlink draft 29.patch ___ Python tracker ___ ___ Python-b

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Jason R. Coombs added the comment: In the last patch, I addressed test_platform running a test under a symlinked Python. I then found the same error resulting from essentially the same code in test_sysconfig. So I refactored those methods into a generator method in test.support. This is patch

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Martin v. Löwis added the comment: The patch has now missed the window for 2.7, so backporting it won't be necessary. -- ___ Python tracker ___ __

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Jason R. Coombs added the comment: This patch includes a routine that adds the directory of the python executable to the path before attempting to run a symlinked executable (in test_platform). -- Added file: http://bugs.python.org/file16821/windows symlink draft 28.patch

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Changes by Jason R. Coombs : Removed file: http://bugs.python.org/file16761/windows symlink draft 26.patch ___ Python tracker ___ ___ Python

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Changes by Jason R. Coombs : Removed file: http://bugs.python.org/file16196/windows symlink draft 25.patch ___ Python tracker ___ ___ Python

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Jason R. Coombs added the comment: I've been fighting a failing test in test_platform (when test.support.can_symlink() is True). Turns out the problem is caused by the fact that the Python DLL cannot be resolved when the executable isn't in the same directory (which is the case when launched

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Jason R. Coombs added the comment: While trying to reproduce the transient test_tarfile errors, I found two more tests that appear to be failing when symlinks cannot be created, now skipped by patch 27. -- Added file: http://bugs.python.org/file16809/windows symlink draft 27.patch __

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Jason R. Coombs added the comment: My initial troubleshooting indicated to me that the intermittent test_tarfile problem exists independent of the symlink patch, so it was not relevant to this issue. I've tried to do some more thorough troubleshooting, and this continues to be my finding, an

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Jason R. Coombs added the comment: This latest patch (26) only merges the latest changes from the repo. -- Added file: http://bugs.python.org/file16761/windows symlink draft 26.patch ___ Python tracker _

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Brian Curtin added the comment: One last issue to solve before this goes in and I start the backport...test_tarfile.test_extract_hardlink is intermittently failing for me, which was an issue for Eric and I on an earlier version of Jason's patch. Sometimes it fails when run as a part of the wh

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

sorin added the comment: For me the patch worked. It would be really nice to have it in 2.7 - I really hate Python functions that are not working on Windows. -- ___ Python tracker __

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Eric Smith added the comment: I had some style issues at one point, but I haven't looked at it closely recently. I won't have time to look at this before next week, so proceed without me. -- ___ Python tracker

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Brian Curtin added the comment: Anyone opposed to this being checked in? Other than some style issues which I'll fix on checkin, I believe this is ready to go. I'd like to get it in so I can backport it to 2.7 before the beta on Saturday. -- assignee: eric.smith -> brian.curtin _

[issue1578269] Add os.symlink() and os.path.islink() support for Windows

Eric Smith added the comment: At the language summit we okay'd using ctypes in the tests for standard lib modules, specifically for this issue. -- ___ Python tracker ___